majic-ansible-roles Changeset

Changeset - 693d6960b1a0

Parent rev.

Child rev.

[Not reviewed]

0 2 0

Branko Majic (branko) - 4 years ago 2020-01-06 16:54:36
branko@majic.rs

MAR-148: Fix test for the common role:

- Do some additional stripping on command outputs to get rid of extra
new lines.

2 files changed with 3 insertions and 3 deletions:

roles/common/molecule/default/tests/test_default.py

roles/common/molecule/default/tests/test_parameters_optional.py

0 comments (0 inline, 0 general)

roles/common/molecule/default/tests/test_default.py

➞

Show inline comments

     # Normalise package names and order.
     expected_packages = sorted([unicode(p.lower()) for p in expected_packages])
     actual_packages = sorted(packages.stdout.lower().split("\n"))
+    actual_packages = sorted(packages.stdout.lower().strip().split("\n"))
     # This is a dummy distro-provided package ignored by the pip-tools.
     if "pkg-resources==0.0.0" in actual_packages:
         major_version = host.run("%s -c %s", python_path, "import sys; print(sys.version_info.major)")
     assert major_version.rc == 0
     assert major_version.stdout == expected_major_version
+    assert major_version.stdout.strip() == expected_major_version
 @pytest.mark.parametrize('wrong_python_path', [

roles/common/molecule/default/tests/test_parameters_optional.py

➞

Show inline comments

@@ @@ -347,7 +347,7 @@ def test_ntp_query_server_count(host): @@
     ntpq = host.command("ntpq -p -n")
     assert ntpq.rc == 0
     assert len(ntpq.stdout.split("\n")) == expected_stdout_line_count
+    assert len(ntpq.stdout.strip().split("\n")) == expected_stdout_line_count
 def test_ntp_listening_interfaces(host):

0 comments (0 inline, 0 general)