Skip to content

Commit

Permalink
Uncomment assertion
Browse files Browse the repository at this point in the history
  • Loading branch information
jsf9k committed May 28, 2024
1 parent 923f21c commit ca734a8
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
8 changes: 1 addition & 7 deletions molecule/default/tests/test_default_basic.py
Original file line number Diff line number Diff line change
Expand Up @@ -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."
4 changes: 3 additions & 1 deletion requirements-test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,6 @@ ansible-core<2.16.3
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

0 comments on commit ca734a8

Please sign in to comment.