Sync analysis_options.yaml & cleanups (#117327)

Author: goderbauer

analysis_options.yaml

@@ -88,12 +88,14 @@ linter:
     # - cascade_invocations # doesn't match the typical style of this repo
     - cast_nullable_to_non_nullable
     # - close_sinks # not reliable enough
+    - collection_methods_unrelated_type
     - combinators_ordering
     # - comment_references # blocked on https://github.com/dart-lang/linter/issues/1142
     - conditional_uri_does_not_exist
     # - constant_identifier_names # needs an opt-out https://github.com/dart-lang/linter/issues/204
     - control_flow_in_finally
     - curly_braces_in_flow_control_structures
+    # - dangling_library_doc_comments # not yet tested
     - depend_on_referenced_packages
     - deprecated_consistency
     # - diagnostic_describe_all_properties # enabled only at the framework level (packages/flutter/lib)
@@ -103,15 +105,18 @@ linter:
     - empty_catches
     - empty_constructor_bodies
     - empty_statements
+    - enable_null_safety
     - eol_at_end_of_file
     - exhaustive_cases
     - file_names
     - flutter_style_todos
     - hash_and_equals
     - implementation_imports
+    - implicit_call_tearoffs
     - iterable_contains_unrelated_type
     # - join_return_with_assignment # not required by flutter style
     - leading_newlines_in_multiline_strings
+    # - library_annotations # not yet tested
     - library_names
     - library_prefixes
     - library_private_types_in_public_api
@@ -204,6 +209,7 @@ linter:
     - unnecessary_getters_setters
     # - unnecessary_lambdas # has false positives: https://github.com/dart-lang/linter/issues/498
     - unnecessary_late
+    - unnecessary_library_directive
     - unnecessary_new
     - unnecessary_null_aware_assignments
     - unnecessary_null_aware_operator_on_extension_on_nullable
@@ -218,6 +224,7 @@ linter:
     - unnecessary_string_interpolations
     - unnecessary_this
     - unnecessary_to_list_in_spreads
+    - unreachable_from_main
     - unrelated_type_equality_checks
     - unsafe_html
     - use_build_context_synchronously
@@ -235,6 +242,7 @@ linter:
     - use_rethrow_when_possible
     - use_setters_to_change_properties
     # - use_string_buffers # has false positives: https://github.com/dart-lang/sdk/issues/34182
+    - use_string_in_part_of_directives
     - use_super_parameters
     - use_test_throws_matchers
     # - use_to_and_as_if_applicable # has false positives, so we prefer to catch this by code-review

dev/benchmarks/test_apps/stocks/lib/main.dart

@@ -2,8 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-library stocks;
-
 import 'package:flutter/material.dart';
 import 'package:flutter/rendering.dart' show
   debugPaintBaselinesEnabled,

dev/bots/analyze_snippet_code.dart

@@ -457,6 +457,7 @@ class _SnippetChecker {
     '// ignore_for_file: directives_ordering',
     '// ignore_for_file: prefer_final_locals',
     '// ignore_for_file: unnecessary_import',
+    '// ignore_for_file: unreachable_from_main',
     '// ignore_for_file: unused_element',
     '// ignore_for_file: unused_local_variable',
   ];

dev/bots/post_process_docs.dart

@@ -12,9 +12,6 @@ import 'package:platform/platform.dart' as platform;
 
 import 'package:process/process.dart';
 
-const String kDocsRoot = 'dev/docs';
-const String kPublishRoot = '$kDocsRoot/doc';
-
 class CommandException implements Exception {}
 
 Future<void> main() async {

dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart

@@ -4,8 +4,7 @@
 
 // @dart = 2.12
 
-// This is a dummy dart:ui package for the sample code analyzer tests to use.
-
+/// This is a dummy dart:ui package for the sample code analyzer tests to use.
 library dart.ui;
 
 /// Bla bla bla bla bla bla bla bla bla.

dev/bots/test/analyze_snippet_code_test.dart

@@ -33,10 +33,10 @@ const List<String> expectedMainErrors = <String>[
 ];
 
 const List<String> expectedUiErrors = <String>[
-  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:15:7: (top-level declaration) (prefer_typing_uninitialized_variables)',
-  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:15:7: (top-level declaration) (missing_const_final_var_or_type)',
-  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:17:20: (top-level declaration) (prefer_final_fields)',
-  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:17:20: (top-level declaration) (unused_field)',
+  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:14:7: (top-level declaration) (prefer_typing_uninitialized_variables)',
+  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:14:7: (top-level declaration) (missing_const_final_var_or_type)',
+  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:16:20: (top-level declaration) (prefer_final_fields)',
+  'dev/bots/test/analyze-snippet-code-test-dart-ui/ui.dart:16:20: (top-level declaration) (unused_field)',
 ];
 
 final RegExp errorPrefixRE = RegExp(r'^([-a-z0-9/_.:]+): .*(\([-a-z_ ]+\) \([-a-z_ ]+\))$');

dev/devicelab/bin/tasks/plugin_test.dart

@@ -7,18 +7,18 @@ import 'package:flutter_devicelab/tasks/plugin_tests.dart';
 
 Future<void> main() async {
   await task(combine(<TaskFunction>[
-    PluginTest('apk', <String>['-a', 'java', '--platforms=android']),
-    PluginTest('apk', <String>['-a', 'kotlin', '--platforms=android']),
+    PluginTest('apk', <String>['-a', 'java', '--platforms=android']).call,
+    PluginTest('apk', <String>['-a', 'kotlin', '--platforms=android']).call,
     // These create the plugins using the new v2 plugin templates but create the
     // apps using the old v1 embedding app templates to make sure new plugins
     // are by default backward compatible.
     PluginTest('apk', <String>['-a', 'java', '--platforms=android'], pluginCreateEnvironment:
-        <String, String>{'ENABLE_ANDROID_EMBEDDING_V2': 'true'}),
+        <String, String>{'ENABLE_ANDROID_EMBEDDING_V2': 'true'}).call,
     PluginTest('apk', <String>['-a', 'kotlin', '--platforms=android'], pluginCreateEnvironment:
-        <String, String>{'ENABLE_ANDROID_EMBEDDING_V2': 'true'}),
+        <String, String>{'ENABLE_ANDROID_EMBEDDING_V2': 'true'}).call,
     // Test that Dart-only plugins are supported.
-    PluginTest('apk', <String>['--platforms=android'], dartOnlyPlugin: true),
+    PluginTest('apk', <String>['--platforms=android'], dartOnlyPlugin: true).call,
     // Test that FFI plugins are supported.
-    PluginTest('apk', <String>['--platforms=android'], template: 'plugin_ffi'),
+    PluginTest('apk', <String>['--platforms=android'], template: 'plugin_ffi').call,
   ]));
 }

dev/devicelab/bin/tasks/plugin_test_ios.dart

@@ -7,14 +7,14 @@ import 'package:flutter_devicelab/tasks/plugin_tests.dart';
 
 Future<void> main() async {
   await task(combine(<TaskFunction>[
-    PluginTest('ios', <String>['-i', 'objc', '--platforms=ios']),
-    PluginTest('ios', <String>['-i', 'swift', '--platforms=ios']),
-    PluginTest('macos', <String>['--platforms=macos']),
+    PluginTest('ios', <String>['-i', 'objc', '--platforms=ios']).call,
+    PluginTest('ios', <String>['-i', 'swift', '--platforms=ios']).call,
+    PluginTest('macos', <String>['--platforms=macos']).call,
     // Test that Dart-only plugins are supported.
-    PluginTest('ios', <String>['--platforms=ios'], dartOnlyPlugin: true),
-    PluginTest('macos', <String>['--platforms=macos'], dartOnlyPlugin: true),
+    PluginTest('ios', <String>['--platforms=ios'], dartOnlyPlugin: true).call,
+    PluginTest('macos', <String>['--platforms=macos'], dartOnlyPlugin: true).call,
     // Test that FFI plugins are supported.
-    PluginTest('ios', <String>['--platforms=ios'], template: 'plugin_ffi'),
-    PluginTest('macos', <String>['--platforms=macos'], template: 'plugin_ffi'),
+    PluginTest('ios', <String>['--platforms=ios'], template: 'plugin_ffi').call,
+    PluginTest('macos', <String>['--platforms=macos'], template: 'plugin_ffi').call,
   ]));
 }

dev/devicelab/bin/tasks/smoke_test_build_test.dart

@@ -10,7 +10,7 @@ import 'package:flutter_devicelab/tasks/build_test_task.dart';
 /// Smoke test of a successful task.
 Future<void> main(List<String> args) async {
   deviceOperatingSystem = DeviceOperatingSystem.fake;
-  await task(FakeBuildTestTask(args));
+  await task(FakeBuildTestTask(args).call);
 }
 
 class FakeBuildTestTask extends BuildTestTask {

dev/devicelab/lib/tasks/gallery.dart

@@ -18,11 +18,11 @@ final Directory galleryDirectory = dir('${flutterDirectory.path}/dev/integration
 ///
 /// https://github.com/flutter/flutter/issues/103542
 TaskFunction createGalleryTransitionBuildTest(List<String> args, {bool semanticsEnabled = false}) {
-  return GalleryTransitionBuildTest(args, semanticsEnabled: semanticsEnabled);
+  return GalleryTransitionBuildTest(args, semanticsEnabled: semanticsEnabled).call;
 }
 
 TaskFunction createGalleryTransitionTest({bool semanticsEnabled = false}) {
-  return GalleryTransitionTest(semanticsEnabled: semanticsEnabled);
+  return GalleryTransitionTest(semanticsEnabled: semanticsEnabled).call;
 }
 
 TaskFunction createGalleryTransitionE2EBuildTest(
@@ -39,7 +39,7 @@ TaskFunction createGalleryTransitionE2EBuildTest(
     timelineTraceFile: null,
     driverFile: 'transitions_perf_e2e_test',
     enableImpeller: enableImpeller,
-  );
+  ).call;
 }
 
 TaskFunction createGalleryTransitionE2ETest({
@@ -56,7 +56,7 @@ TaskFunction createGalleryTransitionE2ETest({
     timelineTraceFile: null,
     driverFile: 'transitions_perf_e2e_test',
     enableImpeller: enableImpeller,
-  );
+  ).call;
 }
 
 TaskFunction createGalleryTransitionHybridBuildTest(
@@ -67,7 +67,7 @@ TaskFunction createGalleryTransitionHybridBuildTest(
     args,
     semanticsEnabled: semanticsEnabled,
     driverFile: semanticsEnabled ? 'transitions_perf_hybrid_with_semantics_test' : 'transitions_perf_hybrid_test',
-  );
+  ).call;
 }
 
 TaskFunction createGalleryTransitionHybridTest({bool semanticsEnabled = false}) {
@@ -76,7 +76,7 @@ TaskFunction createGalleryTransitionHybridTest({bool semanticsEnabled = false})
     driverFile: semanticsEnabled
         ? 'transitions_perf_hybrid_with_semantics_test'
         : 'transitions_perf_hybrid_test',
-  );
+  ).call;
 }
 
 class GalleryTransitionTest {