Chromium Code Reviews

Side by Side Diff: test/runner/engine_test.dart

Issue 1258163002: Split out a RunnerSuite class. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Code review changes Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff |
« no previous file with comments | « lib/test.dart ('k') | test/runner/load_suite_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) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, 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 import 'package:test/src/backend/declarer.dart'; 5 import 'package:test/src/backend/declarer.dart';
6 import 'package:test/src/backend/state.dart'; 6 import 'package:test/src/backend/state.dart';
7 import 'package:test/src/backend/suite.dart';
8 import 'package:test/src/runner/engine.dart'; 7 import 'package:test/src/runner/engine.dart';
8 import 'package:test/src/runner/runner_suite.dart';
9 import 'package:test/test.dart'; 9 import 'package:test/test.dart';
10 10
11 void main() { 11 void main() {
12 var declarer; 12 var declarer;
13 setUp(() => declarer = new Declarer()); 13 setUp(() => declarer = new Declarer());
14 14
15 test("runs each test in each suite in order", () async { 15 test("runs each test in each suite in order", () async {
16 var testsRun = 0; 16 var testsRun = 0;
17 for (var i = 0; i < 4; i++) { 17 for (var i = 0; i < 4; i++) {
18 declarer.test("test ${i + 1}", expectAsync(() { 18 declarer.test("test ${i + 1}", expectAsync(() {
19 expect(testsRun, equals(i)); 19 expect(testsRun, equals(i));
20 testsRun++; 20 testsRun++;
21 }, max: 1)); 21 }, max: 1));
22 } 22 }
23 23
24 var engine = new Engine.withSuites([ 24 var engine = new Engine.withSuites([
25 new Suite(declarer.tests.take(2)), 25 new RunnerSuite(declarer.tests.take(2)),
26 new Suite(declarer.tests.skip(2)) 26 new RunnerSuite(declarer.tests.skip(2))
27 ]); 27 ]);
28 28
29 await engine.run(); 29 await engine.run();
30 expect(testsRun, equals(4)); 30 expect(testsRun, equals(4));
31 }); 31 });
32 32
33 test("runs tests in a suite added after run() was called", () { 33 test("runs tests in a suite added after run() was called", () {
34 var testsRun = 0; 34 var testsRun = 0;
35 for (var i = 0; i < 4; i++) { 35 for (var i = 0; i < 4; i++) {
36 declarer.test("test ${i + 1}", expectAsync(() { 36 declarer.test("test ${i + 1}", expectAsync(() {
37 expect(testsRun, equals(i)); 37 expect(testsRun, equals(i));
38 testsRun++; 38 testsRun++;
39 }, max: 1)); 39 }, max: 1));
40 } 40 }
41 41
42 var engine = new Engine(); 42 var engine = new Engine();
43 expect(engine.run().then((_) { 43 expect(engine.run().then((_) {
44 expect(testsRun, equals(4)); 44 expect(testsRun, equals(4));
45 }), completes); 45 }), completes);
46 46
47 engine.suiteSink.add(new Suite(declarer.tests)); 47 engine.suiteSink.add(new RunnerSuite(declarer.tests));
48 engine.suiteSink.close(); 48 engine.suiteSink.close();
49 }); 49 });
50 50
51 test("emits each test before it starts running and after the previous test " 51 test("emits each test before it starts running and after the previous test "
52 "finished", () { 52 "finished", () {
53 var testsRun = 0; 53 var testsRun = 0;
54 for (var i = 0; i < 3; i++) { 54 for (var i = 0; i < 3; i++) {
55 declarer.test("test ${i + 1}", expectAsync(() => testsRun++, max: 1)); 55 declarer.test("test ${i + 1}", expectAsync(() => testsRun++, max: 1));
56 } 56 }
57 57
58 var engine = new Engine.withSuites([new Suite(declarer.tests)]); 58 var engine = new Engine.withSuites([new RunnerSuite(declarer.tests)]);
59 59
60 engine.onTestStarted.listen(expectAsync((liveTest) { 60 engine.onTestStarted.listen(expectAsync((liveTest) {
61 // [testsRun] should be one less than the test currently running. 61 // [testsRun] should be one less than the test currently running.
62 expect(liveTest.test.name, equals("test ${testsRun + 1}")); 62 expect(liveTest.test.name, equals("test ${testsRun + 1}"));
63 63
64 // [Engine.onTestStarted] is guaranteed to fire before the first 64 // [Engine.onTestStarted] is guaranteed to fire before the first
65 // [LiveTest.onStateChange]. 65 // [LiveTest.onStateChange].
66 expect(liveTest.onStateChange.first, 66 expect(liveTest.onStateChange.first,
67 completion(equals(const State(Status.running, Result.success)))); 67 completion(equals(const State(Status.running, Result.success))));
68 }, count: 3, max: 3)); 68 }, count: 3, max: 3));
69 69
70 return engine.run(); 70 return engine.run();
71 }); 71 });
72 72
73 test(".run() returns true if every test passes", () { 73 test(".run() returns true if every test passes", () {
74 for (var i = 0; i < 2; i++) { 74 for (var i = 0; i < 2; i++) {
75 declarer.test("test ${i + 1}", () {}); 75 declarer.test("test ${i + 1}", () {});
76 } 76 }
77 77
78 var engine = new Engine.withSuites([new Suite(declarer.tests)]); 78 var engine = new Engine.withSuites([new RunnerSuite(declarer.tests)]);
79 expect(engine.run(), completion(isTrue)); 79 expect(engine.run(), completion(isTrue));
80 }); 80 });
81 81
82 test(".run() returns false if any test fails", () { 82 test(".run() returns false if any test fails", () {
83 for (var i = 0; i < 2; i++) { 83 for (var i = 0; i < 2; i++) {
84 declarer.test("test ${i + 1}", () {}); 84 declarer.test("test ${i + 1}", () {});
85 } 85 }
86 declarer.test("failure", () => throw new TestFailure("oh no")); 86 declarer.test("failure", () => throw new TestFailure("oh no"));
87 87
88 var engine = new Engine.withSuites([new Suite(declarer.tests)]); 88 var engine = new Engine.withSuites([new RunnerSuite(declarer.tests)]);
89 expect(engine.run(), completion(isFalse)); 89 expect(engine.run(), completion(isFalse));
90 }); 90 });
91 91
92 test(".run() returns false if any test errors", () { 92 test(".run() returns false if any test errors", () {
93 for (var i = 0; i < 2; i++) { 93 for (var i = 0; i < 2; i++) {
94 declarer.test("test ${i + 1}", () {}); 94 declarer.test("test ${i + 1}", () {});
95 } 95 }
96 declarer.test("failure", () => throw "oh no"); 96 declarer.test("failure", () => throw "oh no");
97 97
98 var engine = new Engine.withSuites([new Suite(declarer.tests)]); 98 var engine = new Engine.withSuites([new RunnerSuite(declarer.tests)]);
99 expect(engine.run(), completion(isFalse)); 99 expect(engine.run(), completion(isFalse));
100 }); 100 });
101 101
102 test(".run() may not be called more than once", () { 102 test(".run() may not be called more than once", () {
103 var engine = new Engine.withSuites([]); 103 var engine = new Engine.withSuites([]);
104 expect(engine.run(), completes); 104 expect(engine.run(), completes);
105 expect(engine.run, throwsStateError); 105 expect(engine.run, throwsStateError);
106 }); 106 });
107 107
108 group("for a skipped test", () { 108 group("for a skipped test", () {
109 test("doesn't run the test's body", () async { 109 test("doesn't run the test's body", () async {
110 var bodyRun = false; 110 var bodyRun = false;
111 declarer.test("test", () => bodyRun = true, skip: true); 111 declarer.test("test", () => bodyRun = true, skip: true);
112 112
113 var engine = new Engine.withSuites([new Suite(declarer.tests)]); 113 var engine = new Engine.withSuites([new RunnerSuite(declarer.tests)]);
114 await engine.run(); 114 await engine.run();
115 expect(bodyRun, isFalse); 115 expect(bodyRun, isFalse);
116 }); 116 });
117 117
118 test("exposes a LiveTest that emits the correct states", () { 118 test("exposes a LiveTest that emits the correct states", () {
119 declarer.test("test", () {}, skip: true); 119 declarer.test("test", () {}, skip: true);
120 120
121 var engine = new Engine.withSuites([new Suite(declarer.tests)]); 121 var engine = new Engine.withSuites([new RunnerSuite(declarer.tests)]);
122 122
123 engine.onTestStarted.listen(expectAsync((liveTest) { 123 engine.onTestStarted.listen(expectAsync((liveTest) {
124 expect(liveTest, same(engine.liveTests.single)); 124 expect(liveTest, same(engine.liveTests.single));
125 expect(liveTest.test, equals(declarer.tests.single)); 125 expect(liveTest.test, equals(declarer.tests.single));
126 126
127 var first = true; 127 var first = true;
128 liveTest.onStateChange.listen(expectAsync((state) { 128 liveTest.onStateChange.listen(expectAsync((state) {
129 expect(state, equals(first 129 expect(state, equals(first
130 ? const State(Status.running, Result.success) 130 ? const State(Status.running, Result.success)
131 : const State(Status.complete, Result.success))); 131 : const State(Status.complete, Result.success)));
132 first = false; 132 first = false;
133 }, count: 2)); 133 }, count: 2));
134 134
135 expect(liveTest.onComplete, completes); 135 expect(liveTest.onComplete, completes);
136 })); 136 }));
137 137
138 return engine.run(); 138 return engine.run();
139 }); 139 });
140 }); 140 });
141 } 141 }
OLDNEW
« no previous file with comments | « lib/test.dart ('k') | test/runner/load_suite_test.dart » ('j') | no next file with comments »

Powered by Google App Engine