Commit 3176863a authored by fmeawad's avatar fmeawad Committed by Commit bot

[Telemetry] Skip disabled Tests when filtering

BUG=462779

Review URL: https://codereview.chromium.org/968493003

Cr-Commit-Position: refs/heads/master@{#318730}
parent 0bcaa8e5
...@@ -141,16 +141,9 @@ class RunTestsCommand(command_line.OptparseCommand): ...@@ -141,16 +141,9 @@ class RunTestsCommand(command_line.OptparseCommand):
def GetClassifier(args, possible_browser): def GetClassifier(args, possible_browser):
def ClassifyTest(test_set, test): def ClassifyTest(test_set, test):
name = test.id() name = test.id()
if args.positional_args: if (not args.positional_args
if _MatchesSelectedTest(name, args.positional_args, or _MatchesSelectedTest(name, args.positional_args,
args.exact_test_filter): args.exact_test_filter)):
assert hasattr(test, '_testMethodName')
method = getattr(test, test._testMethodName) # pylint: disable=W0212
if decorators.ShouldBeIsolated(method, possible_browser):
test_set.isolated_tests.append(typ.TestInput(name))
else:
test_set.parallel_tests.append(typ.TestInput(name))
else:
assert hasattr(test, '_testMethodName') assert hasattr(test, '_testMethodName')
method = getattr(test, test._testMethodName) # pylint: disable=W0212 method = getattr(test, test._testMethodName) # pylint: disable=W0212
should_skip, reason = decorators.ShouldSkip(method, possible_browser) should_skip, reason = decorators.ShouldSkip(method, possible_browser)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment