feat: enhance error handling in FastAPI endpoint tests and increase service wait time

Author: Acuspeedster

.github/workflows/test-mcp-server.yml

@@ -20,8 +20,11 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/[email protected] 
+        uses: actions/[email protected]
 
+      - name: Install jq
+        run: sudo apt-get install -y jq
+        
       - name: Run docker compose
         uses: hoverkraft-tech/[email protected]
         with:
@@ -31,47 +34,75 @@ jobs:
       - name: Wait for services to be ready
         run: |
           echo "Waiting for services to be ready..."
-          sleep 30  # Give services time to initialize
-          docker ps  # Display running containers
+          # Longer wait time for services
+          sleep 60
+          # Show running containers
+          docker ps
 
       - name: Install cmcp client
         run: |
           pip install cmcp
 
-      - name: Test FastAPI endpoints
+      - name: Test FastAPI endpoints with better error handling
         run: |
           echo "Testing FastAPI /mcp/compile endpoint..."
-          curl -s -X POST http://localhost:8000/mcp/compile \
+          RESPONSE=$(curl -s -S -f -X POST http://localhost:8000/mcp/compile \
             -H "Content-Type: application/json" \
             -d '{
               "code": "[filename: Cargo.toml]\n[package]\nname = \"hello_world\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n[dependencies]\n\n[filename: src/main.rs]\nfn main() {\n    println!(\"Hello, World!\");\n}"
-            }' | jq
+            }' || echo "CURL_FAILED")
+          
+          if [ "$RESPONSE" = "CURL_FAILED" ]; then
+            echo "Failed to connect to FastAPI service. Debug information:"
+            docker ps
+            docker logs $(docker ps -q --filter name=api)
+            exit 1
+          fi
+          
+          echo "$RESPONSE" | jq || echo "$RESPONSE"
           
           echo "Testing FastAPI /mcp/compile-and-fix endpoint..."
-          curl -s -X POST http://localhost:8000/mcp/compile-and-fix \
+          RESPONSE=$(curl -s -S -f -X POST http://localhost:8000/mcp/compile-and-fix \
             -H "Content-Type: application/json" \
             -d '{
               "code": "[filename: Cargo.toml]\n[package]\nname = \"hello_world\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n[dependencies]\n\n[filename: src/main.rs]\nfn main() {\n    println!(\"Hello, World!\" // Missing closing parenthesis\n}",
               "description": "A simple hello world program",
               "max_attempts": 3
-            }'
+            }' || echo "CURL_FAILED")
+            
+          if [ "$RESPONSE" = "CURL_FAILED" ]; then
+            echo "Failed to connect to FastAPI service. Debug information:"
+            docker ps
+            docker logs $(docker ps -q --filter name=api)
+            exit 1
+          fi
+          
+          echo "$RESPONSE"
 
       - name: Test MCP Server with cmcp client
         run: |
           echo "Testing MCP server compile method..."
           echo '{
             "code": "[filename: Cargo.toml]\n[package]\nname = \"hello_world\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n[dependencies]\n\n[filename: src/main.rs]\nfn main() {\n    println!(\"Hello, World!\");\n}"
-          }' | cmcp call http://localhost:3000 rust-compiler compile
+          }' | cmcp call http://localhost:3000 rust-compiler compile || { 
+            echo "CMCP call failed. Debug information:"; 
+            docker ps; 
+            exit 1; 
+          }
           
           echo "Testing MCP server compileAndFix method..."
           echo '{
             "code": "[filename: Cargo.toml]\n[package]\nname = \"hello_world\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n[dependencies]\n\n[filename: src/main.rs]\nfn main() {\n    println!(\"Hello, World!\" // Missing closing parenthesis\n}",
             "description": "A simple hello world program",
             "max_attempts": 3
-          }' | cmcp call http://localhost:3000 rust-compiler compileAndFix
+          }' | cmcp call http://localhost:3000 rust-compiler compileAndFix || { 
+            echo "CMCP call failed. Debug information:"; 
+            docker ps; 
+            exit 1; 
+          }
 
       - name: Check Docker logs on failure
         if: failure()
         run: |
           echo "Checking Docker logs for troubleshooting..."
-          docker-compose logs
+          docker compose logs || echo "Failed to get logs"