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

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

Issue 10855099: Allow multiple lines of custom flags in javascript tests. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 8 years, 4 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 | « no previous file | test/mjsunit/array-literal-transitions.js » ('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 2008 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 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 74
75 def GetLabel(self): 75 def GetLabel(self):
76 return "%s %s" % (self.mode, self.GetName()) 76 return "%s %s" % (self.mode, self.GetName())
77 77
78 def GetName(self): 78 def GetName(self):
79 return self.path[-1] 79 return self.path[-1]
80 80
81 def GetCommand(self): 81 def GetCommand(self):
82 result = self.config.context.GetVmCommand(self, self.mode) 82 result = self.config.context.GetVmCommand(self, self.mode)
83 source = open(self.file).read() 83 source = open(self.file).read()
84 flags_match = FLAGS_PATTERN.search(source) 84 flags_match = re.findall(FLAGS_PATTERN, source)
85 if flags_match: 85 for match in flags_match:
86 result += flags_match.group(1).strip().split() 86 result += match.strip().split()
87 result.append(self.file) 87 result.append(self.file)
88 return result 88 return result
89 89
90 def GetSource(self): 90 def GetSource(self):
91 return (open(self.file).read() 91 return (open(self.file).read()
92 + "\n--- expected output ---\n" 92 + "\n--- expected output ---\n"
93 + open(self.expected).read()) 93 + open(self.expected).read())
94 94
95 95
96 class MessageTestConfiguration(test.TestConfiguration): 96 class MessageTestConfiguration(test.TestConfiguration):
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 return ['d8'] 129 return ['d8']
130 130
131 def GetTestStatus(self, sections, defs): 131 def GetTestStatus(self, sections, defs):
132 status_file = join(self.root, 'message.status') 132 status_file = join(self.root, 'message.status')
133 if exists(status_file): 133 if exists(status_file):
134 test.ReadConfigurationInto(status_file, sections, defs) 134 test.ReadConfigurationInto(status_file, sections, defs)
135 135
136 136
137 def GetConfiguration(context, root): 137 def GetConfiguration(context, root):
138 return MessageTestConfiguration(context, root) 138 return MessageTestConfiguration(context, root)
OLDNEW
« no previous file with comments | « no previous file | test/mjsunit/array-literal-transitions.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698