Enhance health check and desktop snapshot tests with detailed debug logging for api_success tracking

Author: abutbul

.github/workflows/test-dynamic-method.yml

@@ -190,8 +190,9 @@ jobs:
           
           if [[ "$health_status" == "200" ]]; then
             echo "✅ Health check passed on attempt $attempt"
-            ((api_success++))
-            echo "DEBUG: api_success value is now $api_success"
+            echo "DEBUG: api_success before: $api_success"
+            api_success=$((api_success + 1))
+            echo "DEBUG: api_success after: $api_success"
             break
           else
             echo "⏳ Health check attempt $attempt failed, retrying..."
@@ -206,16 +207,18 @@ jobs:
         
         # Test desktop snapshot with retry
         echo "Testing desktop snapshot..."
-        echo "DEBUG: Starting desktop snapshot test"
+        echo "DEBUG: Starting desktop snapshot test with set -e state: $-"
+        echo "DEBUG: Testing if we get here"
         for attempt in {1..5}; do
           echo "DEBUG: Attempting desktop snapshot $attempt"
           snapshot_status=$(curl -f -s -w "%{http_code}" http://localhost:5000/desktop-snapshot -o /dev/null || echo "failed")
           echo "DEBUG: Snapshot status code: $snapshot_status"
           
           if [[ "$snapshot_status" == "200" ]]; then
             echo "✅ Desktop snapshot passed on attempt $attempt"
-            ((api_success++))
-            echo "DEBUG: api_success value is now $api_success"
+            echo "DEBUG: api_success before: $api_success"
+            api_success=$((api_success + 1))
+            echo "DEBUG: api_success after: $api_success"
             break
           else
             echo "⏳ Desktop snapshot attempt $attempt failed, retrying..."