Fixes

Author: lukekim

.coverage

@@ -109,8 +109,15 @@ jobs:
           cd spice_qs
           spice add spiceai/quickstart
           spiced &> spice.log &
-          # time to initialize added dataset
-          sleep 10
+          # Wait for runtime to be ready (poll /v1/ready endpoint)
+          for i in {1..60}; do
+            if curl -s http://localhost:8090/v1/ready > /dev/null 2>&1; then
+              echo "Spice runtime is ready"
+              break
+            fi
+            echo "Waiting for Spice runtime to be ready... ($i/60)"
+            sleep 1
+          done
 
       - name: Init and start spice app (Windows)
         if: matrix.os == 'windows-latest'
@@ -119,15 +126,29 @@ jobs:
           cd spice_qs
           spice add spiceai/quickstart
           Start-Process -FilePath spice run
-          # time to initialize added dataset
-          Start-Sleep -Seconds 10
+          # Wait for runtime to be ready (poll /v1/ready endpoint)
+          $timeout = 60
+          $elapsed = 0
+          while ($elapsed -lt $timeout) {
+            try {
+              $response = Invoke-WebRequest -Uri "http://localhost:8090/v1/ready" -UseBasicParsing -TimeoutSec 2 -ErrorAction Stop
+              if ($response.StatusCode -eq 200) {
+                Write-Host "Spice runtime is ready"
+                break
+              }
+            } catch {
+              Write-Host "Waiting for Spice runtime to be ready... ($elapsed/$timeout)"
+            }
+            Start-Sleep -Seconds 1
+            $elapsed++
+          }
         shell: pwsh
 
       - name: Running tests
         env:
           API_KEY: ${{ secrets.API_KEY }}
           # run spice.ai cloud tests only on linux with single python version, to avoid concurrent requests
-          TEST_SPICE_CLOUD: "false"
+          TEST_SPICE_CLOUD: 'false'
         run: |
           pytest -s tests/test_main.py -k "not cloud"
 
@@ -158,7 +179,7 @@ jobs:
       - name: Run cloud tests
         env:
           SPICE_API_KEY: ${{ secrets.SCP_SPICEAI_TPCH_API_KEY }}
-          TEST_SPICE_CLOUD: "true"
+          TEST_SPICE_CLOUD: 'true'
         run: |
           pytest -s tests/test_main.py -k "cloud"
 
@@ -184,7 +205,7 @@ jobs:
           pytest --cov=spicepy --cov-report=xml --cov-report=term-missing --ignore=tests/test_main.py tests/
       - name: Upload coverage to Codecov
         if: matrix.python-version == '3.12'
-        uses: codecov/codecov-action@18283e04ce6e62d37312384ff67231eb8fd56d24  # v4.5.0
+        uses: codecov/codecov-action@18283e04ce6e62d37312384ff67231eb8fd56d24 # v4.5.0
         with:
           files: ./coverage.xml
           fail_ci_if_error: false