Update logging;

Author: JamesonRGrieve

.github/workflows/test-pytest.yml

@@ -95,24 +95,23 @@ jobs:
 
       - name: Install dependencies
         run: |
-          python -m pip install --upgrade pip
-          pip install pytest pytest-json-report pytest-asyncio
+          python -m pip install --upgrade pip --quiet
+          pip install pytest pytest-json-report pytest-asyncio --quiet
           if [ -f requirements.txt ]; then 
-            pip install -r requirements.txt
+            pip install -r requirements.txt --quiet
           fi
 
       - name: Check for test collection errors
         id: check-collection
         run: |
-          # Create verbose debug file for artifact
-          exec 3>&1 4>&2
-          exec 1> >(tee -a debug_collection.log) 2>&1
-
           echo "Running pytest collection check..."
+
+          # Redirect ALL output to files, including subprocess stderr
           python -m pytest --collect-only -v > collection_output.txt 2>&1 || true
 
-          # Restore stdout/stderr for GitHub Actions
-          exec 1>&3 2>&4
+          # Save to debug file for artifact
+          echo "=== COLLECTION CHECK OUTPUT ===" >> debug_collection.log
+          cat collection_output.txt >> debug_collection.log
 
           # Set default values
           HAS_COLLECTION_ERRORS="false"
@@ -183,17 +182,24 @@ jobs:
             echo "has_errors=false" >> $GITHUB_OUTPUT
           fi
 
-          # Save full collection output to debug file for artifact (full details preserved)
-          echo "=== FULL COLLECTION OUTPUT ===" >> debug_collection.log
-          cat collection_output.txt >> debug_collection.log
-
       - name: Run tests on PR Branch
         if: steps.check-collection.outputs.has_collection_errors != 'true'
+        env:
+          PYTHONUNBUFFERED: "1"
+          PYTHONIOENCODING: "utf-8"
+          PYTHONWARNINGS: "ignore"
+          MIGRATION_MANAGER_LOG_LEVEL: "ERROR"
+          MIGRATION_MANAGER_QUIET: "true"
+          SQLALCHEMY_WARN_20: "false"
+          SQLALCHEMY_SILENCE_UBER_WARNING: "1"
         run: |
           echo "Running tests on PR branch..."
 
-          # Run pytest with minimal console output, redirect everything to files
-          python -m pytest --json-report --json-report-file=pr_results.json > test_run_output.log 2>&1 || true
+          # Suppress Python logging and redirect all output
+          python -c "import logging; logging.getLogger().setLevel(logging.ERROR); logging.getLogger('migration_manager').setLevel(logging.ERROR)" 2>/dev/null || true
+
+          # Run pytest with comprehensive output suppression
+          python -m pytest --json-report --json-report-file=pr_results.json --tb=no -q --disable-warnings --log-level=ERROR > test_run_output.log 2>&1 || true
 
           # Save output to debug file for artifact
           echo "=== PYTEST RUN OUTPUT ===" >> debug_test_run.log
@@ -359,24 +365,23 @@ jobs:
 
       - name: Install dependencies
         run: |
-          python -m pip install --upgrade pip
-          pip install pytest pytest-json-report pytest-asyncio
+          python -m pip install --upgrade pip --quiet
+          pip install pytest pytest-json-report pytest-asyncio --quiet
           if [ -f requirements.txt ]; then 
-            pip install -r requirements.txt
+            pip install -r requirements.txt --quiet
           fi
 
       - name: Check for test collection errors
         id: check-collection
         run: |
-          # Create verbose debug file for artifact
-          exec 3>&1 4>&2
-          exec 1> >(tee -a debug_target_collection.log) 2>&1
-
           echo "Running pytest collection check..."
+
+          # Redirect ALL output to files, including subprocess stderr
           python -m pytest --collect-only -v > collection_output.txt 2>&1 || true
 
-          # Restore stdout/stderr for GitHub Actions
-          exec 1>&3 2>&4
+          # Save to debug file for artifact
+          echo "=== COLLECTION CHECK OUTPUT ===" >> debug_target_collection.log
+          cat collection_output.txt >> debug_target_collection.log
 
           # Set default values
           HAS_COLLECTION_ERRORS="false"
@@ -447,17 +452,24 @@ jobs:
             echo "has_errors=false" >> $GITHUB_OUTPUT
           fi
 
-          # Save full collection output to debug file for artifact (full details preserved)
-          echo "=== FULL COLLECTION OUTPUT ===" >> debug_target_collection.log
-          cat collection_output.txt >> debug_target_collection.log
-
       - name: Run tests on target branch
         if: steps.check-collection.outputs.has_collection_errors != 'true'
+        env:
+          PYTHONUNBUFFERED: "1"
+          PYTHONIOENCODING: "utf-8"
+          PYTHONWARNINGS: "ignore"
+          MIGRATION_MANAGER_LOG_LEVEL: "ERROR"
+          MIGRATION_MANAGER_QUIET: "true"
+          SQLALCHEMY_WARN_20: "false"
+          SQLALCHEMY_SILENCE_UBER_WARNING: "1"
         run: |
           echo "Running tests on target branch..."
 
-          # Run pytest with minimal console output, redirect everything to files
-          python -m pytest --json-report --json-report-file=target_results.json > target_test_run_output.log 2>&1 || true
+          # Suppress Python logging and redirect all output
+          python -c "import logging; logging.getLogger().setLevel(logging.ERROR); logging.getLogger('migration_manager').setLevel(logging.ERROR)" 2>/dev/null || true
+
+          # Run pytest with comprehensive output suppression
+          python -m pytest --json-report --json-report-file=target_results.json --tb=no -q --disable-warnings --log-level=ERROR > target_test_run_output.log 2>&1 || true
 
           # Save output to debug file for artifact
           echo "=== PYTEST RUN OUTPUT ===" >> debug_target_test_run.log
@@ -552,6 +564,7 @@ jobs:
           path: |
             target_test_data.json
             debug_target_*.log
+            target_test_run_output.log
             collection_output.txt
             error_details.txt
             target_results.json