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: https://github.com/nodejs/node/pull/9694 Fixes: https://github.com/nodejs/node/issues/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>
This commit is contained in:
parent
ca37fa527f
commit
951ba0d0a9
@ -298,7 +298,7 @@ class TapProgressIndicator(SimpleProgressIndicator):
|
|||||||
|
|
||||||
if output.HasCrashed():
|
if output.HasCrashed():
|
||||||
self.severity = 'crashed'
|
self.severity = 'crashed'
|
||||||
exit_code = output.output.exit_code
|
exit_code = output.output.exit_code
|
||||||
self.traceback = "oh no!\nexit code: " + PrintCrashed(exit_code)
|
self.traceback = "oh no!\nexit code: " + PrintCrashed(exit_code)
|
||||||
|
|
||||||
if output.HasTimedOut():
|
if output.HasTimedOut():
|
||||||
@ -1461,6 +1461,13 @@ def SplitPath(s):
|
|||||||
stripped = [ c.strip() for c in s.split('/') ]
|
stripped = [ c.strip() for c in s.split('/') ]
|
||||||
return [ Pattern(s) for s in stripped if len(s) > 0 ]
|
return [ Pattern(s) for s in stripped if len(s) > 0 ]
|
||||||
|
|
||||||
|
def NormalizePath(path):
|
||||||
|
# strip the extra path information of the specified test
|
||||||
|
if path.startswith('test/'):
|
||||||
|
path = path[5:]
|
||||||
|
if path.endswith('.js'):
|
||||||
|
path = path[:-3]
|
||||||
|
return path
|
||||||
|
|
||||||
def GetSpecialCommandProcessor(value):
|
def GetSpecialCommandProcessor(value):
|
||||||
if (not value) or (value.find('@') == -1):
|
if (not value) or (value.find('@') == -1):
|
||||||
@ -1534,7 +1541,7 @@ def Main():
|
|||||||
else:
|
else:
|
||||||
paths = [ ]
|
paths = [ ]
|
||||||
for arg in args:
|
for arg in args:
|
||||||
path = SplitPath(arg)
|
path = SplitPath(NormalizePath(arg))
|
||||||
paths.append(path)
|
paths.append(path)
|
||||||
|
|
||||||
# Check for --valgrind option. If enabled, we overwrite the special
|
# Check for --valgrind option. If enabled, we overwrite the special
|
||||||
|
Loading…
x
Reference in New Issue
Block a user