Skip to content

Commit 2c1bc9f

Browse files
committed
🐛 Fixed issues where async tests where execute correctly
1 parent 1369d34 commit 2c1bc9f

File tree

5 files changed

+27
-27
lines changed

5 files changed

+27
-27
lines changed

lib/src/test_options/group_test_options.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class GroupTestOptions {
2424
final int? retry;
2525

2626
void groupTest(dynamic Function() body) {
27-
dart_test.group(
27+
dart_test.group(
2828
description,
2929
() {
3030
if (setUp != null) {

lib/src/test_options/test_options.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ class TestOptions {
1515
final Map<String, dynamic>? onPlatform;
1616
final int? retry;
1717

18-
void test(Object description, dynamic Function() body) {
18+
dynamic test(Object description, dynamic Function() body) async {
1919
dart_test.test(
2020
description,
2121
body,

lib/src/test_parameters.dart

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ abstract interface class TestParameters {
4141
abstract final Function body;
4242
abstract final int count;
4343

44-
void mapBody<R>(Iterable<R> values);
44+
dynamic mapBody<R>(Iterable<R> values);
4545
}
4646

4747
/// Test body class that handles 1 test parameter value.
@@ -61,9 +61,9 @@ class TestParameters1<A1> implements TestParameters {
6161
final int count = 1;
6262

6363
@override
64-
void mapBody<R>(Iterable<R> values) {
64+
dynamic mapBody<R>(Iterable<R> values) {
6565
final A1 a1 = castElementAt(values, 0);
66-
body(a1);
66+
return body(a1);
6767
}
6868
}
6969

@@ -84,10 +84,10 @@ class TestParameters2<A1, A2> implements TestParameters {
8484
final int count = 2;
8585

8686
@override
87-
void mapBody<R>(Iterable<R> values) {
87+
dynamic mapBody<R>(Iterable<R> values) {
8888
final A1 a1 = castElementAt(values, 0);
8989
final A2 a2 = castElementAt(values, 1);
90-
body(a1, a2);
90+
return body(a1, a2);
9191
}
9292
}
9393

@@ -108,11 +108,11 @@ class TestParameters3<A1, A2, A3> implements TestParameters {
108108
final int count = 3;
109109

110110
@override
111-
void mapBody<R>(Iterable<R> values) {
111+
dynamic mapBody<R>(Iterable<R> values) {
112112
final A1 a1 = castElementAt(values, 0);
113113
final A2 a2 = castElementAt(values, 1);
114114
final A3 a3 = castElementAt(values, 2);
115-
body(a1, a2, a3);
115+
return body(a1, a2, a3);
116116
}
117117
}
118118

@@ -133,12 +133,12 @@ class TestParameters4<A1, A2, A3, A4> implements TestParameters {
133133
final int count = 4;
134134

135135
@override
136-
void mapBody<R>(Iterable<R> values) {
136+
dynamic mapBody<R>(Iterable<R> values) {
137137
final A1 a1 = castElementAt(values, 0);
138138
final A2 a2 = castElementAt(values, 1);
139139
final A3 a3 = castElementAt(values, 2);
140140
final A4 a4 = castElementAt(values, 3);
141-
body(a1, a2, a3, a4);
141+
return body(a1, a2, a3, a4);
142142
}
143143
}
144144

@@ -159,13 +159,13 @@ class TestParameters5<A1, A2, A3, A4, A5> implements TestParameters {
159159
final int count = 5;
160160

161161
@override
162-
void mapBody<R>(Iterable<R> values) {
162+
dynamic mapBody<R>(Iterable<R> values) {
163163
final A1 a1 = castElementAt(values, 0);
164164
final A2 a2 = castElementAt(values, 1);
165165
final A3 a3 = castElementAt(values, 2);
166166
final A4 a4 = castElementAt(values, 3);
167167
final A5 a5 = castElementAt(values, 4);
168-
body(a1, a2, a3, a4, a5);
168+
return body(a1, a2, a3, a4, a5);
169169
}
170170
}
171171

@@ -186,14 +186,14 @@ class TestParameters6<A1, A2, A3, A4, A5, A6> implements TestParameters {
186186
final int count = 6;
187187

188188
@override
189-
void mapBody<R>(Iterable<R> values) {
189+
dynamic mapBody<R>(Iterable<R> values) {
190190
final A1 a1 = castElementAt(values, 0);
191191
final A2 a2 = castElementAt(values, 1);
192192
final A3 a3 = castElementAt(values, 2);
193193
final A4 a4 = castElementAt(values, 3);
194194
final A5 a5 = castElementAt(values, 4);
195195
final A6 a6 = castElementAt(values, 5);
196-
body(a1, a2, a3, a4, a5, a6);
196+
return body(a1, a2, a3, a4, a5, a6);
197197
}
198198
}
199199

@@ -214,15 +214,15 @@ class TestParameters7<A1, A2, A3, A4, A5, A6, A7> implements TestParameters {
214214
final int count = 7;
215215

216216
@override
217-
void mapBody<R>(Iterable<R> values) {
217+
dynamic mapBody<R>(Iterable<R> values) {
218218
final A1 a1 = castElementAt(values, 0);
219219
final A2 a2 = castElementAt(values, 1);
220220
final A3 a3 = castElementAt(values, 2);
221221
final A4 a4 = castElementAt(values, 3);
222222
final A5 a5 = castElementAt(values, 4);
223223
final A6 a6 = castElementAt(values, 5);
224224
final A7 a7 = castElementAt(values, 6);
225-
body(a1, a2, a3, a4, a5, a6, a7);
225+
return body(a1, a2, a3, a4, a5, a6, a7);
226226
}
227227
}
228228

@@ -244,7 +244,7 @@ class TestParameters8<A1, A2, A3, A4, A5, A6, A7, A8>
244244
final int count = 8;
245245

246246
@override
247-
void mapBody<R>(Iterable<R> values) {
247+
dynamic mapBody<R>(Iterable<R> values) {
248248
final A1 a1 = castElementAt(values, 0);
249249
final A2 a2 = castElementAt(values, 1);
250250
final A3 a3 = castElementAt(values, 2);
@@ -253,7 +253,7 @@ class TestParameters8<A1, A2, A3, A4, A5, A6, A7, A8>
253253
final A6 a6 = castElementAt(values, 5);
254254
final A7 a7 = castElementAt(values, 6);
255255
final A8 a8 = castElementAt(values, 7);
256-
body(a1, a2, a3, a4, a5, a6, a7, a8);
256+
return body(a1, a2, a3, a4, a5, a6, a7, a8);
257257
}
258258
}
259259

@@ -275,7 +275,7 @@ class TestParameters9<A1, A2, A3, A4, A5, A6, A7, A8, A9>
275275
final int count = 9;
276276

277277
@override
278-
void mapBody<R>(Iterable<R> values) {
278+
dynamic mapBody<R>(Iterable<R> values) {
279279
final A1 a1 = castElementAt(values, 0);
280280
final A2 a2 = castElementAt(values, 1);
281281
final A3 a3 = castElementAt(values, 2);
@@ -285,7 +285,7 @@ class TestParameters9<A1, A2, A3, A4, A5, A6, A7, A8, A9>
285285
final A7 a7 = castElementAt(values, 6);
286286
final A8 a8 = castElementAt(values, 7);
287287
final A9 a9 = castElementAt(values, 8);
288-
body(a1, a2, a3, a4, a5, a6, a7, a8, a9);
288+
return body(a1, a2, a3, a4, a5, a6, a7, a8, a9);
289289
}
290290
}
291291

@@ -307,7 +307,7 @@ class TestParameters10<A1, A2, A3, A4, A5, A6, A7, A8, A9, A10>
307307
final int count = 10;
308308

309309
@override
310-
void mapBody<R>(Iterable<R> values) {
310+
dynamic mapBody<R>(Iterable<R> values) {
311311
final A1 a1 = castElementAt(values, 0);
312312
final A2 a2 = castElementAt(values, 1);
313313
final A3 a3 = castElementAt(values, 2);
@@ -318,6 +318,6 @@ class TestParameters10<A1, A2, A3, A4, A5, A6, A7, A8, A9, A10>
318318
final A8 a8 = castElementAt(values, 7);
319319
final A9 a9 = castElementAt(values, 8);
320320
final A10 a10 = castElementAt(values, 9);
321-
body(a1, a2, a3, a4, a5, a6, a7, a8, a9, a10);
321+
return body(a1, a2, a3, a4, a5, a6, a7, a8, a9, a10);
322322
}
323323
}

lib/src/util/test_value_helpers.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ void mapTests(
1010
for (final ValueWithTestOptions value in values) {
1111
value.testOptions.test('$value', () {
1212
validityCheck(value, length);
13-
body(value.value);
13+
return body(value.value);
1414
});
1515
}
1616
}
@@ -23,7 +23,7 @@ void mapGroups(
2323
for (final ValueWithTestOptions value in values) {
2424
value.toGroupOptions.groupTest(() {
2525
validityCheck(value, length);
26-
body(value.value);
26+
return body(value.value);
2727
});
2828
}
2929
}

lib/src/value_source.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ class _ValueSourceImpl implements ValueSource {
2929
) {
3030
_groupTestOptions.groupTest(() {
3131
mapTests(_values, body.count, (value) {
32-
body.mapBody(value);
32+
return body.mapBody(value);
3333
});
3434
});
3535
}
@@ -38,7 +38,7 @@ class _ValueSourceImpl implements ValueSource {
3838
void executeGroup(TestParameters body) {
3939
_groupTestOptions.groupTest(() {
4040
mapGroups(_values, body.count, (value) {
41-
body.mapBody(value);
41+
return body.mapBody(value);
4242
});
4343
});
4444
}

0 commit comments

Comments
 (0)