openstack/tempest
Code Issues Proposed changes

Fix skip tracker regex for multi-line skips

Browse Source 
This commit changes the regex used for detecting a skip line by
removing the requirement on an '(' immediately following 'skip'.
This changes the skip tracker to match multiline skips (assuming
'skip' is on the same line as the bug number. It also enables
conditional skips that have a bug number from being picked up.

Change-Id: I95a0521661084034f7625c3c517ae9210120de4c
This commit is contained in:
Matthew Treinish
2013-08-16 17:48:29 -04:00
parent 1ac6776ee9
commit bd6fcb68d7
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tools/skip_tracker.py
View File

@@ -61,7 +61,7 @@ def find_skips_in_file(path):
"""
Return the skip tuples in a test file
"""
BUG_RE = re.compile(r'.*skip\(.*bug:*\s*\#*(\d+)', re.IGNORECASE)
BUG_RE = re.compile(r'.*skip.*bug:*\s*\#*(\d+)', re.IGNORECASE)
DEF_RE = re.compile(r'.*def (\w+)\(')
bug_found = False
results = []