Merge pull request #10 from ElcanoTek/codex/move-tests-to-dedicated-folder-and-clean-up-du

Organize tests and auto-clean local database

Author: bradflaugher

.github/workflows/test-victoria.yml

@@ -39,9 +39,11 @@ jobs:
         if [[ "${{ runner.os }}" == "Windows" ]]; then
           export PYTHONIOENCODING=utf-8
           chcp 65001 || true  # Set Windows console to UTF-8
-          python test_victoria.py
+          python tests/test_victoria.py
+          python tests/test_non_interactive.py
         else
-          python3 test_victoria.py
+          python3 tests/test_victoria.py
+          python3 tests/test_non_interactive.py
         fi
 
     - name: Test script execution with different terminal environments

TESTING.md

@@ -26,16 +26,17 @@ The GitHub Actions workflow (`.github/workflows/test-victoria.yml`) automaticall
 
 ### Running Tests Locally
 
-To run the test suite locally:
+All test scripts now reside in the `tests/` directory. To run the full test suite:
 
 ```bash
-python test_victoria.py
+pytest tests/test_victoria.py tests/test_non_interactive.py
 ```
 
-Or with Python 3 explicitly:
+Or run the scripts directly:
 
 ```bash
-python3 test_victoria.py
+python tests/test_victoria.py
+python tests/test_non_interactive.py
 ```
 
 ### Test Script Features

test_non_interactive.py tests/test_non_interactive.pytest_non_interactive.py renamed to tests/test_non_interactive.py

@@ -81,20 +81,29 @@ def test_victoria_with_interrupt():
     try:
         python_cmd = sys.executable
 
+        # On Windows we need a process group to send CTRL_BREAK_EVENT
+        creationflags = 0
+        if platform.system() == 'Windows':
+            creationflags = subprocess.CREATE_NEW_PROCESS_GROUP
+
         # Start the process
         process = subprocess.Popen(
             [python_cmd, 'victoria.py'],
             stdin=subprocess.PIPE,
             stdout=subprocess.PIPE,
             stderr=subprocess.PIPE,
-            text=True
+            text=True,
+            creationflags=creationflags
         )
-        
+
         # Let it run for a moment
         time.sleep(1)
-        
+
         # Send interrupt signal (Ctrl+C)
-        process.send_signal(signal.SIGINT)
+        if platform.system() == 'Windows':
+            process.send_signal(signal.CTRL_BREAK_EVENT)
+        else:
+            process.send_signal(signal.SIGINT)
 
         try:
             stdout, stderr = process.communicate(timeout=5)

test_victoria.py tests/test_victoria.pytest_victoria.py renamed to tests/test_victoria.py

@@ -369,4 +369,4 @@ def main():
         return 1
 
 if __name__ == "__main__":
-    sys.exit(main())
+    sys.exit(main())

victoria.py

@@ -657,9 +657,22 @@ def course_menu() -> str:
         warn("Invalid selection. Please choose 1 or 2.")
 
 # ------------------ Main ------------------
+def remove_local_duckdb():
+    """Remove local DuckDB file to ensure a clean start."""
+    db_path = Path("data") / "adtech.duckdb"
+    try:
+        if db_path.exists():
+            db_path.unlink()
+            info(f"Removed local database: {db_path}")
+        else:
+            info(f"No local database found at {db_path}")
+    except Exception as e:
+        warn(f"Could not remove {db_path}: {e}")
+
 def main():
     clear_screen()
     banner()
+    remove_local_duckdb()
 
     print(f"{T.GREEN}{T.ROCKET} Welcome to Victoria - Your AdTech Data Navigator!{T.NC}")