From 178092954d2b6d5293f6211bd4e2f9f37bb8c525 Mon Sep 17 00:00:00 2001 From: Ian Cordasco Date: Sat, 27 May 2017 20:16:15 -0500 Subject: [PATCH] Add extra test cases for violation decisions --- tests/unit/test_style_guide.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/tests/unit/test_style_guide.py b/tests/unit/test_style_guide.py index 10550f8..7ae77d7 100644 --- a/tests/unit/test_style_guide.py +++ b/tests/unit/test_style_guide.py @@ -128,6 +128,10 @@ def test_is_user_selected_excludes_errors(select_list, error_code): (['E'], defaults.IGNORE, 'E126', style_guide.Decision.Selected), (['W'], defaults.IGNORE, 'E126', style_guide.Decision.Ignored), (['E'], defaults.IGNORE, 'W391', style_guide.Decision.Ignored), + (['E', 'W'], ['E13'], 'E131', style_guide.Decision.Ignored), + (defaults.SELECT, ['E13'], 'E131', style_guide.Decision.Ignored), + (defaults.SELECT, defaults.IGNORE, 'E126', style_guide.Decision.Ignored), + (defaults.SELECT, defaults.IGNORE, 'W391', style_guide.Decision.Selected), ]) def test_should_report_error(select_list, ignore_list, error_code, expected): """Verify we decide when to report an error.""" @@ -159,10 +163,20 @@ def test_should_report_error(select_list, ignore_list, error_code, expected): style_guide.Decision.Ignored), (defaults.SELECT, ['U401'], [], ['U4'], 'U402', style_guide.Decision.Selected), + (['E', 'W'], ['E13'], [], [], 'E131', style_guide.Decision.Ignored), + (['E', 'W'], ['E13'], [], [], 'E126', style_guide.Decision.Selected), (['E2'], ['E21'], [], [], 'E221', style_guide.Decision.Selected), (['E2'], ['E21'], [], [], 'E212', style_guide.Decision.Ignored), (['F', 'W'], ['C90'], ['I1'], [], 'C901', style_guide.Decision.Ignored), + (defaults.SELECT, defaults.IGNORE, [], ['I'], 'I101', + style_guide.Decision.Selected), + (defaults.SELECT, defaults.IGNORE, ['G'], ['I'], 'G101', + style_guide.Decision.Selected), + (defaults.SELECT, ['G1'], ['G'], ['I'], 'G101', + style_guide.Decision.Ignored), + (defaults.SELECT, ['E126'], [], ['I'], 'I101', + style_guide.Decision.Selected), ] ) def test_decision_for_logic(select, ignore, extend_select, enabled_extensions,