Skip to content

Commit 82f1c85

Browse files
author
Kimberly Pennington
authored
moved impact_included method inside Axe class (#52)
1 parent a62dd42 commit 82f1c85

File tree

1 file changed

+20
-17
lines changed

1 file changed

+20
-17
lines changed

axe_selenium_python/axe.py

Lines changed: 20 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -8,22 +8,6 @@
88
_DEFAULT_SCRIPT = os.path.join(os.path.dirname(__file__), 'src', 'axe.min.js')
99

1010

11-
def impact_included(rule, impact):
12-
if impact == 'minor' or impact is None:
13-
return True
14-
elif impact == 'moderate':
15-
if rule['impact'] != 'minor':
16-
return True
17-
elif impact == 'severe':
18-
if rule['impact'] == 'severe' or rule['impact'] == 'critical':
19-
return True
20-
elif impact == 'critical':
21-
if rule['impact'] == 'critical':
22-
return True
23-
else:
24-
return False
25-
26-
2711
class Axe(object):
2812

2913
def __init__(self, selenium, script_url=_DEFAULT_SCRIPT):
@@ -75,10 +59,29 @@ def run(self, context=None, options=None, impact=None):
7559
"""
7660
self.inject()
7761
data = self.execute(context, options)
78-
violations = dict((rule['id'], rule) for rule in data['violations'] if impact_included(rule, impact))
62+
violations = dict((rule['id'], rule) for rule in data['violations'] if self.impact_included(rule, impact))
7963

8064
return violations
8165

66+
def impact_included(rule, impact):
67+
"""
68+
Function to filter for violations iwht specified impact level, and all
69+
violations with a higher impact level.
70+
"""
71+
if impact == 'minor' or impact is None:
72+
return True
73+
elif impact == 'moderate':
74+
if rule['impact'] != 'minor':
75+
return True
76+
elif impact == 'severe':
77+
if rule['impact'] == 'severe' or rule['impact'] == 'critical':
78+
return True
79+
elif impact == 'critical':
80+
if rule['impact'] == 'critical':
81+
return True
82+
else:
83+
return False
84+
8285
def report(self, violations):
8386
"""
8487
Return readable report of accessibility violations found.

0 commit comments

Comments
 (0)