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

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

Issue 17260002: Add new test suite for migrated blink tests. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Reworked review comments. Created 7 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « test/webkit/resources/standalone-pre.js ('k') | test/webkit/webkit.status » ('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 2008 the V8 project authors. All rights reserved. 1 # Copyright 2013 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
11 # with the distribution. 11 # with the distribution.
12 # * Neither the name of Google Inc. nor the names of its 12 # * Neither the name of Google Inc. nor the names of its
13 # contributors may be used to endorse or promote products derived 13 # contributors may be used to endorse or promote products derived
14 # from this software without specific prior written permission. 14 # from this software without specific prior written permission.
15 # 15 #
16 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 16 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
17 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 17 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
18 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 18 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
19 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 19 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
20 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 20 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
21 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 21 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
22 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 22 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
23 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 23 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
24 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 24 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 25 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
26 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 27
28 import itertools
28 import os 29 import os
29 import re 30 import re
30 31
31 from testrunner.local import testsuite 32 from testrunner.local import testsuite
32 from testrunner.objects import testcase 33 from testrunner.objects import testcase
33 34
34 FLAGS_PATTERN = re.compile(r"//\s+Flags:(.*)") 35 FLAGS_PATTERN = re.compile(r"//\s+Flags:(.*)")
35 FILES_PATTERN = re.compile(r"//\s+Files:(.*)") 36 FILES_PATTERN = re.compile(r"//\s+Files:(.*)")
36 SELF_SCRIPT_PATTERN = re.compile(r"//\s+Env: TEST_FILE_NAME") 37 SELF_SCRIPT_PATTERN = re.compile(r"//\s+Env: TEST_FILE_NAME")
37 38
38 39
39 class MjsunitTestSuite(testsuite.TestSuite): 40 # TODO (machenbach): Share commonalities with mjstest.
41 class WebkitTestSuite(testsuite.TestSuite):
40 42
41 def __init__(self, name, root): 43 def __init__(self, name, root):
42 super(MjsunitTestSuite, self).__init__(name, root) 44 super(WebkitTestSuite, self).__init__(name, root)
43 45
44 def ListTests(self, context): 46 def ListTests(self, context):
45 tests = [] 47 tests = []
48 exclude = os.path.join(self.root, 'resources')
46 for dirname, dirs, files in os.walk(self.root): 49 for dirname, dirs, files in os.walk(self.root):
47 for dotted in [x for x in dirs if x.startswith('.')]: 50 for dotted in [x for x in dirs if x.startswith('.')]:
48 dirs.remove(dotted) 51 dirs.remove(dotted)
52 if dirname == exclude:
Jakob Kummerow 2013/06/18 08:43:40 I guess this works, but it doesn't fit with the ex
53 del(dirs)
54 continue
49 dirs.sort() 55 dirs.sort()
50 files.sort() 56 files.sort()
51 for filename in files: 57 for filename in files:
52 if filename.endswith(".js") and filename != "mjsunit.js": 58 if filename.endswith(".js"):
53 testname = os.path.join(dirname[len(self.root) + 1:], filename[:-3]) 59 testname = os.path.join(dirname[len(self.root) + 1:], filename[:-3])
54 test = testcase.TestCase(self, testname) 60 test = testcase.TestCase(self, testname)
55 tests.append(test) 61 tests.append(test)
56 return tests 62 return tests
57 63
58 def GetFlagsForTestCase(self, testcase, context): 64 def GetFlagsForTestCase(self, testcase, context):
59 source = self.GetSourceForTest(testcase) 65 source = self.GetSourceForTest(testcase)
60 flags = [] + context.mode_flags 66 flags = [] + context.mode_flags
61 flags_match = re.findall(FLAGS_PATTERN, source) 67 flags_match = re.findall(FLAGS_PATTERN, source)
62 for match in flags_match: 68 for match in flags_match:
63 flags += match.strip().split() 69 flags += match.strip().split()
64 70
65 files_list = [] # List of file names to append to command arguments. 71 files_list = [] # List of file names to append to command arguments.
66 files_match = FILES_PATTERN.search(source); 72 files_match = FILES_PATTERN.search(source);
67 # Accept several lines of 'Files:'. 73 # Accept several lines of 'Files:'.
68 while True: 74 while True:
69 if files_match: 75 if files_match:
70 files_list += files_match.group(1).strip().split() 76 files_list += files_match.group(1).strip().split()
71 files_match = FILES_PATTERN.search(source, files_match.end()) 77 files_match = FILES_PATTERN.search(source, files_match.end())
72 else: 78 else:
73 break 79 break
74 files = [ os.path.normpath(os.path.join(self.root, '..', '..', f)) 80 files = [ os.path.normpath(os.path.join(self.root, '..', '..', f))
75 for f in files_list ] 81 for f in files_list ]
76 testfilename = os.path.join(self.root, testcase.path + self.suffix()) 82 testfilename = os.path.join(self.root, testcase.path + self.suffix())
77 if SELF_SCRIPT_PATTERN.search(source): 83 if SELF_SCRIPT_PATTERN.search(source):
78 env = ["-e", "TEST_FILE_NAME=\"%s\"" % testfilename.replace("\\", "\\\\")] 84 env = ["-e", "TEST_FILE_NAME=\"%s\"" % testfilename.replace("\\", "\\\\")]
79 files = env + files 85 files = env + files
80 files.append(os.path.join(self.root, "mjsunit.js")) 86 files.append(os.path.join(self.root, "resources/standalone-pre.js"))
81 files.append(testfilename) 87 files.append(testfilename)
88 files.append(os.path.join(self.root, "resources/standalone-post.js"))
82 89
83 flags += files 90 flags += files
84 if context.isolates: 91 if context.isolates:
85 flags.append("--isolate") 92 flags.append("--isolate")
86 flags += files 93 flags += files
87 94
88 return testcase.flags + flags 95 return testcase.flags + flags
89 96
90 def GetSourceForTest(self, testcase): 97 def GetSourceForTest(self, testcase):
91 filename = os.path.join(self.root, testcase.path + self.suffix()) 98 filename = os.path.join(self.root, testcase.path + self.suffix())
92 with open(filename) as f: 99 with open(filename) as f:
93 return f.read() 100 return f.read()
94 101
102 # TODO(machenbach): Share with test/message/testcfg.py
103 def _IgnoreLine(self, string):
104 """Ignore empty lines, valgrind output and Android output."""
105 if not string: return True
106 return (string.startswith("==") or string.startswith("**") or
107 string.startswith("ANDROID") or
108 # These five patterns appear in normal Native Client output.
109 string.startswith("DEBUG MODE ENABLED") or
110 string.startswith("tools/nacl-run.py") or
111 string.find("BYPASSING ALL ACL CHECKS") > 0 or
112 string.find("Native Client module will be loaded") > 0 or
113 string.find("NaClHostDescOpen:") > 0)
114
115 def IsFailureOutput(self, output, testpath):
116 if super(WebkitTestSuite, self).IsFailureOutput(output, testpath):
117 return True
118 file_name = os.path.join(self.root, testpath) + "-expected.txt"
119 with file(file_name, "r") as expected:
120 def ExpIterator():
121 for line in expected.readlines():
122 if line.startswith("#") or not line.strip(): continue
Jakob Kummerow 2013/06/18 08:43:40 Don't you need to have this line in ActIterator to
123 yield line.strip()
124 def ActIterator():
125 for line in output.stdout.splitlines():
126 if self._IgnoreLine(line.strip()): continue
127 yield line.strip()
128 for (expected, actual) in itertools.izip(ExpIterator(), ActIterator()):
129 if expected != actual:
130 return True
131 return False
132
95 133
96 def GetSuite(name, root): 134 def GetSuite(name, root):
97 return MjsunitTestSuite(name, root) 135 return WebkitTestSuite(name, root)
OLDNEW
« no previous file with comments | « test/webkit/resources/standalone-pre.js ('k') | test/webkit/webkit.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698