mirror of
https://github.com/flutter/flutter.git
synced 2026-02-20 02:29:02 +08:00
There is a lot of cross-library importing in the framework unit tests that I'd like to clean up, see the design doc for more: https://docs.google.com/document/d/1UHxALQqCbmgjnM1RNV9xE2pK3IGyx-UktGX1D7hYCjs/edit?tab=t.0 This PR cleans up a few obvious instances and adds TODOs for others. I created this while doing an investigation for the design doc linked above. I hope that we'll be able to follow up with fixes for all of the problematic tests (tracked in the issue below). Part of https://github.com/flutter/flutter/issues/177028
78 lines
2.3 KiB
Dart
78 lines
2.3 KiB
Dart
// Copyright 2014 The Flutter Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
import 'package:flutter/widgets.dart';
|
|
import 'package:flutter_test/flutter_test.dart';
|
|
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
|
|
|
|
class InvalidOnInitLifecycleWidget extends StatefulWidget {
|
|
const InvalidOnInitLifecycleWidget({super.key});
|
|
|
|
@override
|
|
InvalidOnInitLifecycleWidgetState createState() => InvalidOnInitLifecycleWidgetState();
|
|
}
|
|
|
|
class InvalidOnInitLifecycleWidgetState extends State<InvalidOnInitLifecycleWidget> {
|
|
@override
|
|
Future<void> initState() async {
|
|
super.initState();
|
|
}
|
|
|
|
@override
|
|
Widget build(BuildContext context) {
|
|
return Container();
|
|
}
|
|
}
|
|
|
|
class InvalidDidUpdateWidgetLifecycleWidget extends StatefulWidget {
|
|
const InvalidDidUpdateWidgetLifecycleWidget({super.key, required this.color});
|
|
|
|
final Color color;
|
|
|
|
@override
|
|
InvalidDidUpdateWidgetLifecycleWidgetState createState() =>
|
|
InvalidDidUpdateWidgetLifecycleWidgetState();
|
|
}
|
|
|
|
class InvalidDidUpdateWidgetLifecycleWidgetState
|
|
extends State<InvalidDidUpdateWidgetLifecycleWidget> {
|
|
@override
|
|
Future<void> didUpdateWidget(InvalidDidUpdateWidgetLifecycleWidget oldWidget) async {
|
|
super.didUpdateWidget(oldWidget);
|
|
}
|
|
|
|
@override
|
|
Widget build(BuildContext context) {
|
|
return ColoredBox(color: widget.color);
|
|
}
|
|
}
|
|
|
|
void main() {
|
|
const red = Color(0xffff0000);
|
|
const green = Color(0xff00ff00);
|
|
|
|
testWidgets(
|
|
'async onInit throws FlutterError',
|
|
experimentalLeakTesting: LeakTesting.settings
|
|
.withIgnoredAll(), // leaking by design because of exception
|
|
(WidgetTester tester) async {
|
|
await tester.pumpWidget(const InvalidOnInitLifecycleWidget());
|
|
|
|
expect(tester.takeException(), isFlutterError);
|
|
},
|
|
);
|
|
|
|
testWidgets(
|
|
'async didUpdateWidget throws FlutterError',
|
|
experimentalLeakTesting: LeakTesting.settings
|
|
.withIgnoredAll(), // leaking by design because of exception
|
|
(WidgetTester tester) async {
|
|
await tester.pumpWidget(const InvalidDidUpdateWidgetLifecycleWidget(color: green));
|
|
await tester.pumpWidget(const InvalidDidUpdateWidgetLifecycleWidget(color: red));
|
|
|
|
expect(tester.takeException(), isFlutterError);
|
|
},
|
|
);
|
|
}
|