chore: remove unused imports and apply minor test cleanup adjustments

pavanpodila · pavanpodila · commit 1e93f98b7691 · 2026-01-06T12:00:54.000+05:30
diff --git a/packages/vyuh_node_flow/test/unit/connections/connection_labels_validation_test.dart b/packages/vyuh_node_flow/test/unit/connections/connection_labels_validation_test.dart
@@ -12,14 +12,11 @@
 @Tags(['unit'])
 library;
 
-import 'dart:ui';
-
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 
 import '../../helpers/test_factories.dart';
-import '../../helpers/test_utils.dart';
 
 void main() {
   setUp(() {
diff --git a/packages/vyuh_node_flow/test/unit/connections/connection_rules_test.dart b/packages/vyuh_node_flow/test/unit/connections/connection_rules_test.dart
@@ -16,7 +16,6 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 
 import '../../helpers/test_factories.dart';
-import '../../helpers/test_utils.dart';
 
 void main() {
   late NodeFlowController<String, dynamic> controller;
diff --git a/packages/vyuh_node_flow/test/unit/connections/label_calculator_test.dart b/packages/vyuh_node_flow/test/unit/connections/label_calculator_test.dart
@@ -11,16 +11,12 @@
 @Tags(['unit'])
 library;
 
-import 'dart:ui';
-
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 import 'package:vyuh_node_flow/src/connections/styles/label_calculator.dart';
-import 'package:vyuh_node_flow/src/connections/styles/endpoint_position_calculator.dart';
 
 import '../../helpers/test_factories.dart';
-import '../../helpers/test_utils.dart';
 
 void main() {
   setUp(() {
diff --git a/packages/vyuh_node_flow/test/unit/connections/temporary_connection_test.dart b/packages/vyuh_node_flow/test/unit/connections/temporary_connection_test.dart
@@ -13,7 +13,6 @@ import 'dart:ui';
 
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/src/connections/temporary_connection.dart';
-import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 
 void main() {
   // ===========================================================================
diff --git a/packages/vyuh_node_flow/test/unit/connections/waypoint_builder_test.dart b/packages/vyuh_node_flow/test/unit/connections/waypoint_builder_test.dart
@@ -19,8 +19,6 @@ import 'dart:ui';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 import 'package:vyuh_node_flow/src/connections/styles/waypoint_builder.dart';
-import 'package:vyuh_node_flow/src/connections/styles/connection_style_base.dart';
-import 'package:vyuh_node_flow/src/connections/styles/path_segments.dart';
 
 import '../../helpers/test_factories.dart';
 
diff --git a/packages/vyuh_node_flow/test/unit/controller/controller_extensions_test.dart b/packages/vyuh_node_flow/test/unit/controller/controller_extensions_test.dart
@@ -23,8 +23,6 @@
 @Tags(['unit'])
 library;
 
-import 'dart:ui';
-
 import 'package:flutter/services.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/src/connections/temporary_connection.dart';
diff --git a/packages/vyuh_node_flow/test/unit/controller/dirty_tracking_test.dart b/packages/vyuh_node_flow/test/unit/controller/dirty_tracking_test.dart
@@ -10,7 +10,6 @@
 @Tags(['unit'])
 library;
 
-import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 
diff --git a/packages/vyuh_node_flow/test/unit/editor/auto_pan_mixin_test.dart b/packages/vyuh_node_flow/test/unit/editor/auto_pan_mixin_test.dart
@@ -13,7 +13,6 @@ library;
 
 import 'dart:async';
 
-import 'package:flutter/gestures.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
diff --git a/packages/vyuh_node_flow/test/unit/editor/hit_testing_test.dart b/packages/vyuh_node_flow/test/unit/editor/hit_testing_test.dart
@@ -17,7 +17,6 @@ import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 import 'package:vyuh_node_flow/src/shared/spatial/graph_spatial_index.dart';
-import 'package:vyuh_node_flow/src/shared/spatial/spatial_item.dart';
 
 import '../../helpers/test_factories.dart';
 
diff --git a/packages/vyuh_node_flow/test/unit/editor/interaction_layer_test.dart b/packages/vyuh_node_flow/test/unit/editor/interaction_layer_test.dart
@@ -1216,7 +1216,7 @@ void main() {
               child: NodeFlowEditor<String, dynamic>(
                 controller: controller,
                 nodeBuilder: (context, node) =>
-                    Container(width: 100, height: 60),
+                    SizedBox(width: 100, height: 60),
                 theme: NodeFlowTheme.light,
               ),
             ),
@@ -1251,7 +1251,7 @@ void main() {
               child: NodeFlowEditor<String, dynamic>(
                 controller: controller,
                 nodeBuilder: (context, node) =>
-                    Container(width: 100, height: 60),
+                    SizedBox(width: 100, height: 60),
                 theme: NodeFlowTheme.light,
               ),
             ),
@@ -1282,7 +1282,7 @@ void main() {
               child: NodeFlowEditor<String, dynamic>(
                 controller: controller,
                 nodeBuilder: (context, node) =>
-                    Container(width: 100, height: 60),
+                    SizedBox(width: 100, height: 60),
                 theme: NodeFlowTheme.light,
               ),
             ),
diff --git a/packages/vyuh_node_flow/test/unit/editor/node_flow_config_test.dart b/packages/vyuh_node_flow/test/unit/editor/node_flow_config_test.dart
@@ -14,8 +14,6 @@
 @Tags(['unit'])
 library;
 
-import 'dart:ui';
-
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mobx/mobx.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
diff --git a/packages/vyuh_node_flow/test/unit/effects/connection_effects_test.dart b/packages/vyuh_node_flow/test/unit/effects/connection_effects_test.dart
@@ -3,8 +3,6 @@
 /// Tests all connection effects through the public API.
 library;
 
-import 'dart:ui';
-
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
diff --git a/packages/vyuh_node_flow/test/unit/events/extension_events_test.dart b/packages/vyuh_node_flow/test/unit/events/extension_events_test.dart
@@ -19,7 +19,6 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 
 import '../../helpers/test_factories.dart';
-import '../../helpers/test_utils.dart';
 
 void main() {
   setUp(() {
diff --git a/packages/vyuh_node_flow/test/unit/events/node_flow_events_test.dart b/packages/vyuh_node_flow/test/unit/events/node_flow_events_test.dart
@@ -11,8 +11,6 @@
 @Tags(['unit'])
 library;
 
-import 'dart:ui';
-
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 
diff --git a/packages/vyuh_node_flow/test/unit/gestures/hit_testing_test.dart b/packages/vyuh_node_flow/test/unit/gestures/hit_testing_test.dart
@@ -19,7 +19,6 @@ import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 import 'package:vyuh_node_flow/src/shared/spatial/graph_spatial_index.dart';
-import 'package:vyuh_node_flow/src/shared/spatial/spatial_item.dart';
 
 import '../../helpers/test_factories.dart';
 
diff --git a/packages/vyuh_node_flow/test/unit/graph/graph_test.dart b/packages/vyuh_node_flow/test/unit/graph/graph_test.dart
@@ -10,8 +10,6 @@
 @Tags(['unit'])
 library;
 
-import 'dart:convert';
-
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
diff --git a/packages/vyuh_node_flow/test/unit/nodes/comment_node_test.dart b/packages/vyuh_node_flow/test/unit/nodes/comment_node_test.dart
@@ -19,7 +19,6 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:mobx/mobx.dart';
 import 'package:vyuh_node_flow/vyuh_node_flow.dart';
 import 'package:vyuh_node_flow/src/editor/resizer_widget.dart';
-import 'package:vyuh_node_flow/src/nodes/mixins/resizable_mixin.dart';
 
 import '../../helpers/test_factories.dart';
 import '../../helpers/test_utils.dart';
diff --git a/packages/vyuh_node_flow/test/unit/nodes/group_node_test.dart b/packages/vyuh_node_flow/test/unit/nodes/group_node_test.dart
@@ -448,7 +448,7 @@ void main() {
 
         expect(ids, containsAll(['node-1', 'node-2']));
         // The returned set should be unmodifiable
-        expect(() => (ids as Set<String>).add('new'), throwsUnsupportedError);
+        expect(() => (ids).add('new'), throwsUnsupportedError);
       });
     });
   });
@@ -470,7 +470,7 @@ void main() {
       final originalSize = group.size.value;
 
       // Create a mock lookup
-      NodeLookup lookup = (id) => createTestNode(
+      Node<dynamic>? lookup(String id) => createTestNode(
         id: id,
         position: const Offset(100, 100),
         size: const Size(150, 100),
@@ -495,7 +495,7 @@ void main() {
       final originalPosition = group.position.value;
       final originalSize = group.size.value;
 
-      NodeLookup lookup = (id) => createTestNode(
+      Node<dynamic>? lookup(String id) => createTestNode(
         id: id,
         position: const Offset(100, 100),
         size: const Size(150, 100),
@@ -519,7 +519,7 @@ void main() {
       final originalPosition = group.position.value;
       final originalSize = group.size.value;
 
-      NodeLookup lookup = (_) => null;
+      Node<dynamic>? lookup(String _) => null;
 
       group.fitToNodes(lookup);
 
@@ -539,7 +539,7 @@ void main() {
         padding: const EdgeInsets.all(20),
       );
 
-      NodeLookup lookup = (id) {
+      Node<dynamic>? lookup(String id) {
         if (id == 'node-1') {
           return createTestNode(
             id: 'node-1',
@@ -548,7 +548,7 @@ void main() {
           );
         }
         return null;
-      };
+      }
 
       group.fitToNodes(lookup);
 
@@ -573,7 +573,7 @@ void main() {
         padding: const EdgeInsets.all(10),
       );
 
-      NodeLookup lookup = (id) {
+      Node<dynamic>? lookup(String id) {
         if (id == 'node-1') {
           return createTestNode(
             id: 'node-1',
@@ -589,7 +589,7 @@ void main() {
           );
         }
         return null;
-      };
+      }
 
       group.fitToNodes(lookup);
 
@@ -615,7 +615,7 @@ void main() {
         // Uses kGroupNodeDefaultPadding (20, 40, 20, 20)
       );
 
-      NodeLookup lookup = (id) {
+      Node<dynamic>? lookup(String id) {
         if (id == 'node-1') {
           return createTestNode(
             id: 'node-1',
@@ -624,7 +624,7 @@ void main() {
           );
         }
         return null;
-      };
+      }
 
       group.fitToNodes(lookup);
 
@@ -648,7 +648,7 @@ void main() {
         padding: const EdgeInsets.all(10),
       );
 
-      NodeLookup lookup = (id) {
+      Node<dynamic>? lookup(String id) {
         if (id == 'node-1') {
           return createTestNode(
             id: 'node-1',
@@ -664,7 +664,7 @@ void main() {
           );
         }
         return null; // node-missing returns null
-      };
+      }
 
       group.fitToNodes(lookup);
 
@@ -690,7 +690,7 @@ void main() {
       final originalPosition = group.position.value;
       final originalSize = group.size.value;
 
-      NodeLookup lookup = (_) => null;
+      Node<dynamic>? lookup(String _) => null;
 
       group.fitToNodes(lookup);
 
@@ -710,7 +710,7 @@ void main() {
         padding: const EdgeInsets.all(10),
       );
 
-      NodeLookup lookup = (id) {
+      Node<dynamic>? lookup(String id) {
         if (id == 'node-1') {
           return createTestNode(
             id: 'node-1',
@@ -719,7 +719,7 @@ void main() {
           );
         }
         return null;
-      };
+      }
 
       group.fitToNodes(lookup);
 
@@ -1476,7 +1476,7 @@ void main() {
         padding: const EdgeInsets.all(10),
       );
 
-      NodeLookup lookup = (id) {
+      Node<dynamic>? lookup(String id) {
         if (id == 'node-1') {
           return createTestNode(
             id: 'node-1',
@@ -1488,7 +1488,7 @@ void main() {
           );
         }
         return null;
-      };
+      }
 
       group.setBehavior(
         GroupBehavior.explicit,
diff --git a/packages/vyuh_node_flow/test/unit/nodes/interaction_state_test.dart b/packages/vyuh_node_flow/test/unit/nodes/interaction_state_test.dart
@@ -14,7 +14,6 @@ library;
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_test/flutter_test.dart';
-import 'package:mobx/mobx.dart';
 import 'package:vyuh_node_flow/src/connections/temporary_connection.dart';
 import 'package:vyuh_node_flow/src/editor/resizer_widget.dart';
 import 'package:vyuh_node_flow/src/graph/coordinates.dart';
diff --git a/packages/vyuh_node_flow/test/unit/nodes/node_container_test.dart b/packages/vyuh_node_flow/test/unit/nodes/node_container_test.dart
diff --git a/packages/vyuh_node_flow/test/unit/nodes/resizable_mixin_test.dart b/packages/vyuh_node_flow/test/unit/nodes/resizable_mixin_test.dart
diff --git a/packages/vyuh_node_flow/test/unit/nodes/special_nodes_test.dart b/packages/vyuh_node_flow/test/unit/nodes/special_nodes_test.dart
diff --git a/packages/vyuh_node_flow/test/unit/ports/port_widget_test.dart b/packages/vyuh_node_flow/test/unit/ports/port_widget_test.dart
diff --git a/packages/vyuh_node_flow/test/unit/spatial/graph_spatial_index_test.dart b/packages/vyuh_node_flow/test/unit/spatial/graph_spatial_index_test.dart
diff --git a/packages/vyuh_node_flow/test/unit/themes/all_themes_test.dart b/packages/vyuh_node_flow/test/unit/themes/all_themes_test.dart
diff --git a/packages/vyuh_node_flow/test/unit/themes/node_flow_theme_test.dart b/packages/vyuh_node_flow/test/unit/themes/node_flow_theme_test.dart
