From e8c6a1e2f5aa98273474ada751613dbf7c39d813 Mon Sep 17 00:00:00 2001 From: Ian Cordasco Date: Sun, 4 Jun 2017 14:51:01 -0500 Subject: [PATCH] Get *one* more line of test coverage This was mostly to prove to myself that we could possibly reach that return more than actually covering that return. This just shows how gnarly this logic actually is. I wish there were a better way to write it. --- tests/unit/test_decision_engine.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/unit/test_decision_engine.py b/tests/unit/test_decision_engine.py index aad0176..58ce54c 100644 --- a/tests/unit/test_decision_engine.py +++ b/tests/unit/test_decision_engine.py @@ -162,6 +162,9 @@ def test_decision_for(select_list, ignore_list, error_code, expected): style_guide.Decision.Ignored), (defaults.SELECT, ['E126'], [], ['I'], 'I101', style_guide.Decision.Selected), + # This next one should exercise the catch-all return + (['E', 'W'], defaults.IGNORE, ['I'], [], 'I101', + style_guide.Decision.Selected), ] ) def test_more_specific_decision_for_logic(select, ignore, extend_select,