fix: Enhance Makefile testing commands and improve Unity controller error handling

This commit updates the Makefile to skip testing for packages without test directories and modifies the static analysis and test commands to use `--no-fatal-infos` for better error handling. Additionally, it refines the Unity controller implementation by replacing `print` statements with `debugPrint` for improved logging consistency. A new method for setting the streaming cache path is added to the Unreal controller, and minor adjustments are made to various test files for consistency and clarity.

Author: juicycleff

Makefile

@@ -44,8 +44,12 @@ setup: bootstrap ## Alias for bootstrap (install all dependencies)
 test: ## Run all tests across workspace packages
 	@echo "$(BLUE)Running tests across workspace...$(NC)"
 	@for pkg in $(PACKAGES) $(EXAMPLE); do \
-		echo "$(BLUE)Testing $$pkg...$(NC)"; \
-		cd $$pkg && flutter test && cd - > /dev/null || exit 1; \
+		if [ -d "$$pkg/test" ]; then \
+			echo "$(BLUE)Testing $$pkg...$(NC)"; \
+			cd $$pkg && flutter test && cd - > /dev/null || exit 1; \
+		else \
+			echo "$(YELLOW)Skipping $$pkg (no tests)$(NC)"; \
+		fi; \
 	done
 	@echo "$(GREEN)✓ All tests passed!$(NC)"
 
@@ -66,7 +70,7 @@ analyze: ## Run static analysis across workspace
 	@echo "$(BLUE)Running static analysis across workspace...$(NC)"
 	@for pkg in $(PACKAGES) $(EXAMPLE); do \
 		echo "$(BLUE)Analyzing $$pkg...$(NC)"; \
-		cd $$pkg && flutter analyze && cd - > /dev/null || exit 1; \
+		cd $$pkg && flutter analyze --no-fatal-infos && cd - > /dev/null || exit 1; \
 	done
 	@echo "$(GREEN)✓ No issues found!$(NC)"
 
@@ -86,12 +90,16 @@ lint: ## Run all linting checks (format, analyze, test) across workspace
 	@dart format --set-exit-if-changed . > /dev/null 2>&1 && echo "$(GREEN)  ✓ Format check passed$(NC)" || (echo "$(RED)  ✗ Format check failed$(NC)" && exit 1)
 	@echo "2. Running static analysis..."
 	@for pkg in $(PACKAGES) $(EXAMPLE); do \
-		cd $$pkg && flutter analyze > /dev/null 2>&1 && cd - > /dev/null || (echo "$(RED)  ✗ Analysis failed in $$pkg$(NC)" && exit 1); \
+		(cd $$pkg && flutter analyze --no-fatal-infos > /dev/null 2>&1) || (echo "$(RED)  ✗ Analysis failed in $$pkg$(NC)" && exit 1); \
 	done
 	@echo "$(GREEN)  ✓ Analysis passed$(NC)"
 	@echo "3. Running tests..."
 	@for pkg in $(PACKAGES) $(EXAMPLE); do \
-		cd $$pkg && flutter test > /dev/null 2>&1 && cd - > /dev/null || (echo "$(RED)  ✗ Tests failed in $$pkg$(NC)" && exit 1); \
+		if [ -d "$$pkg/test" ]; then \
+			(cd $$pkg && flutter test > /dev/null 2>&1) || (echo "$(RED)  ✗ Tests failed in $$pkg$(NC)" && exit 1); \
+		else \
+			echo "$(YELLOW)  ⊘ Skipping $$pkg (no tests)$(NC)"; \
+		fi; \
 	done
 	@echo "$(GREEN)  ✓ Tests passed$(NC)"
 	@echo ""

engines/unity/dart/lib/src/unity_controller.dart

@@ -1,4 +1,5 @@
 import 'dart:async';
+import 'package:flutter/foundation.dart';
 import 'package:flutter/services.dart';
 import 'package:gameframework/gameframework.dart';
 
@@ -150,11 +151,11 @@ class UnityController implements GameEngineController {
         if (attempt == 0) {
           // First attempt is expected to fail, don't log
         } else if (attempt < 3) {
-          print(
+          debugPrint(
               'Platform view not ready, retrying in ${delayMs}ms (attempt ${attempt + 1}/$maxAttempts)');
         } else {
-          print(
-              'Warning: Platform view still not ready after ${attempt} attempts, retrying in ${delayMs}ms');
+          debugPrint(
+              'Warning: Platform view still not ready after $attempt attempts, retrying in ${delayMs}ms');
         }
 
         await Future.delayed(Duration(milliseconds: delayMs));
@@ -271,11 +272,11 @@ class UnityController implements GameEngineController {
         if (attempt == 0) {
           // First attempt failure is expected, don't log
         } else if (attempt < 3) {
-          print(
+          debugPrint(
               'Platform view not ready for create(), retrying in ${delayMs}ms (attempt ${attempt + 1}/$maxAttempts)');
         } else {
-          print(
-              'Warning: Platform view still not ready for create() after ${attempt} attempts, retrying in ${delayMs}ms');
+          debugPrint(
+              'Warning: Platform view still not ready for create() after $attempt attempts, retrying in ${delayMs}ms');
         }
 
         await Future.delayed(Duration(milliseconds: delayMs));

engines/unity/dart/lib/src/unity_controller_web.dart

@@ -1,6 +1,9 @@
 import 'dart:async';
+// ignore: deprecated_member_use, avoid_web_libraries_in_flutter
 import 'dart:html' as html;
+// ignore: deprecated_member_use, avoid_web_libraries_in_flutter
 import 'dart:js' as js;
+import 'package:flutter/foundation.dart';
 import 'package:gameframework/gameframework.dart';
 
 /// Unity WebGL-specific implementation of GameEngineController
@@ -288,6 +291,14 @@ class UnityControllerWeb implements GameEngineController {
     dispose();
   }
 
+  @override
+  Future<void> setStreamingCachePath(String path) async {
+    // WebGL doesn't support local file system cache paths
+    // Streaming assets are loaded via HTTP from the web server
+    // This is a no-op for web platform
+    return;
+  }
+
   @override
   void dispose() {
     if (_disposed) return;
@@ -368,7 +379,7 @@ class UnityControllerWeb implements GameEngineController {
       _unityInstance!.callMethod('SetFullscreen', [fullscreen ? 1 : 0]);
     } catch (e) {
       // Fullscreen might not be supported
-      print('Failed to set fullscreen: $e');
+      debugPrint('Failed to set fullscreen: $e');
     }
   }
 }

engines/unity/dart/lib/src/unity_engine_plugin.dart

@@ -1,5 +1,4 @@
 import 'package:gameframework/gameframework.dart';
-import 'unity_controller.dart';
 import 'unity_controller_web.dart' if (dart.library.io) 'unity_controller.dart'
     as platform;
 

engines/unity/dart/test/unity_controller_test.dart

@@ -246,7 +246,7 @@ void main() {
 
   group('createUnityController', () {
     test('should create UnityController instance', () {
-      final config = const GameEngineConfig();
+      const config = GameEngineConfig();
       final controller = createUnityController(1, config);
 
       expect(controller, isA<UnityController>());

engines/unreal/dart/lib/src/unreal_controller.dart

@@ -428,6 +428,24 @@ class UnrealController implements GameEngineController {
     }
   }
 
+  @override
+  Future<void> setStreamingCachePath(String path) async {
+    _throwIfDisposed();
+
+    try {
+      await _channel.invokeMethod('engine#setStreamingCachePath', {
+        'path': path,
+      });
+    } catch (e) {
+      throw EngineCommunicationException(
+        'Failed to set streaming cache path: $e',
+        target: 'UnrealController',
+        method: 'setStreamingCachePath',
+        engineType: engineType,
+      );
+    }
+  }
+
   @override
   Future<void> dispose() async {
     if (_isDisposed) return;

example/integration_test/unity_embedding_test.dart

@@ -128,15 +128,12 @@ void main() {
     testWidgets('Unity handles dispose during initialization gracefully',
         (WidgetTester tester) async {
       // This test verifies race condition fixes
-      var initStarted = false;
-
       await tester.pumpWidget(
         MaterialApp(
           home: Scaffold(
             body: GameWidget(
               engineType: GameEngineType.unity,
               onEngineCreated: (controller) {
-                initStarted = true;
                 debugPrint('Unity initialization started');
               },
             ),

example/lib/main.dart

@@ -206,7 +206,7 @@ class _UnityExampleScreenState extends State<UnityExampleScreen> {
               color: Colors.white,
               boxShadow: [
                 BoxShadow(
-                  color: Colors.black.withOpacity(0.1),
+                  color: Colors.black.withValues(alpha: 0.1),
                   blurRadius: 4,
                   offset: const Offset(0, -2),
                 ),

example/lib/platform_view_modes_example.dart

@@ -1,6 +1,5 @@
 import 'package:flutter/material.dart';
 import 'package:gameframework/gameframework.dart';
-import 'package:gameframework_unity/gameframework_unity.dart';
 
 /// Example demonstrating Android platform view mode switching
 ///
@@ -254,7 +253,7 @@ class PlatformViewModeInfoCard extends StatelessWidget {
               padding: const EdgeInsets.only(left: 32, top: 4),
               child: Row(
                 children: [
-                  Icon(Icons.check, size: 16, color: Colors.green),
+                  const Icon(Icons.check, size: 16, color: Colors.green),
                   const SizedBox(width: 8),
                   Expanded(child: Text(feature)),
                 ],

example/test/widget_test.dart

@@ -11,16 +11,31 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:gameframework_example/main.dart';
 
 void main() {
-  testWidgets('Verify Platform version', (WidgetTester tester) async {
+  testWidgets('App loads and displays title', (WidgetTester tester) async {
     // Build our app and trigger a frame.
     await tester.pumpWidget(const MyApp());
 
-    // Verify that platform version is retrieved.
+    // Verify that the app title is displayed in AppBar.
     expect(
-      find.byWidgetPredicate(
-        (Widget widget) =>
-            widget is Text && widget.data!.startsWith('Running on:'),
-      ),
+      find.widgetWithText(AppBar, 'Flutter Game Framework'),
+      findsOneWidget,
+    );
+
+    // Verify that the games icon is present.
+    expect(
+      find.byIcon(Icons.games),
+      findsOneWidget,
+    );
+  });
+
+  testWidgets('App displays Unity example button', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(const MyApp());
+    await tester.pumpAndSettle();
+
+    // Verify that Unity Example button is displayed.
+    expect(
+      find.text('Unity Example'),
       findsOneWidget,
     );
   });