Commit bd7c1e7
tools: allow test.py to use full paths of tests
Allow test.py to run tests with a 'tests/' prefix or a '.js' postfix
PR-URL: #9694
Fixes: #9684
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>1 parent 8a91616 commit bd7c1e7
1 file changed
Lines changed: 9 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -298,7 +298,7 @@ def HasRun(self, output): | |||
| 298 | 298 | ||
| 299 | 299 | if output.HasCrashed(): | |
| 300 | 300 | self.severity = 'crashed' | |
| 301 | - exit_code = output.output.exit_code | ||
| 301 | + exit_code = output.output.exit_code | ||
| 302 | 302 | self.traceback = "oh no!\nexit code: " + PrintCrashed(exit_code) | |
| 303 | 303 | ||
| 304 | 304 | if output.HasTimedOut(): | |
@@ -1461,6 +1461,13 @@ def SplitPath(s): | |||
| 1461 | 1461 | stripped = [ c.strip() for c in s.split('/') ] | |
| 1462 | 1462 | return [ Pattern(s) for s in stripped if len(s) > 0 ] | |
| 1463 | 1463 | ||
| 1464 | + def NormalizePath(path): | ||
| 1465 | + # strip the extra path information of the specified test | ||
| 1466 | + if path.startswith('test/'): | ||
| 1467 | + path = path[5:] | ||
| 1468 | + if path.endswith('.js'): | ||
| 1469 | + path = path[:-3] | ||
| 1470 | + return path | ||
| 1464 | 1471 | ||
| 1465 | 1472 | def GetSpecialCommandProcessor(value): | |
| 1466 | 1473 | if (not value) or (value.find('@') == -1): | |
@@ -1534,7 +1541,7 @@ def Main(): | |||
| 1534 | 1541 | else: | |
| 1535 | 1542 | paths = [ ] | |
| 1536 | 1543 | for arg in args: | |
| 1537 | - path = SplitPath(arg) | ||
| 1544 | + path = SplitPath(NormalizePath(arg)) | ||
| 1538 | 1545 | paths.append(path) | |
| 1539 | 1546 | ||
| 1540 | 1547 | # Check for --valgrind option. If enabled, we overwrite the special | |
0 commit comments