Bump dart_flutter_team_lints from 2.1.1 to ^3.0.0 in all packages (#2228

) * Bump dart_flutter_team_lints from 2.1.1 to 3.1.0 in /pkgs/test_core * remove unnecessary library names * update min SDK constraints to 3.2 Bumps [dart_flutter_team_lints](https://github.com/dart-lang/ecosystem/tree/main/pkgs) from 2.1.1 to 3.1.0. - [Release notes](https://github.com/dart-lang/ecosystem/releases) - [Commits](https://github.com/dart-lang/ecosystem/commits/dart_flutter_team_lints-v3.1.0/pkgs) --- updated-dependencies: - dependency-name: dart_flutter_team_lints dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jake Macdonald <jakemac@google.com>
dart-lang · May 15, 2024 · 4b6f029 · 4b6f029
1 parent 84d2a2b
commit 4b6f029
Show file tree

Hide file tree

Showing 25 changed files with 119 additions and 110 deletions.
diff --git a/.github/workflows/dart.yml b/.github/workflows/dart.yml
diff --git a/integration_tests/regression/pubspec.yaml b/integration_tests/regression/pubspec.yaml
@@ -1,11 +1,11 @@
 name: regression_tests
 publish_to: none
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 dependencies:
   test: any
 dev_dependencies:
-  dart_flutter_team_lints: ^2.1.1
+  dart_flutter_team_lints: ^3.0.0
 dependency_overrides:
   test:
     path: ../../pkgs/test

diff --git a/integration_tests/spawn_hybrid/pubspec.yaml b/integration_tests/spawn_hybrid/pubspec.yaml
@@ -1,13 +1,13 @@
 name: spawn_hybrid
 publish_to: none
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 dependencies:
   async: ^2.9.0
   path: ^1.8.2
   stream_channel: ^2.1.0
 dev_dependencies:
-  dart_flutter_team_lints: ^2.0.0
+  dart_flutter_team_lints: ^3.0.0
   other_package:
     path: other_package/
   test: any

diff --git a/integration_tests/wasm/pubspec.yaml b/integration_tests/wasm/pubspec.yaml
@@ -1,9 +1,9 @@
 name: wasm_tests
 publish_to: none
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 dev_dependencies:
-  dart_flutter_team_lints: ^2.1.1
+  dart_flutter_team_lints: ^3.0.0
   test: any
 dependency_overrides:
   test:

diff --git a/pkgs/checks/CHANGELOG.md b/pkgs/checks/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 0.3.1-wip
+
+- Update min SDK constraint to 3.2.0.
+
 ## 0.3.0
 
 -   **Breaking Changes**

diff --git a/pkgs/checks/lib/src/checks.dart b/pkgs/checks/lib/src/checks.dart
@@ -574,9 +574,9 @@ final class Extracted<T> {
   }
 
   Extracted<T> _fillActual(Object? actual) => _rejection == null ||
-          _rejection!.actual.isNotEmpty
+          _rejection.actual.isNotEmpty
       ? this
-      : Extracted.rejection(actual: literal(actual), which: _rejection!.which);
+      : Extracted.rejection(actual: literal(actual), which: _rejection.which);
 }
 
 abstract interface class _Optional<T> {
@@ -796,7 +796,7 @@ final class _TestContext<T> implements Context<T>, _ClauseDescription {
   _TestContext get _root {
     _TestContext<dynamic> current = this;
     while (current._parent != null) {
-      current = current._parent!;
+      current = current._parent;
     }
     return current;
   }

diff --git a/pkgs/checks/pubspec.yaml b/pkgs/checks/pubspec.yaml
@@ -1,18 +1,18 @@
 name: checks
-version: 0.3.0
+version: 0.3.1-wip
 description: >-
   A framework for checking values against expectations and building custom
   expectations.
 repository: https://github.com/dart-lang/test/tree/master/pkgs/checks
 
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 
 dependencies:
   async: ^2.8.0
   meta: ^1.9.0
   test_api: ">=0.5.0 <0.8.0"
 
 dev_dependencies:
-  dart_flutter_team_lints: ^2.0.0
+  dart_flutter_team_lints: ^3.0.0
   test: ^1.21.3
diff --git a/pkgs/test/CHANGELOG.md b/pkgs/test/CHANGELOG.md
@@ -1,6 +1,7 @@
 ## 1.25.6-wip
 
 * Point API doc links to `package:test` canonical libraries.
+* Update min SDK constraint to 3.2.0.
 
 ## 1.25.5
 

diff --git a/pkgs/test/lib/src/runner/node/socket_channel.dart b/pkgs/test/lib/src/runner/node/socket_channel.dart
@@ -3,7 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 @JS()
-library node;
+library;
 
 import 'dart:async';
 import 'dart:convert';

diff --git a/pkgs/test/pubspec.yaml b/pkgs/test/pubspec.yaml
@@ -5,7 +5,7 @@ description: >-
 repository: https://github.com/dart-lang/test/tree/master/pkgs/test
 
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 
 dependencies:
   analyzer: '>=5.12.0 <7.0.0'
@@ -34,16 +34,16 @@ dependencies:
   stream_channel: ^2.1.0
 
   # Use an exact version until the test_api and test_core package are stable.
-  test_api: 0.7.1
-  test_core: 0.6.2
+  test_api: 0.7.2
+  test_core: 0.6.3
 
   typed_data: ^1.3.0
   web_socket_channel: '>=2.0.0 <4.0.0'
   webkit_inspection_protocol: ^1.0.0
   yaml: ^3.0.0
 
 dev_dependencies:
-  dart_flutter_team_lints: ^2.1.1
+  dart_flutter_team_lints: ^3.0.0
   fake_async: ^1.0.0
   glob: ^2.0.0
   test_descriptor: ^2.0.0

diff --git a/pkgs/test/tool/host.dart b/pkgs/test/tool/host.dart
@@ -3,7 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 @JS()
-library test.host;
+library;
 
 import 'dart:async';
 import 'dart:convert';

diff --git a/pkgs/test_api/CHANGELOG.md b/pkgs/test_api/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 0.7.2-wip
+
+* Update min SDK constraint to 3.2.0.
+
 ## 0.7.1
 
 - Added [`@doNotSubmit`](https://pub.dev/documentation/meta/latest/meta/doNotSubmit-constant.html) to `test(solo: ...)` and `group(solo: ...)`. In

diff --git a/pkgs/test_api/lib/fake.dart b/pkgs/test_api/lib/fake.dart
@@ -8,6 +8,6 @@
 
 @Deprecated('package:test_api is not intended for general use. '
     'Please use package:test.')
-library test_api.fake;
+library;
 
 export 'src/frontend/fake.dart';
diff --git a/pkgs/test_api/lib/scaffolding.dart b/pkgs/test_api/lib/scaffolding.dart
@@ -8,7 +8,7 @@
 /// {@canonicalFor tags.Tags}
 /// {@canonicalFor test_on.TestOn}
 /// {@canonicalFor timeout.Timeout}
-library test_api.scaffolding;
+library;
 
 export 'src/backend/configuration/on_platform.dart' show OnPlatform;
 export 'src/backend/configuration/retry.dart' show Retry;

diff --git a/pkgs/test_api/lib/src/backend/declarer.dart b/pkgs/test_api/lib/src/backend/declarer.dart
@@ -236,7 +236,7 @@ class Declarer {
     _checkNotBuilt('group');
 
     final fullTestPrefix = _prefix(name);
-    if (_fullTestName != null && !_fullTestName!.startsWith(fullTestPrefix)) {
+    if (_fullTestName != null && !_fullTestName.startsWith(fullTestPrefix)) {
       return;
     }
 
@@ -348,7 +348,7 @@ class Declarer {
   /// If no set-up functions are declared, this returns a [Future] that
   /// completes immediately.
   Future _runSetUps() async {
-    if (_parent != null) await _parent!._runSetUps();
+    if (_parent != null) await _parent._runSetUps();
     // TODO: why does type inference not work here?
     await Future.forEach<Function>(_setUps, (setUp) => setUp());
   }

diff --git a/pkgs/test_api/lib/src/backend/remote_listener.dart b/pkgs/test_api/lib/src/backend/remote_listener.dart
@@ -261,7 +261,7 @@ final class RemoteListener {
     });
 
     liveTest.onMessage.listen((message) {
-      if (_printZone != null) _printZone!.print(message.text);
+      if (_printZone != null) _printZone.print(message.text);
       channel.sink.add({
         'type': 'message',
         'message-type': message.type.name,

diff --git a/pkgs/test_api/lib/test_api.dart b/pkgs/test_api/lib/test_api.dart
@@ -4,7 +4,7 @@
 
 @Deprecated('package:test_api is not intended for general use. '
     'Please use package:test.')
-library test_api;
+library;
 
 export 'hooks.dart' show TestFailure;
 export 'scaffolding.dart';
diff --git a/pkgs/test_api/pubspec.yaml b/pkgs/test_api/pubspec.yaml
@@ -1,11 +1,11 @@
 name: test_api
-version: 0.7.1
+version: 0.7.2-wip
 description: >-
   The user facing API for structuring Dart tests and checking expectations.
 repository: https://github.com/dart-lang/test/tree/master/pkgs/test_api
 
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 
 dependencies:
   async: ^2.5.0
@@ -20,7 +20,7 @@ dependencies:
 
 dev_dependencies:
   analyzer: '>=2.1.0 <7.0.0'
-  dart_flutter_team_lints: ^2.0.0
+  dart_flutter_team_lints: ^3.0.0
   fake_async: ^1.2.0
   glob: ^2.0.0
   graphs: ^2.0.0

diff --git a/pkgs/test_core/CHANGELOG.md b/pkgs/test_core/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 0.6.3-wip
+
+* Update min SDK constraint to 3.2.0.
+
 ## 0.6.2
 
 * Add `@doNotSubmit` to more declarations of the `solo` parameter.

diff --git a/pkgs/test_core/lib/backend.dart b/pkgs/test_core/lib/backend.dart
@@ -4,7 +4,7 @@
 
 @Deprecated('package:test_core is not intended for general use. '
     'Please use package:test.')
-library test_core.backend;
+library;
 
 export 'package:test_api/backend.dart'
     show Compiler, Metadata, PlatformSelector, Runtime, SuitePlatform;

diff --git a/pkgs/test_core/lib/src/runner/configuration.dart b/pkgs/test_core/lib/src/runner/configuration.dart
@@ -791,10 +791,10 @@ class Configuration {
           }
           return config;
         }()) {
-    if (_filename != null && _filename!.context.style != p.style) {
+    if (_filename != null && _filename.context.style != p.style) {
       throw ArgumentError(
           "filename's context must match the current operating system, was "
-          '${_filename!.context.style}.');
+          '${_filename.context.style}.');
     }
 
     if ((shardIndex == null) != (totalShards == null)) {
@@ -897,15 +897,15 @@ class Configuration {
     var foldTraceExcept = other._foldTraceExcept ?? _foldTraceExcept;
     if (_foldTraceOnly != null) {
       if (other._foldTraceExcept != null) {
-        foldTraceOnly = _foldTraceOnly!.difference(other._foldTraceExcept!);
+        foldTraceOnly = _foldTraceOnly.difference(other._foldTraceExcept);
       } else if (other._foldTraceOnly != null) {
-        foldTraceOnly = other._foldTraceOnly!.intersection(_foldTraceOnly!);
+        foldTraceOnly = other._foldTraceOnly.intersection(_foldTraceOnly);
       }
     } else if (_foldTraceExcept != null) {
       if (other._foldTraceOnly != null) {
-        foldTraceOnly = other._foldTraceOnly!.difference(_foldTraceExcept!);
+        foldTraceOnly = other._foldTraceOnly.difference(_foldTraceExcept);
       } else if (other._foldTraceExcept != null) {
-        foldTraceExcept = other._foldTraceExcept!.union(_foldTraceExcept!);
+        foldTraceExcept = other._foldTraceExcept.union(_foldTraceExcept);
       }
     }
 

diff --git a/pkgs/test_core/lib/src/runner/suite.dart b/pkgs/test_core/lib/src/runner/suite.dart
@@ -110,7 +110,7 @@ final class SuiteConfiguration {
   /// The set of runtimes on which to run tests.
   List<String> get runtimes => _runtimes == null
       ? const ['vm']
-      : List.unmodifiable(_runtimes!.map((runtime) => runtime.name));
+      : List.unmodifiable(_runtimes.map((runtime) => runtime.name));
   final List<RuntimeSelection>? _runtimes;
 
   /// Configuration for particular tags.

diff --git a/pkgs/test_core/lib/test_core.dart b/pkgs/test_core/lib/test_core.dart
@@ -4,7 +4,7 @@
 
 @Deprecated('package:test_core is not intended for general use. '
     'Please use package:test.')
-library test_core;
+library;
 
 export 'package:test_api/hooks.dart' show TestFailure;
 

diff --git a/pkgs/test_core/pubspec.yaml b/pkgs/test_core/pubspec.yaml
@@ -1,10 +1,10 @@
 name: test_core
-version: 0.6.2
+version: 0.6.3-wip
 description: A basic library for writing tests and running them on the VM.
 repository: https://github.com/dart-lang/test/tree/master/pkgs/test_core
 
 environment:
-  sdk: ^3.0.0
+  sdk: ^3.2.0
 
 dependencies:
   analyzer: '>=3.3.0 <7.0.0'
@@ -26,9 +26,9 @@ dependencies:
   stack_trace: ^1.10.0
   stream_channel: ^2.1.0
   # Use an exact version until the test_api package is stable.
-  test_api: 0.7.1
+  test_api: 0.7.2
   vm_service: ">=6.0.0 <15.0.0"
   yaml: ^3.0.0
 
 dev_dependencies:
-  dart_flutter_team_lints: ^2.1.1
+  dart_flutter_team_lints: ^3.0.0
diff --git a/tool/ci.sh b/tool/ci.sh