Chromium Code Reviews| Index: sdk/lib/_internal/pub/test/build/compiles_dart_entrypoints_to_dart_and_js_test.dart |
| diff --git a/sdk/lib/_internal/pub/test/build/compiles_dart_entrypoints_to_dart_and_js_test.dart b/sdk/lib/_internal/pub/test/build/compiles_dart_entrypoints_to_dart_and_js_test.dart |
| index 7f8d89880bd8ae4603cfd6abd5fced7fcdb57c3f..5d0b67a7cd4b1367d8b43112741bce3698b4f576 100644 |
| --- a/sdk/lib/_internal/pub/test/build/compiles_dart_entrypoints_to_dart_and_js_test.dart |
| +++ b/sdk/lib/_internal/pub/test/build/compiles_dart_entrypoints_to_dart_and_js_test.dart |
| @@ -2,7 +2,6 @@ |
| // for details. All rights reserved. Use of this source code is governed by a |
| // BSD-style license that can be found in the LICENSE file. |
| -import 'package:path/path.dart' as path; |
| import 'package:scheduled_test/scheduled_test.dart'; |
| import '../descriptor.dart' as d; |
| @@ -11,6 +10,8 @@ import '../test_pub.dart'; |
| main() { |
| initConfig(); |
| + // TODO(rnystrom): Should also add tests that other transformers work. |
|
nweiz
2013/10/16 21:40:22
Is there a bug for this?
Bob Nystrom
2013/10/29 00:03:30
There is now.
|
| + |
| integration("compiles Dart entrypoints to Dart and JS", () { |
| // Dart2js can take a long time to compile dart code, so we increase the |
| // timeout to cope with that. |
| @@ -20,6 +21,7 @@ main() { |
| d.appPubspec(), |
| d.dir('web', [ |
| d.file('file.dart', 'void main() => print("hello");'), |
| + d.file('lib.dart', 'void foo() => print("hello");'), |
| d.dir('subdir', [ |
| d.file('subfile.dart', 'void main() => print("ping");') |
| ]) |
| @@ -27,23 +29,17 @@ main() { |
| ]).create(); |
| schedulePub(args: ["build"], |
| - output: ''' |
| -Finding entrypoints... |
| -Copying web| => build| |
| -Compiling web|file.dart => build|file.dart.js |
| -Compiling web|file.dart => build|file.dart |
| -Compiling web|subdir|subfile.dart => build|subdir|subfile.dart.js |
| -Compiling web|subdir|subfile.dart => build|subdir|subfile.dart |
| -'''.replaceAll('|', path.separator), |
| + output: new RegExp(r"Built 2 files!"), |
|
nweiz
2013/10/16 21:40:22
It feels weird not to test any other aspects of th
Bob Nystrom
2013/10/29 00:03:30
Added a TODO. The current command-line output for
|
| exitCode: 0); |
| d.dir(appPath, [ |
| d.dir('build', [ |
| d.matcherFile('file.dart.js', isNot(isEmpty)), |
| - d.matcherFile('file.dart', isNot(isEmpty)), |
| + d.nothing('file.dart'), |
| + d.nothing('lib.dart'), |
| d.dir('subdir', [ |
| d.matcherFile('subfile.dart.js', isNot(isEmpty)), |
| - d.matcherFile('subfile.dart', isNot(isEmpty)) |
| + d.nothing('subfile.dart') |
| ]) |
| ]) |
| ]).validate(); |