Skip to content

Commit

Permalink
Changed variable names
Browse files Browse the repository at this point in the history
Adressing Issue freedomofpress#6694
Changed names of variables to not treat the functions as PyTest
  • Loading branch information
wallbloggerbeing authored Dec 1, 2022
1 parent ca4ee55 commit 3b58fdc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions molecule/builder-focal/tests/test_security_updates.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
testinfra_hosts = ["docker://{}-sd-sec-update".format(SECUREDROP_TARGET_DISTRIBUTION)]


def test_should_run():
def should_run():
command = ["git", "describe", "--all"]
version = check_output(command).decode("utf8")[0:-1]
candidates = (
Expand All @@ -24,7 +24,7 @@ def test_should_run():
return False


@pytest.mark.skipif(not test_should_run(), reason="Only tested for RCs and builder updates")
@pytest.mark.skipif(not should_run(), reason="Only tested for RCs and builder updates")
def test_ensure_no_updates_avail(host):
"""
Test to make sure that there are no security-updates in the
Expand Down

0 comments on commit 3b58fdc

Please sign in to comment.