[policies] fix LinuxPolicy performance regression (test suite)

Commit 285873a introduces a regression in the performance of
LinuxProfile class initialisation:

  commit 285873a4f753822a88d475a1b030ab622bf4c72e
  Author: Bryan Quigley <bryan.quigley@canonical.com>
  Date:   Tue Feb 11 15:03:16 2020 -0800

      [policies] Detect systemd use instead of hardcoding it

All the patch does is to switch from testing self.init to
determine whether to use the SystemdInit() class or the
basic InitSystem() one, to looking for '/run/systemd/system'
in the file system.

This has more broad side effects than it might seem since
the test suite uses LinuxPolicy() as a mock policy object
for a large number of tests. Since SystemdInit() calls out
to systemctl to obtain the state of the init system this
both increases the run time for the tests and causes high
resource use in systemd itself:

PID USER      PR  NI    VIRT    RES    SHR S  %CPU  %MEM     TIME+ COMMAND
  1 root      20   0  177196  13504   4512 R  92.7   0.1   5:24.04 systemd

This causes the run time for the Python2 and 3 test suite
(plus pep8/pycodestyle) to grow from ~5s to over 1m20s on
my test system:

285873a~1 real 0m5.683s
285873a   real 1m20.353s

Allow direct users of LinuxPolicy to override the init
system detection by specifying an init= kwarg initialised
to an InitSystem-like object directly, and use this in the
test suite to avoid the cost of initialising the full
SystemdInit() class.

Resolves: #1953

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/tests/option_tests.py b/tests/option_tests.py
index f0587367..4af16270 100644
--- a/tests/option_tests.py
+++ b/tests/option_tests.py
@@ -3,7 +3,7 @@
 import unittest
 
 from sos.plugins import Plugin
-from sos.policies import LinuxPolicy
+from sos.policies import LinuxPolicy, InitSystem
 
 
 class MockOptions(object):
@@ -18,7 +18,7 @@ class GlobalOptionTest(unittest.TestCase):
     def setUp(self):
         self.commons = {
             'sysroot': '/',
-            'policy': LinuxPolicy(),
+            'policy': LinuxPolicy(init=InitSystem()),
             'cmdlineopts': MockOptions()
         }
         self.plugin = Plugin(self.commons)