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

Side by Side Diff: test/test262/testcfg.py

Issue 1379093002: [Interpreter]: Add ignition variant to test runner. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 2 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 unified diff | Download patch
« no previous file with comments | « no previous file | tools/testrunner/local/testsuite.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2012 the V8 project authors. All rights reserved. 1 # Copyright 2012 the V8 project authors. All rights reserved.
2 # Redistribution and use in source and binary forms, with or without 2 # Redistribution and use in source and binary forms, with or without
3 # modification, are permitted provided that the following conditions are 3 # modification, are permitted provided that the following conditions are
4 # met: 4 # met:
5 # 5 #
6 # * Redistributions of source code must retain the above copyright 6 # * Redistributions of source code must retain the above copyright
7 # notice, this list of conditions and the following disclaimer. 7 # notice, this list of conditions and the following disclaimer.
8 # * Redistributions in binary form must reproduce the above 8 # * Redistributions in binary form must reproduce the above
9 # copyright notice, this list of conditions and the following 9 # copyright notice, this list of conditions and the following
10 # disclaimer in the documentation and/or other materials provided 10 # disclaimer in the documentation and/or other materials provided
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 for filename in files: 121 for filename in files:
122 if filename.endswith(".js"): 122 if filename.endswith(".js"):
123 fullpath = os.path.join(dirname, filename) 123 fullpath = os.path.join(dirname, filename)
124 relpath = fullpath[len(self.testroot) + 1 : -3] 124 relpath = fullpath[len(self.testroot) + 1 : -3]
125 testname = relpath.replace(os.path.sep, "/") 125 testname = relpath.replace(os.path.sep, "/")
126 case = testcase.TestCase(self, testname) 126 case = testcase.TestCase(self, testname)
127 tests.append(case) 127 tests.append(case)
128 return tests 128 return tests
129 129
130 def GetFlagsForTestCase(self, testcase, context): 130 def GetFlagsForTestCase(self, testcase, context):
131 return (testcase.flags + context.mode_flags + self.harness + 131 # TODO(rmcilroy) Remove ignition filter modification once ignition can
132 # support the test262 test harness code.
133 ignition_filter = "--ignition-filter=" + self.testroot + "/*"
Michael Achenbach 2015/10/01 08:44:53 Please make ignition_filter a class member as it o
rmcilroy 2015/10/06 12:43:24 Done.
134 flags = [ ignition_filter if "ignition-filter" in flag else flag
135 for flag in testcase.flags]
136
137 return (flags + context.mode_flags + self.harness +
132 self.GetIncludesForTest(testcase) + ["--harmony"] + 138 self.GetIncludesForTest(testcase) + ["--harmony"] +
133 [os.path.join(self.testroot, testcase.path + ".js")]) 139 [os.path.join(self.testroot, testcase.path + ".js")])
134 140
135 def _VariantGeneratorFactory(self): 141 def _VariantGeneratorFactory(self):
136 return Test262VariantGenerator 142 return Test262VariantGenerator
137 143
138 def LoadParseTestRecord(self): 144 def LoadParseTestRecord(self):
139 if not self.ParseTestRecord: 145 if not self.ParseTestRecord:
140 root = os.path.join(self.root, *TEST_262_TOOLS_PATH) 146 root = os.path.join(self.root, *TEST_262_TOOLS_PATH)
141 f = None 147 f = None
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 # Magic incantation to allow longer path names on Windows. 234 # Magic incantation to allow longer path names on Windows.
229 archive.extractall(u"\\\\?\\%s" % self.root) 235 archive.extractall(u"\\\\?\\%s" % self.root)
230 else: 236 else:
231 archive.extractall(self.root) 237 archive.extractall(self.root)
232 os.rename(os.path.join(self.root, "tc39-test262-%s" % revision), 238 os.rename(os.path.join(self.root, "tc39-test262-%s" % revision),
233 directory_name) 239 directory_name)
234 240
235 241
236 def GetSuite(name, root): 242 def GetSuite(name, root):
237 return Test262TestSuite(name, root) 243 return Test262TestSuite(name, root)
OLDNEW
« no previous file with comments | « no previous file | tools/testrunner/local/testsuite.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698