fix: improve workflow test execution and error handling

- Improve test output capture with tee for better visibility
- Separate test execution from report generation failures
- Add proper error handling for test report generation
- Ensure workflow fails only on actual test failures, not report issues
- Enhance logging and feedback for debugging workflow issues

📋 Change summary:
* .github/workflows/test.yml: Enhanced test execution and error handling

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: PaulDuvall

.github/workflows/test.yml

@@ -106,27 +106,32 @@ jobs:
           # Run comprehensive test suite with report generation
           echo "▶️  Running comprehensive test suite with report generation..."
           
-          # Run tests and capture output for report
-          if npm test > test-output.log 2>&1; then
+          # Run tests and capture actual exit code
+          echo "▶️  Running comprehensive test suite..."
+          if npm test 2>&1 | tee test-output.log; then
             echo "✅ All consolidated tests passed"
             TEST_RESULT=0
           else
             echo "❌ Consolidated tests failed"
             TEST_RESULT=1
           fi
           
-          # Generate comprehensive test report
+          # Generate comprehensive test report (don't let this fail the build)
           echo "📊 Generating test report..."
-          node scripts/generate-test-report.js || true
-          
-          # Display test output
-          cat test-output.log
+          if node scripts/generate-test-report.js 2>/dev/null; then
+            echo "✅ Test report generated successfully"
+          else
+            echo "⚠️ Test report generation had issues, but continuing..."
+          fi
           
           cd ..
           
-          # Exit with test result
+          # Exit with actual test result, not report generation result
           if [ $TEST_RESULT -ne 0 ]; then
+            echo "❌ Tests failed - check output above"
             exit 1
+          else
+            echo "✅ All tests passed"
           fi
         else
           echo "❌ No claude-dev-toolkit test directory found"