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

Side by Side Diff: test/promises-aplus/testcfg.py

Issue 258763002: Fix promises-aplus testcfg.py (Closed) Base URL: git://github.com/v8/v8.git@master
Patch Set: Created 6 years, 8 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 | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 the V8 project authors. All rights reserved. 1 # Copyright 2014 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 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 78
79 def GetFlagsForTestCase(self, testcase, context): 79 def GetFlagsForTestCase(self, testcase, context):
80 return (testcase.flags + context.mode_flags + ['--harmony'] + 80 return (testcase.flags + context.mode_flags + ['--harmony'] +
81 self.helper_files_pre + 81 self.helper_files_pre +
82 [os.path.join(self.test_files_root, testcase.path + '.js')] + 82 [os.path.join(self.test_files_root, testcase.path + '.js')] +
83 self.helper_files_post) 83 self.helper_files_post)
84 84
85 def GetSourceForTest(self, testcase): 85 def GetSourceForTest(self, testcase):
86 filename = os.path.join(self.root, TEST_NAME, 86 filename = os.path.join(self.root, TEST_NAME,
87 'lib', 'tests', testcase.path + '.js') 87 'lib', 'tests', testcase.path + '.js')
88 return 'print("FAIL: fail");'
89 with open(filename) as f: 88 with open(filename) as f:
90 return f.read() 89 return f.read()
91 90
92 def IsNegativeTest(self, testcase): 91 def IsNegativeTest(self, testcase):
93 return '@negative' in self.GetSourceForTest(testcase) 92 return '@negative' in self.GetSourceForTest(testcase)
94 93
95 def IsFailureOutput(self, output, testpath): 94 def IsFailureOutput(self, output, testpath):
96 if output.exit_code != 0: 95 if output.exit_code != 0:
97 return True 96 return True
98 return not 'All tests have run.' in output.stdout or \ 97 return not 'All tests have run.' in output.stdout or \
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 os.remove(path) 138 os.remove(path)
140 raise Exception('Hash mismatch of test data file') 139 raise Exception('Hash mismatch of test data file')
141 140
142 def DownloadData(self): 141 def DownloadData(self):
143 self.DownloadTestData() 142 self.DownloadTestData()
144 self.DownloadSinon() 143 self.DownloadSinon()
145 144
146 145
147 def GetSuite(name, root): 146 def GetSuite(name, root):
148 return PromiseAplusTestSuite(name, root) 147 return PromiseAplusTestSuite(name, root)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698