test: increase code coverage to 85%

- Add comprehensive tests for MonitorManager (SQLite-specific methods, query stats)
- Add tests for Seed class helper methods (schema, find, execute, insert, update, delete, raw)
- Add tests for ExternalReferenceProcessingTrait (WHERE EXISTS/NOT EXISTS scenarios)
- Add tests for BaseCliCommand (loadEnvFile, buildConfigFromEnv, readInput, readConfirmation, loadCacheConfig)
- Extend RetryableConnection tests (waitBeforeRetry, config validation, retry operations)
- Fix readPassword() to return empty string in non-interactive mode to prevent blocking
- Fix SeedGenerator to skip interactive prompts in non-interactive mode
- Ensure all CLI tests set PDODB_NON_INTERACTIVE and PHPUNIT environment variables
- Add non-interactive mode check in bin/pdodb script
- Improve environment variable cleanup in test tearDown methods

All tests pass without blocking or requiring user input.

Author: tommyknocker

bin/pdodb

@@ -48,6 +48,12 @@ require_once $autoloadPath;
 
 use tommyknocker\pdodb\cli\Application;
 
+// Ensure non-interactive mode for tests and CI
+if (getenv('PHPUNIT') !== false || getenv('PDODB_NON_INTERACTIVE') !== false) {
+    putenv('PDODB_NON_INTERACTIVE=1');
+    putenv('PHPUNIT=1');
+}
+
 $application = new Application();
 exit($application->run($argv));
 

src/cli/BaseCliCommand.php

@@ -440,9 +440,9 @@ protected static function readPassword(string $prompt): string
             || !stream_isatty(STDIN);
 
         if ($nonInteractive) {
-            // In non-interactive mode, try to read from stdin if available
-            $input = trim((string)fgets(STDIN));
-            return $input;
+            // In non-interactive mode, return empty string to avoid blocking on fgets(STDIN)
+            // Tests and CI environments should provide passwords via environment variables or other means
+            return '';
         }
 
         // Use stty to hide input on Unix-like systems

src/cli/SeedGenerator.php

@@ -44,19 +44,25 @@ public static function generate(?string $name = null, ?string $seedPath = null):
             }
         }
 
-        // Suggest seed type based on name
-        $suggestions = static::suggestSeedType($name);
-        if (!empty($suggestions)) {
-            echo "\nSuggested seed types:\n";
-            foreach ($suggestions as $i => $suggestion) {
-                echo '  ' . ($i + 1) . ". {$suggestion}\n";
-            }
-            echo "  0. Custom (manual)\n";
-            $choice = static::readInput("\nSelect seed type", '0');
-            $selectedIndex = (int)$choice - 1;
-            if ($selectedIndex >= 0 && $selectedIndex < count($suggestions)) {
-                $seedType = $suggestions[$selectedIndex];
-                static::info("Selected: {$seedType}");
+        // Suggest seed type based on name (skip in non-interactive mode)
+        $nonInteractive = getenv('PDODB_NON_INTERACTIVE') !== false
+            || getenv('PHPUNIT') !== false
+            || !stream_isatty(STDIN);
+        
+        if (!$nonInteractive) {
+            $suggestions = static::suggestSeedType($name);
+            if (!empty($suggestions)) {
+                echo "\nSuggested seed types:\n";
+                foreach ($suggestions as $i => $suggestion) {
+                    echo '  ' . ($i + 1) . ". {$suggestion}\n";
+                }
+                echo "  0. Custom (manual)\n";
+                $choice = static::readInput("\nSelect seed type", '0');
+                $selectedIndex = (int)$choice - 1;
+                if ($selectedIndex >= 0 && $selectedIndex < count($suggestions)) {
+                    $seedType = $suggestions[$selectedIndex];
+                    static::info("Selected: {$seedType}");
+                }
             }
         }