Merge pull request #8 from cisagov/testing/uncomment-assertion

Uncomment assertion
cisagov · Jun 25, 2024 · 2c9236c · 2c9236c
2 parents 00700d3 + b780cad
commit 2c9236c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/molecule/default/tests/test_default_basic.py b/molecule/default/tests/test_default_basic.py
@@ -24,12 +24,6 @@ def test_packages(host):
 def test_services(host):
     """Verify that the expected services are present."""
     s = host.service("systemd-resolved")
-    # TODO - This assertion currently fails because of
-    # pytest-dev/pytest-testinfra#757.  Once
-    # pytest-dev/pytest-testinfra#754 has been merged and a new
-    # release is created the following line can be uncommented.
-    #
-    # See #3 for more details.
-    # assert s.exists, "systemd-resolved service does not exist."
+    assert s.exists, "systemd-resolved service does not exist."
     assert s.is_enabled, "systemd-resolved service is not enabled."
     assert s.is_running, "systemd-resolved service is not running."
diff --git a/requirements-test.txt b/requirements-test.txt
@@ -30,4 +30,6 @@ ansible-core>=2.16.7
 molecule>=5.0.1
 molecule-plugins[docker]
 pre-commit
-pytest-testinfra
+# pytest-testinfra contains a fix for SystemdService.exists that is
+# required by this role's Molecule test code.
+pytest-testinfra>=10.1.1