Skip to content

Commit

Permalink
Merge pull request #2322 from AndreySV/fix-test-with-false
Browse files Browse the repository at this point in the history
test: use false instead of /usr/bin/false
  • Loading branch information
nejch committed Oct 17, 2022
2 parents 9410acb + 51964b3 commit 4eca9b9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/unit/test_config.py
Expand Up @@ -305,7 +305,7 @@ def test_data_from_helper(m_open, monkeypatch, tmp_path):
@mock.patch("builtins.open")
@pytest.mark.skipif(sys.platform.startswith("win"), reason="Not supported on Windows")
def test_from_helper_subprocess_error_raises_error(m_open, monkeypatch):
# using /usr/bin/false here to force a non-zero return code
# using false here to force a non-zero return code
fd = io.StringIO(
dedent(
"""\
Expand All @@ -314,7 +314,7 @@ def test_from_helper_subprocess_error_raises_error(m_open, monkeypatch):
[helper]
url = https://helper.url
oauth_token = helper: /usr/bin/false
oauth_token = helper: false
"""
)
)
Expand Down

0 comments on commit 4eca9b9

Please sign in to comment.