Skip to content

Commit 95bc66f

Browse files
committed
Add todo for flutter integration tests
1 parent 4532e9a commit 95bc66f

1 file changed

Lines changed: 11 additions & 0 deletions

File tree

netcoresync_moor/test/integration_tests/netcoresync_sync_test.dart

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,17 @@ import '../utils/net_core_test_server.dart';
1313
import '../utils/helper.dart';
1414
import '../data/database.dart';
1515

16+
// TODO: 210830: Consider adding a real Flutter integration test in
17+
// `netcoresync_moor` tests, because this package is supposed to be used in a
18+
// real Flutter application, so we need to ensure it always works properly. One
19+
// case happened earlier where in Generator, to score more pub points, the
20+
// `analyzer` package is upgraded from 1.7.0 to 2.0.0, which by transitive deps
21+
// also silently upgrades `meta` to 1.7.0. This is fine in the libraries,
22+
// because they only depends on Dart and have no dependencies to Flutter, but,
23+
// in a Flutter application, it turns out that Flutter (at the time of writing)
24+
// still depends on `meta` 1.3.0, thus breaks the code generation in the client
25+
// project.
26+
1627
void main() async {
1728
bool testPrint = true;
1829
void logTest(Object? object) {

0 commit comments

Comments
 (0)