]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test-oomd-util: remove memory_pressure == 0 checks
authorAnita Zhang <the.anitazha@gmail.com>
Thu, 12 Nov 2020 21:53:20 +0000 (13:53 -0800)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Tue, 17 Nov 2020 09:08:32 +0000 (10:08 +0100)
test_oomd_cgroup_context_acquire_and_insert reads the live cgroup data used
by the unit test. Under certain conditions, the memory pressure for the cgroup
can be non-zero (although most of the time it is 0 since these tests don't
generate much pressure).

Since these values are too dependent on the state of the system, remove the
checks. The type used is always >= 0 and test-psi-util already unit tests that
PSI values are parsed correctly from files so this test is redundant anyways.

src/oom/test-oomd-util.c

index 68c498e2d1781b6a5269d2ee1c15b0d87afe0f44..5df57107f9cdde9167d2d7ccfef616547d9c784a 100644 (file)
@@ -104,10 +104,6 @@ static void test_oomd_cgroup_context_acquire_and_insert(void) {
         assert_se(oomd_cgroup_context_acquire(cgroup, &ctx) == 0);
 
         assert_se(streq(ctx->path, cgroup));
-        assert_se(ctx->memory_pressure.avg10 == 0);
-        assert_se(ctx->memory_pressure.avg60 == 0);
-        assert_se(ctx->memory_pressure.avg300 == 0);
-        assert_se(ctx->memory_pressure.total == 0);
         assert_se(ctx->current_memory_usage > 0);
         assert_se(ctx->memory_min == 0);
         assert_se(ctx->memory_low == 0);