fix: replace setEvents with internalUpdateEvents across tests for consistency

fix: remove redundant theme initialization in NodeFlowController

Author: pavanpodila

packages/demo/lib/examples/advanced/workbench.dart

@@ -50,7 +50,7 @@ class _WorkbenchExampleState extends State<WorkbenchExample> {
     _controller = NodeFlowController<Map<String, dynamic>>(
       config: _nodeFlowConfig,
     );
-    _controller.setTheme(_nodeFlowTheme); // Set initial theme in controller
+    // NOTE: Theme is handled by NodeFlowEditor - no need to set it on the controller
     _loadInitialWorkflow();
   }
 

packages/vyuh_node_flow/test/behavior/connections/connection_creation_test.dart

@@ -56,7 +56,7 @@ void main() {
       controller.addNode(node2);
 
       Connection? createdConnection;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents(onCreated: (c) => createdConnection = c),
         ),
@@ -473,7 +473,7 @@ void main() {
       Node<String>? startNode;
       Port? startPort;
 
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents(
             onConnectStart: (node, port) {
@@ -589,7 +589,7 @@ void main() {
     test('completeConnectionDrag fires onConnectEnd with target', () {
       Node<String>? endTargetNode;
       Port? endTargetPort;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents(
             onConnectEnd: (node, port) {
@@ -682,7 +682,7 @@ void main() {
       bool callbackFired = false;
       Node<String>? endTargetNode;
       Port? endTargetPort;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents(
             onConnectEnd: (node, port) {
@@ -848,7 +848,7 @@ void main() {
       );
       controller.addNode(node);
 
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents<String>(
             onBeforeStart: (context) {
@@ -882,7 +882,7 @@ void main() {
       controller.addNode(source);
       controller.addNode(target);
 
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents<String>(
             onBeforeComplete: (context) {
@@ -926,7 +926,7 @@ void main() {
       controller.addNode(target);
 
       ConnectionCompleteContext<String>? capturedContext;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents<String>(
             onBeforeComplete: (context) {

packages/vyuh_node_flow/test/behavior/dragging/node_drag_test.dart

@@ -79,7 +79,7 @@ void main() {
       controller.addNode(node);
 
       Node<String>? draggedNode;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onDragStart: (n) {
@@ -148,7 +148,7 @@ void main() {
       controller.addNode(node);
 
       Node<String>? movedNode;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onDrag: (n) {
@@ -215,7 +215,7 @@ void main() {
       controller.addNode(node2);
 
       final movedNodeIds = <String>[];
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onDrag: (n) {
@@ -305,7 +305,7 @@ void main() {
       controller.addNode(node);
 
       Node<String>? stoppedNode;
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onDragStop: (n) {
@@ -328,7 +328,7 @@ void main() {
       controller.addNode(node2);
 
       final stoppedNodeIds = <String>[];
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onDragStop: (n) {
@@ -402,7 +402,7 @@ void main() {
       controller.addNode(node);
 
       final events = <String>[];
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onDragStart: (n) => events.add('start'),

packages/vyuh_node_flow/test/behavior/selection/connection_selection_test.dart

@@ -474,7 +474,7 @@ void main() {
       controller.addNode(node2);
       controller.addConnection(conn);
 
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents<String>(
             onSelected: (c) {
@@ -504,7 +504,7 @@ void main() {
       controller.addNode(node2);
       controller.addConnection(conn);
 
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           connection: ConnectionEvents<String>(
             onSelected: (c) {

packages/vyuh_node_flow/test/behavior/selection/node_selection_test.dart

@@ -414,7 +414,7 @@ void main() {
       String? selectedNodeId;
       final node = createTestNode(id: 'node1');
       controller.addNode(node);
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onSelected: (n) {
@@ -433,7 +433,7 @@ void main() {
       String? lastSelectedNodeId = 'initial';
       final node = createTestNode(id: 'node1');
       controller.addNode(node);
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onSelected: (n) {

packages/vyuh_node_flow/test/edge_cases/concurrent_operations_test.dart

@@ -459,7 +459,7 @@ void main() {
 
       controller = createTestController();
       // Create events directly to avoid type inference issues with copyWith
-      controller.setEvents(
+      controller.internalUpdateEvents(
         NodeFlowEvents<String>(
           node: NodeEvents<String>(
             onCreated: (node) => nodeCreatedCount++,