Skip to content

Commit c5d017c

Browse files
authored
Annotate checkThat with usResult (#1882)
It would never be useful to set up a `Subject` and not check any expectations on it.
1 parent 4487808 commit c5d017c

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

pkgs/checks/lib/src/checks.dart

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
// TODO Add doc about how failure strings work.
66
import 'dart:async';
77

8+
import 'package:meta/meta.dart' as meta;
89
import 'package:test_api/hooks.dart';
910

1011
import 'describe.dart';
@@ -58,6 +59,7 @@ extension Skip<T> on Subject<T> {
5859
/// ```dart
5960
/// checkThat(actual).equals(expected);
6061
/// ```
62+
@meta.useResult
6163
Subject<T> checkThat<T>(T value, {String? because}) =>
6264
Subject._(_TestContext._root(
6365
value: _Present(value),

pkgs/checks/pubspec.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ environment:
1010

1111
dependencies:
1212
async: ^2.8.0
13+
meta: ^1.9.0
1314
test_api: ^0.4.0
1415

1516
dev_dependencies:

0 commit comments

Comments
 (0)