From 56d53cd60bece912fba37bc6e1c54dc0fd5cdffe Mon Sep 17 00:00:00 2001 From: Tzu-ping Chung Date: Thu, 14 May 2020 15:11:10 +0800 Subject: [PATCH] Add a reason why we're xfail-ing this test This test uses xfail to test the negation of what's described in the test, i.e. the assertion should "fail" if the command does what we expect. I think this is a terrible way to write a test, but don't bother to rewrite. This commit message is left here to tell future maintainers what the test is all about by git blame. --- tests/functional/test_freeze.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/functional/test_freeze.py b/tests/functional/test_freeze.py index 44278c866..2199bb482 100644 --- a/tests/functional/test_freeze.py +++ b/tests/functional/test_freeze.py @@ -191,7 +191,12 @@ def test_freeze_svn(script, tmpdir): @pytest.mark.git -@pytest.mark.xfail +@pytest.mark.xfail( + condition=True, + reason="xfail means editable is not in output", + run=True, + strict=True, +) def test_freeze_exclude_editable(script, tmpdir): """ Test excluding editable from freezing list.