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

Side by Side Diff: test/runner/configuration/configuration_test.dart

Issue 1890853003: Add support for sharding test runs. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Code review changes Created 4 years, 8 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
« no previous file with comments | « lib/src/runner/configuration/args.dart ('k') | test/runner/shard_test.dart » ('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 (c) 2016, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2016, the Dart project authors. Please see the AUTHORS 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 @TestOn("vm") 5 @TestOn("vm")
6 6
7 import 'package:boolean_selector/boolean_selector.dart'; 7 import 'package:boolean_selector/boolean_selector.dart';
8 import 'package:path/path.dart' as p; 8 import 'package:path/path.dart' as p;
9 import 'package:test/test.dart'; 9 import 'package:test/test.dart';
10 10
11 import 'package:test/src/backend/platform_selector.dart'; 11 import 'package:test/src/backend/platform_selector.dart';
12 import 'package:test/src/backend/test_platform.dart'; 12 import 'package:test/src/backend/test_platform.dart';
13 import 'package:test/src/runner/configuration.dart'; 13 import 'package:test/src/runner/configuration.dart';
14 import 'package:test/src/runner/configuration/values.dart'; 14 import 'package:test/src/runner/configuration/values.dart';
15 import 'package:test/src/util/io.dart'; 15 import 'package:test/src/util/io.dart';
16 16
17 void main() { 17 void main() {
18 group("merge", () { 18 group("merge", () {
19 group("for most fields", () { 19 group("for most fields", () {
20 test("if neither is defined, preserves the default", () { 20 test("if neither is defined, preserves the default", () {
21 var merged = new Configuration().merge(new Configuration()); 21 var merged = new Configuration().merge(new Configuration());
22 expect(merged.help, isFalse); 22 expect(merged.help, isFalse);
23 expect(merged.version, isFalse); 23 expect(merged.version, isFalse);
24 expect(merged.verboseTrace, isFalse); 24 expect(merged.verboseTrace, isFalse);
25 expect(merged.jsTrace, isFalse); 25 expect(merged.jsTrace, isFalse);
26 expect(merged.skip, isFalse); 26 expect(merged.skip, isFalse);
27 expect(merged.skipReason, isNull); 27 expect(merged.skipReason, isNull);
28 expect(merged.pauseAfterLoad, isFalse); 28 expect(merged.pauseAfterLoad, isFalse);
29 expect(merged.color, equals(canUseSpecialChars)); 29 expect(merged.color, equals(canUseSpecialChars));
30 expect(merged.shardIndex, isNull);
31 expect(merged.totalShards, isNull);
30 expect(merged.packageRoot, equals(p.join(p.current, 'packages'))); 32 expect(merged.packageRoot, equals(p.join(p.current, 'packages')));
31 expect(merged.reporter, equals(defaultReporter)); 33 expect(merged.reporter, equals(defaultReporter));
32 expect(merged.pubServeUrl, isNull); 34 expect(merged.pubServeUrl, isNull);
33 expect(merged.platforms, equals([TestPlatform.vm])); 35 expect(merged.platforms, equals([TestPlatform.vm]));
34 expect(merged.paths, equals(["test"])); 36 expect(merged.paths, equals(["test"]));
35 }); 37 });
36 38
37 test("if only the old configuration's is defined, uses it", () { 39 test("if only the old configuration's is defined, uses it", () {
38 var merged = new Configuration( 40 var merged = new Configuration(
39 help: true, 41 help: true,
40 version: true, 42 version: true,
41 verboseTrace: true, 43 verboseTrace: true,
42 jsTrace: true, 44 jsTrace: true,
43 skip: true, 45 skip: true,
44 skipReason: "boop", 46 skipReason: "boop",
45 pauseAfterLoad: true, 47 pauseAfterLoad: true,
46 color: true, 48 color: true,
49 shardIndex: 3,
50 totalShards: 10,
47 packageRoot: "root", 51 packageRoot: "root",
48 reporter: "json", 52 reporter: "json",
49 pubServePort: 1234, 53 pubServePort: 1234,
50 platforms: [TestPlatform.chrome], 54 platforms: [TestPlatform.chrome],
51 paths: ["bar"]) 55 paths: ["bar"])
52 .merge(new Configuration()); 56 .merge(new Configuration());
53 57
54 expect(merged.help, isTrue); 58 expect(merged.help, isTrue);
55 expect(merged.version, isTrue); 59 expect(merged.version, isTrue);
56 expect(merged.verboseTrace, isTrue); 60 expect(merged.verboseTrace, isTrue);
57 expect(merged.jsTrace, isTrue); 61 expect(merged.jsTrace, isTrue);
58 expect(merged.skip, isTrue); 62 expect(merged.skip, isTrue);
59 expect(merged.skipReason, equals("boop")); 63 expect(merged.skipReason, equals("boop"));
60 expect(merged.pauseAfterLoad, isTrue); 64 expect(merged.pauseAfterLoad, isTrue);
61 expect(merged.color, isTrue); 65 expect(merged.color, isTrue);
66 expect(merged.shardIndex, equals(3));
67 expect(merged.totalShards, equals(10));
62 expect(merged.packageRoot, equals("root")); 68 expect(merged.packageRoot, equals("root"));
63 expect(merged.reporter, equals("json")); 69 expect(merged.reporter, equals("json"));
64 expect(merged.pubServeUrl.port, equals(1234)); 70 expect(merged.pubServeUrl.port, equals(1234));
65 expect(merged.platforms, equals([TestPlatform.chrome])); 71 expect(merged.platforms, equals([TestPlatform.chrome]));
66 expect(merged.paths, equals(["bar"])); 72 expect(merged.paths, equals(["bar"]));
67 }); 73 });
68 74
69 test("if only the new configuration's is defined, uses it", () { 75 test("if only the new configuration's is defined, uses it", () {
70 var merged = new Configuration().merge(new Configuration( 76 var merged = new Configuration().merge(new Configuration(
71 help: true, 77 help: true,
72 version: true, 78 version: true,
73 verboseTrace: true, 79 verboseTrace: true,
74 jsTrace: true, 80 jsTrace: true,
75 skip: true, 81 skip: true,
76 skipReason: "boop", 82 skipReason: "boop",
77 pauseAfterLoad: true, 83 pauseAfterLoad: true,
78 color: true, 84 color: true,
85 shardIndex: 3,
86 totalShards: 10,
79 packageRoot: "root", 87 packageRoot: "root",
80 reporter: "json", 88 reporter: "json",
81 pubServePort: 1234, 89 pubServePort: 1234,
82 platforms: [TestPlatform.chrome], 90 platforms: [TestPlatform.chrome],
83 paths: ["bar"])); 91 paths: ["bar"]));
84 92
85 expect(merged.help, isTrue); 93 expect(merged.help, isTrue);
86 expect(merged.version, isTrue); 94 expect(merged.version, isTrue);
87 expect(merged.verboseTrace, isTrue); 95 expect(merged.verboseTrace, isTrue);
88 expect(merged.jsTrace, isTrue); 96 expect(merged.jsTrace, isTrue);
89 expect(merged.skip, isTrue); 97 expect(merged.skip, isTrue);
90 expect(merged.skipReason, equals("boop")); 98 expect(merged.skipReason, equals("boop"));
91 expect(merged.pauseAfterLoad, isTrue); 99 expect(merged.pauseAfterLoad, isTrue);
92 expect(merged.color, isTrue); 100 expect(merged.color, isTrue);
101 expect(merged.shardIndex, equals(3));
102 expect(merged.totalShards, equals(10));
93 expect(merged.packageRoot, equals("root")); 103 expect(merged.packageRoot, equals("root"));
94 expect(merged.reporter, equals("json")); 104 expect(merged.reporter, equals("json"));
95 expect(merged.pubServeUrl.port, equals(1234)); 105 expect(merged.pubServeUrl.port, equals(1234));
96 expect(merged.platforms, equals([TestPlatform.chrome])); 106 expect(merged.platforms, equals([TestPlatform.chrome]));
97 expect(merged.paths, equals(["bar"])); 107 expect(merged.paths, equals(["bar"]));
98 }); 108 });
99 109
100 test("if the two configurations conflict, uses the new configuration's " 110 test("if the two configurations conflict, uses the new configuration's "
101 "values", () { 111 "values", () {
102 var older = new Configuration( 112 var older = new Configuration(
103 help: true, 113 help: true,
104 version: false, 114 version: false,
105 verboseTrace: true, 115 verboseTrace: true,
106 jsTrace: false, 116 jsTrace: false,
107 skip: true, 117 skip: true,
108 skipReason: "foo", 118 skipReason: "foo",
109 pauseAfterLoad: true, 119 pauseAfterLoad: true,
110 color: false, 120 color: false,
121 shardIndex: 2,
122 totalShards: 4,
111 packageRoot: "root", 123 packageRoot: "root",
112 reporter: "json", 124 reporter: "json",
113 pubServePort: 1234, 125 pubServePort: 1234,
114 platforms: [TestPlatform.chrome], 126 platforms: [TestPlatform.chrome],
115 paths: ["bar"]); 127 paths: ["bar"]);
116 var newer = new Configuration( 128 var newer = new Configuration(
117 help: false, 129 help: false,
118 version: true, 130 version: true,
119 verboseTrace: false, 131 verboseTrace: false,
120 jsTrace: true, 132 jsTrace: true,
121 skip: true, 133 skip: true,
122 skipReason: "bar", 134 skipReason: "bar",
123 pauseAfterLoad: false, 135 pauseAfterLoad: false,
124 color: true, 136 color: true,
137 shardIndex: 3,
138 totalShards: 10,
125 packageRoot: "boot", 139 packageRoot: "boot",
126 reporter: "compact", 140 reporter: "compact",
127 pubServePort: 5678, 141 pubServePort: 5678,
128 platforms: [TestPlatform.dartium], 142 platforms: [TestPlatform.dartium],
129 paths: ["blech"]); 143 paths: ["blech"]);
130 var merged = older.merge(newer); 144 var merged = older.merge(newer);
131 145
132 expect(merged.help, isFalse); 146 expect(merged.help, isFalse);
133 expect(merged.version, isTrue); 147 expect(merged.version, isTrue);
134 expect(merged.verboseTrace, isFalse); 148 expect(merged.verboseTrace, isFalse);
135 expect(merged.jsTrace, isTrue); 149 expect(merged.jsTrace, isTrue);
136 expect(merged.skipReason, equals("bar")); 150 expect(merged.skipReason, equals("bar"));
137 expect(merged.pauseAfterLoad, isFalse); 151 expect(merged.pauseAfterLoad, isFalse);
138 expect(merged.color, isTrue); 152 expect(merged.color, isTrue);
153 expect(merged.shardIndex, equals(3));
154 expect(merged.totalShards, equals(10));
139 expect(merged.packageRoot, equals("boot")); 155 expect(merged.packageRoot, equals("boot"));
140 expect(merged.reporter, equals("compact")); 156 expect(merged.reporter, equals("compact"));
141 expect(merged.pubServeUrl.port, equals(5678)); 157 expect(merged.pubServeUrl.port, equals(5678));
142 expect(merged.platforms, equals([TestPlatform.dartium])); 158 expect(merged.platforms, equals([TestPlatform.dartium]));
143 expect(merged.paths, equals(["blech"])); 159 expect(merged.paths, equals(["blech"]));
144 }); 160 });
145 }); 161 });
146 162
147 group("for testOn", () { 163 group("for testOn", () {
148 test("if neither is defined, preserves the default", () { 164 test("if neither is defined, preserves the default", () {
(...skipping 269 matching lines...) Expand 10 before | Expand all | Expand 10 after
418 .merge(new Configuration()); 434 .merge(new Configuration());
419 435
420 expect(configuration.presets, isEmpty); 436 expect(configuration.presets, isEmpty);
421 expect(configuration.chosenPresets, equals(["foo"])); 437 expect(configuration.chosenPresets, equals(["foo"]));
422 expect(configuration.knownPresets, equals(["foo"])); 438 expect(configuration.knownPresets, equals(["foo"]));
423 expect(configuration.verboseTrace, isTrue); 439 expect(configuration.verboseTrace, isTrue);
424 }); 440 });
425 }); 441 });
426 }); 442 });
427 } 443 }
OLDNEW
« no previous file with comments | « lib/src/runner/configuration/args.dart ('k') | test/runner/shard_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698