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

Side by Side Diff: test/dart2js/supports_valid_options_test.dart

Issue 1408423006: Fix the tests. (Closed) Base URL: git@github.com:dart-lang/pub.git@master
Patch Set: Created 5 years, 2 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
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS d.file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS d.file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 import 'package:scheduled_test/scheduled_test.dart'; 5 import 'package:scheduled_test/scheduled_test.dart';
6 6
7 import '../descriptor.dart' as d; 7 import '../descriptor.dart' as d;
8 import '../test_pub.dart'; 8 import '../test_pub.dart';
9 import '../serve/utils.dart'; 9 import '../serve/utils.dart';
10 10
11 main() { 11 main() {
12 integration("supports most dart2js command-line options", () { 12 integration("supports most dart2js command-line options", () {
13 d.dir(appPath, [ 13 d.dir(appPath, [
14 d.pubspec({ 14 d.pubspec({
15 "name": "myapp", 15 "name": "myapp",
16 "transformers": [{ 16 "transformers": [{
17 "\$dart2js": { 17 "\$dart2js": {
18 "commandLineOptions": ["--enable-diagnostic-colors"], 18 "commandLineOptions": ["--enable-diagnostic-colors"],
19 "checked": true, 19 "checked": true,
20 "csp": true, 20 "csp": true,
21 "minify": true, 21 "minify": true,
22 "verbose": true, 22 "verbose": true,
23 "environment": {"name": "value"}, 23 "environment": {"name": "value"},
24 "suppressWarnings": true, 24 // TOOD(nweiz): Re-add these when sdk#24653 is fixed.
25 "suppressHints": true, 25 // "suppressWarnings": true,
26 // "suppressHints": true,
26 "suppressPackageWarnings": false, 27 "suppressPackageWarnings": false,
27 "terse": true, 28 "terse": true,
28 "sourceMaps": false 29 "sourceMaps": false
29 } 30 }
30 }] 31 }]
31 }), 32 }),
32 d.dir("web", [ 33 d.dir("web", [
33 d.file("main.dart", "void main() => print('Hello!');") 34 d.file("main.dart", "void main() => print('Hello!');")
34 ]) 35 ])
35 ]).create(); 36 ]).create();
36 37
37 pubGet(); 38 pubGet();
38 39
39 // None of these options should be rejected, either by pub or by dart2js. 40 // None of these options should be rejected, either by pub or by dart2js.
40 pubServe(); 41 pubServe();
41 requestShouldSucceed("main.dart.js", isNot(isEmpty)); 42 requestShouldSucceed("main.dart.js", isNot(isEmpty));
42 endPubServe(); 43 endPubServe();
43 }); 44 });
44 } 45 }
OLDNEW
« pubspec.yaml ('K') | « pubspec.yaml ('k') | test/descriptor.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698