feat: minimal working interactive

Author: developer239

tests/sdl/SDLTestApp.cpp

@@ -30,11 +30,8 @@ class RobotTestApp {
             exit(1);
         }
 
-        // Create window
+        // Create window - IMPORTANT: Use SDL_WINDOW_SHOWN flag to ensure the window is visible
         Uint32 windowFlags = SDL_WINDOW_SHOWN;
-        if (headless) {
-            windowFlags |= SDL_WINDOW_HIDDEN;
-        }
 
         window = SDL_CreateWindow(
             "Robot CPP Testing Framework",
@@ -48,8 +45,8 @@ class RobotTestApp {
             exit(1);
         }
 
-        // Create renderer
-        renderer = SDL_CreateRenderer(window, -1, SDL_RENDERER_ACCELERATED);
+        // Create renderer with VSYNC to prevent rendering issues
+        renderer = SDL_CreateRenderer(window, -1, SDL_RENDERER_ACCELERATED | SDL_RENDERER_PRESENTVSYNC);
 
         if (!renderer) {
             std::cerr << "Could not create renderer: " << SDL_GetError() << std::endl;
@@ -60,6 +57,12 @@ class RobotTestApp {
         mouseTests = std::make_unique<MouseTests>(renderer, window);
         keyboardTests = std::make_unique<KeyboardTests>(renderer, window);
         screenTests = std::make_unique<ScreenTests>(renderer, window);
+
+        // Force the window to be on top
+        SDL_RaiseWindow(window);
+
+        // Position window consistently
+        SDL_SetWindowPosition(window, 50, 50);
     }
 
     ~RobotTestApp() {
@@ -83,18 +86,8 @@ class RobotTestApp {
 
         std::cout << "===== Robot CPP Test Suite =====" << std::endl;
 
-        // Make the window visible for tests even in headless mode
-        // This helps ensure the window is properly composited
-        if (headless) {
-            SDL_ShowWindow(window);
-        }
-
-        // Make sure the window is front and center
-        SDL_RaiseWindow(window);
-        SDL_SetWindowPosition(window, SDL_WINDOWPOS_CENTERED, SDL_WINDOWPOS_CENTERED);
-
-        // Wait a bit for window to be fully shown and composited
-        Robot::delay(1000);
+        // Make sure the window is properly initialized and visible
+        prepareForTests();
 
         // Run mouse tests
         std::cout << "\n----- Mouse Tests -----" << std::endl;
@@ -127,11 +120,6 @@ class RobotTestApp {
         std::cout << "\n===== Test Results =====" << std::endl;
         std::cout << (allTestsPassed ? "✅ ALL TESTS PASSED" : "❌ SOME TESTS FAILED") << std::endl;
 
-        // Hide window again if in headless mode
-        if (headless) {
-            SDL_HideWindow(window);
-        }
-
         return allTestsPassed;
     }
 
@@ -151,7 +139,7 @@ class RobotTestApp {
     }
 
     void render() {
-        // Clear screen
+        // Clear screen with a dark gray background
         SDL_SetRenderDrawColor(renderer, 40, 40, 40, 255);
         SDL_RenderClear(renderer);
 
@@ -165,10 +153,43 @@ class RobotTestApp {
         keyboardTests->draw();
         screenTests->draw();
 
-        // Present
+        // Present render to the screen
         SDL_RenderPresent(renderer);
     }
 
+    void prepareForTests() {
+        std::cout << "Preparing test environment..." << std::endl;
+
+        // Make sure window is visible
+        SDL_ShowWindow(window);
+
+        // Ensure window is positioned correctly
+        SDL_SetWindowPosition(window, 50, 50);
+
+        // Make sure the window is on top
+        SDL_RaiseWindow(window);
+
+        // Render several frames to ensure the window is properly displayed
+        for (int i = 0; i < 5; i++) {
+            render();
+            SDL_Delay(100);
+        }
+
+        // Process any pending events
+        SDL_Event event;
+        while (SDL_PollEvent(&event)) {
+            // Just drain the event queue
+        }
+
+        // Additional delay to ensure window is ready
+        SDL_Delay(500);
+
+        // Get and display window position for debugging
+        int x, y;
+        SDL_GetWindowPosition(window, &x, &y);
+        std::cout << "Window position: (" << x << ", " << y << ")" << std::endl;
+    }
+
     int width, height;
     bool running;
     bool headless;
@@ -181,25 +202,32 @@ class RobotTestApp {
 };
 
 int main(int argc, char* argv[]) {
-    bool headless = false;
     bool runTests = false;
+    int waitTime = 2000; // Default wait time in ms before tests
 
     // Parse command line arguments
     for (int i = 1; i < argc; i++) {
         std::string arg = argv[i];
-        if (arg == "--headless") {
-            headless = true;
-        }
-        else if (arg == "--run-tests") {
+        if (arg == "--run-tests") {
             runTests = true;
         }
+        else if (arg == "--wait-time" && i + 1 < argc) {
+            waitTime = std::stoi(argv[i + 1]);
+            i++;
+        }
     }
 
-    // Create test application
-    RobotTestApp app(800, 600, headless);
+    // Create test application (never headless to ensure window is visible)
+    RobotTestApp app(800, 600, false);
 
     // Either run tests or interactive mode
     if (runTests) {
+        std::cout << "Initializing test window..." << std::endl;
+
+        // Wait before starting tests to ensure window is ready
+        std::cout << "Waiting " << waitTime/1000.0 << " seconds before starting tests..." << std::endl;
+        SDL_Delay(waitTime);
+
         bool success = app.runTests();
         return success ? 0 : 1;
     } else {