Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(526)

Unified Diff: dart/tests/co19/testcfg.py

Issue 9325024: Remove "complex matching" from co19. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: rebased Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dart/tests/co19/test_config.dart ('k') | dart/tools/testing/dart/status_file_parser.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/tests/co19/testcfg.py
diff --git a/dart/tests/co19/testcfg.py b/dart/tests/co19/testcfg.py
index 2732bb62ee0336a2cf68e2d6b5f6714f9a13efd9..3d765f8ac93e34cda07120932939a481c655e6cf 100644
--- a/dart/tests/co19/testcfg.py
+++ b/dart/tests/co19/testcfg.py
@@ -84,25 +84,13 @@ class Co19TestConfiguration(test.TestConfiguration):
test_path.extend(root.split(os.path.sep)[strip:])
test_name = short_name = f
- # shotlen test_name
- # remove repeats
- if short_name.startswith(test_path[-1]):
- short_name = short_name[len(test_path[-1]) : ]
-
# remove suffixes
if short_name.endswith(".dart"):
short_name = short_name[:-5] # Remove .dart suffix.
- # now .app suffix discarded at self.IsTest()
- #elif short_name.endswith(".app"):
- # short_name = short_name[:-4] # Remove .app suffix.
else:
raise Error('Unknown suffix in "%s", fix IsTest() predicate' % f)
-
- while short_name.startswith('_'):
- short_name = short_name[1:]
-
- test_path.extend(short_name.split('_'))
+ test_path.append(short_name)
# test full name and shorted name matches given path pattern
if self.Contains(path, test_path): pass
« no previous file with comments | « dart/tests/co19/test_config.dart ('k') | dart/tools/testing/dart/status_file_parser.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698