chore: fix lints

.github/actions/flutter_package/action.yaml

@@ -52,7 +52,6 @@ runs:
       shell: ${{ inputs.shell }}
       run: |
         flutter pub global activate very_good_cli
-        very_good --analytics false
         very_good packages get --recursive
 
     - working-directory: ${{ inputs.working_directory }}

example/actions/expect_environment_text/lib/src/expect_environment_text.dart

@@ -16,8 +16,8 @@ class ExpectEnvironmentText extends Action {
   /// Called when it executes the action in a flow file.
   @override
   Future<bool> execute(Tester tester) async {
-    if (!await ExpectVisible(
-      text: "Environment: (Development|Staging|Production|None){1}",
+    if (!await const ExpectVisible(
+      text: 'Environment: (Development|Staging|Production|None){1}',
     ).execute(tester)) {
       return false;
     }

example/actions/expect_environment_text/pubspec.yaml

@@ -6,6 +6,6 @@ environment:
   sdk: ">=3.0.0 <4.0.0"
 
 dependencies:
-  fluttium: ^0.1.0-dev.1
   flutter:
     sdk: flutter
+  fluttium: ^0.1.0-dev.1

example/lib/simple_menu/view/simple_menu.dart

@@ -36,7 +36,7 @@ class _SimpleMenuPageState extends State<SimpleMenuPage> {
                   child: Row(
                     children: [Text('Menu Item 1')],
                   ),
-                )
+                ),
               ],
               context: context,
               position: _getRelativeRect(widgetKey),

packages/fluttium/lib/src/actions/clear_text.dart

@@ -58,7 +58,7 @@ class ClearText extends Action {
         SystemChannels.textInput.codec.encodeMethodCall(
           const MethodCall('TextInputClient.performSelectors', [
             -1,
-            ['deleteBackward:']
+            ['deleteBackward:'],
           ]),
         ),
       );

packages/fluttium/lib/src/actions/swipe.dart

@@ -15,7 +15,7 @@ import 'package:fluttium/src/actions/scroll.dart';
 /// ```
 /// {@endtemplate}
 class Swipe extends Scroll {
-  /// {@template swipe}
+  /// {@macro swipe}
   Swipe({
     required super.within,
     required super.until,

packages/fluttium/lib/src/registry.dart

@@ -38,7 +38,7 @@ class Registry {
         timeout: timeout,
       ),
       aliases: const [
-        Alias(['in'], #within)
+        Alias(['in'], #within),
       ],
     ),
     'swipe': ActionRegistration(
@@ -57,7 +57,7 @@ class Registry {
         timeout: timeout,
       ),
       aliases: const [
-        Alias(['in'], #within)
+        Alias(['in'], #within),
       ],
     ),
   };

packages/fluttium/lib/src/tester.dart

@@ -23,7 +23,8 @@ class Tester {
 
   final Registry _registry;
 
-  SemanticsOwner get _semanticsOwner => _binding.pipelineOwner.semanticsOwner!;
+  SemanticsOwner? get _semanticsOwner =>
+      _binding.rootPipelineOwner.semanticsOwner;
 
   /// The current screen's media query information.
   MediaQueryData get mediaQuery =>
@@ -103,23 +104,23 @@ class Tester {
   ///
   /// The [text] can be a [String] that can also be used as a [RegExp].
   Future<SemanticsNode?> find(String text, {Duration? timeout}) async {
-    var nodes = _findNodes(_semanticsOwner.rootSemanticsNode!, text);
+    var nodes = _findNodes(_semanticsOwner!.rootSemanticsNode, text);
 
     final end = clock.now().add(timeout ?? const Duration(seconds: 10));
     while (nodes.isEmpty) {
       await pump();
       if (clock.now().isAfter(end)) {
         return null;
       }
-      nodes = _findNodes(_semanticsOwner.rootSemanticsNode!, text);
+      nodes = _findNodes(_semanticsOwner!.rootSemanticsNode, text);
     }
 
     return nodes.first;
   }
 
-  List<SemanticsNode> _findNodes(SemanticsNode node, String text) {
+  List<SemanticsNode> _findNodes(SemanticsNode? node, String text) {
     final nodes = <SemanticsNode>[];
-    node.visitChildren((n) {
+    node?.visitChildren((n) {
       // Add all descendants that match the pattern.
       if (!n.mergeAllDescendantsIntoThisNode) {
         nodes.addAll(_findNodes(n, text));
@@ -169,8 +170,7 @@ class Tester {
 
   /// Wait for the semantics tree to be fully build.
   Future<void> ready() async {
-    while (_binding.pipelineOwner.semanticsOwner == null ||
-        _binding.pipelineOwner.semanticsOwner!.rootSemanticsNode == null) {
+    while (_semanticsOwner!.rootSemanticsNode == null) {
       await _binding.endOfFrame;
     }
   }

packages/fluttium/test/helpers/matchers/is_delete_backwards.dart

@@ -6,7 +6,7 @@ Matcher get isDeleteBackward {
     'method': 'TextInputClient.performSelectors',
     'args': [
       -1,
-      ['deleteBackward:']
+      ['deleteBackward:'],
     ],
   });
 

packages/fluttium/test/helpers/matchers/is_text.dart

@@ -9,7 +9,7 @@ Matcher isText(String text) {
       TextEditingValue(
         text: text,
         selection: TextSelection.collapsed(offset: text.length),
-      ).toJSON()
+      ).toJSON(),
     ],
   });
 

packages/fluttium/test/src/registry_test.dart

@@ -65,15 +65,15 @@ void main() {
           'action',
           _TestActionWithArguments.new,
           aliases: [
-            Alias(['withKey'], #key)
+            Alias(['withKey'], #key),
           ],
         );
 
         expect(registry.actions.containsKey('action'), isTrue);
         expect(
           registry.actions['action']!.aliases,
           equals([
-            Alias(['withKey'], #key)
+            Alias(['withKey'], #key),
           ]),
         );
       });
@@ -130,7 +130,7 @@ void main() {
           'action',
           _TestActionWithArguments.new,
           aliases: [
-            Alias(['withKey'], #key)
+            Alias(['withKey'], #key),
           ],
         );
 

packages/fluttium/test/src/tester_test.dart

@@ -26,7 +26,12 @@ class _MockRegistry extends Mock implements Registry {}
 
 class _MockChannelBuffers extends Mock implements ChannelBuffers {}
 
-class _MockPipelineOwner extends Mock implements PipelineOwner {}
+class _MockPipelineOwner extends Mock implements PipelineOwner {
+  @override
+  String toString({DiagnosticLevel minLevel = DiagnosticLevel.info}) {
+    return super.toString();
+  }
+}
 
 class _MockSemanticsOwner extends Mock implements SemanticsOwner {}
 
@@ -302,7 +307,7 @@ void main() {
         semanticsOwner = _MockSemanticsOwner();
 
         final pipelineOwner = _MockPipelineOwner();
-        when(() => binding.pipelineOwner).thenReturn(pipelineOwner);
+        when(() => binding.rootPipelineOwner).thenReturn(pipelineOwner);
         when(() => pipelineOwner.semanticsOwner).thenReturn(semanticsOwner);
 
         rootNode = MockSemanticsNode();
@@ -488,7 +493,7 @@ void main() {
         semanticsOwner = _MockSemanticsOwner();
 
         final pipelineOwner = _MockPipelineOwner();
-        when(() => binding.pipelineOwner).thenReturn(pipelineOwner);
+        when(() => binding.rootPipelineOwner).thenReturn(pipelineOwner);
         when(() => pipelineOwner.semanticsOwner).thenReturn(semanticsOwner);
 
         rootNode = MockSemanticsNode();
@@ -498,7 +503,8 @@ void main() {
       test('resolve when ready', () {
         expect(tester.ready(), completes);
 
-        verify(() => binding.pipelineOwner.semanticsOwner).called(equals(2));
+        verify(() => binding.rootPipelineOwner.semanticsOwner)
+            .called(equals(2));
         verify(() => semanticsOwner.rootSemanticsNode).called(equals(1));
       });
 
@@ -510,7 +516,8 @@ void main() {
 
         expect(tester.ready(), completes);
 
-        verify(() => binding.pipelineOwner.semanticsOwner).called(equals(2));
+        verify(() => binding.rootPipelineOwner.semanticsOwner)
+            .called(equals(2));
         verify(() => semanticsOwner.rootSemanticsNode).called(equals(1));
         verify(() => binding.endOfFrame).called(equals(1));
       });

packages/fluttium_cli/test/src/commands/test_command/test_command_test.dart

@@ -769,9 +769,9 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
             fileStep.copyWith(
               status: StepStatus.done,
               files: {
-                'file.txt': [1, 2, 3]
+                'file.txt': [1, 2, 3],
               },
-            )
+            ),
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -834,7 +834,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
         stepStateController.add([
           step1.copyWith(status: StepStatus.done),
           step2.copyWith(status: StepStatus.running),
-          step3
+          step3,
         ]);
         await Future<void>.delayed(Duration.zero);
 
@@ -845,7 +845,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
         stepStateController.add([
           step1.copyWith(status: StepStatus.done),
           step2.copyWith(status: StepStatus.failed),
-          step3
+          step3,
         ]);
         await Future<void>.delayed(Duration.zero);
 
@@ -911,7 +911,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.running),
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -923,7 +923,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.done),
-            step3.copyWith(status: StepStatus.done)
+            step3.copyWith(status: StepStatus.done),
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -970,7 +970,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.running),
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -982,7 +982,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.failed),
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -1087,7 +1087,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.running),
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -1132,7 +1132,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.running),
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -1143,7 +1143,7 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
           stepStateController.add([
             step1.copyWith(status: StepStatus.done),
             step2.copyWith(status: StepStatus.failed),
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 
@@ -1235,11 +1235,11 @@ Either adjust the constraint in the Fluttium configuration or update the CLI to
             step1.copyWith(
               status: StepStatus.done,
               files: {
-                'test_file': [1, 2, 3]
+                'test_file': [1, 2, 3],
               },
             ),
             step2,
-            step3
+            step3,
           ]);
           await Future<void>.delayed(Duration.zero);
 

packages/fluttium_cli/test/src/json_decode_safely_test.dart

@@ -66,7 +66,7 @@ Some output
           'b': {
             'd': 5,
             'e': 6,
-            'f': [7, 8, 9]
+            'f': [7, 8, 9],
           },
           'c': 4,
         }),