Type Function variables with actual function signatures (#6944)

flutter · Dec 11, 2023 · 88709a8 · 88709a8
1 parent af32dfb
commit 88709a8
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/packages/devtools_app/test/test_infra/fixtures/flutter_error_app/lib/main.dart b/packages/devtools_app/test/test_infra/fixtures/flutter_error_app/lib/main.dart
@@ -82,7 +82,7 @@ class _MyHomePageState extends State<MyHomePage> {
   }
 }
 
-Map<String, Function> navigateCallbacks = {};
+Map<String, void Function()> navigateCallbacks = {};
 
 // Hook to navigate to a specific screen.
 void navigateToScreen(String title) {

diff --git a/packages/devtools_app_shared/lib/src/ui/dialogs.dart b/packages/devtools_app_shared/lib/src/ui/dialogs.dart
@@ -231,7 +231,7 @@ final class DialogCancelButton extends StatelessWidget {
 final class DialogApplyButton extends StatelessWidget {
   const DialogApplyButton({super.key, required this.onPressed});
 
-  final Function onPressed;
+  final void Function() onPressed;
 
   @override
   Widget build(BuildContext context) {

diff --git a/packages/devtools_test/lib/src/helpers/utils.dart b/packages/devtools_test/lib/src/helpers/utils.dart
@@ -37,8 +37,8 @@ final screenIds = <String>[
 /// Tests that `listener` has actually been invoked.
 Future<void> addListenerScope({
   required Listenable listenable,
-  required Function listener,
-  required Function callback,
+  required void Function() listener,
+  required Future<void> Function() callback,
 }) async {
   bool listenerCalled = false;
   void listenerWrapped() {

diff --git a/packages/devtools_test/lib/src/mocks/fake_service_manager.dart b/packages/devtools_test/lib/src/mocks/fake_service_manager.dart
@@ -156,7 +156,7 @@ class FakeServiceManager extends Fake
 
   final List<String> availableLibraries;
 
-  final Function? onVmServiceOpened;
+  final void Function()? onVmServiceOpened;
 
   final Map<String, Response> serviceExtensionResponses;