languages/dart-sdk
languages/dart-sdk
1
0
Fork 0
mirror of https://github.com/dart-lang/sdk synced 2024-10-05 03:15:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Removed the 'we don't run multitests on browsers'-hack

Browse source 
Review URL: https://codereview.chromium.org//11886064

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@17183 260f80e4-7a28-3924-810f-c04153c831b5
This commit is contained in:
kustermann@google.com 2013-01-17 08:59:05 +00:00
parent 4a6c2ca155
commit 442e7fc3b9
2 changed files with 10 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
tests/language/language.status
View file

@ -285,6 +285,11 @@ call_through_getter_test: Fail
[ $compiler == none && $runtime == drt ]
call_constructor_on_unresolvable_class_test/01: Fail # Issue 2264
call_constructor_on_unresolvable_class_test/02: Fail # Issue 2264
call_constructor_on_unresolvable_class_test/03: Fail # Issue 2264
call_constructor_on_unresolvable_class_test/07: Fail # Issue 2264
call_constructor_on_unresolvable_class_test/none: Fail # Issue 2264
final_variable_assignment_test/01: Fail
final_variable_assignment_test/02: Fail
final_variable_assignment_test/03: Fail

10
tools/testing/dart/test_suite.dart
View file

@ -627,8 +627,6 @@ class StandardTestSuite extends TestSuite {
String testName;
if (optionsFromFile['isMultitest']) {
// Multitests do not run on browsers.
if (TestUtils.isBrowserRuntime(configuration['runtime'])) return;
// Multitests are in [build directory]/generated_tests/... .
// The test name will be '[test filename (no extension)]/[multitest key].
String name = filePath.filenameWithoutExtension;
@ -835,8 +833,10 @@ class StandardTestSuite extends TestSuite {
bool isLibraryDefinition = optionsFromFile['isLibraryDefinition'];
if (isWrappingRequired
&& !isLibraryDefinition && optionsFromFile['containsSourceOrImport']) {
print('Warning for $filename: Browser tests require #library '
'in any file that uses #import, #source, or #resource');
print('Warning for $filename: Browser tests require "library" '
'in any file that uses "import" or "source". '
'Please mark this test as failing in the status file. '
'(see http://dartbug.com/2264)');
}
final String compiler = configuration['compiler'];
@ -1310,7 +1310,7 @@ class StandardTestSuite extends TestSuite {
RegExp libraryDefinitionRegExp =
new RegExp(r"^[#]?library[\( ]", multiLine: true);
RegExp sourceOrImportRegExp =
new RegExp("^(#source|#import|part)[ \t]+[\('\"]", multiLine: true);
new RegExp("^(#?source|#?import|part)[ \t]+[\('\"]", multiLine: true);
var bytes = new File.fromPath(filePath).readAsBytesSync();
String contents = decodeUtf8(bytes);