pr ci include test runs against api mock

Using openapi.yml godon-api mock deliberately. That avoids having to
start godon-api dependencies and keeps the test flow simple.

Author: cherusk

.github/workflows/ci.yml

@@ -32,7 +32,7 @@ jobs:
 
         # Set environment variables for this session
         export SSL_CERT_FILE="/etc/ssl/certs/ca-bundle.crt"
-        export NIX_SSL_CERT_FILE="/etc/ssl/certs/ca-bundle.crt" 
+        export NIX_SSL_CERT_FILE="/etc/ssl/certs/ca-bundle.crt"
         export CURL_CA_BUNDLE="/etc/ssl/certs/ca-bundle.crt"
 
         # Add to nix.conf for daemon
@@ -58,3 +58,90 @@ jobs:
         ls -la $(nix path-info .#default)/bin/ || echo "$out/bin not found"
         echo "Testing compiled binary with direct path..."
         $(nix path-info .#default)/bin/godon_cli --help
+
+    - name: Start Prism mock container
+      run: |
+        # Download the OpenAPI spec with retry
+        echo "Downloading OpenAPI spec..."
+        curl -L --retry 3 --retry-delay 5 -o openapi.yml \
+          https://raw.githubusercontent.com/godon-dev/godon-images/main/images/godon-api/openapi.yml
+
+        # Verify the spec was downloaded
+        if [ ! -f openapi.yml ]; then
+          echo "Failed to download OpenAPI spec"
+          exit 1
+        fi
+        echo "OpenAPI spec downloaded successfully"
+        head -5 openapi.yml
+
+        # Start Prism container in background
+        docker run -d --name prism -p 4010:4010 \
+          -v $(pwd)/openapi.yml:/tmp/openapi.yml \
+          stoplight/prism:4 \
+          mock --host 0.0.0.0 /tmp/openapi.yml
+
+        # Wait for Prism to start
+        echo "Waiting for Prism to start..."
+        sleep 15
+
+        # Verify Prism is running with retry
+        for i in {1..5}; do
+          if curl -f http://localhost:4010/health; then
+            echo "Prism container started successfully on port 4010"
+            break
+          else
+            echo "Attempt $i: Prism not ready yet, waiting..."
+            sleep 5
+          fi
+        done
+
+    - name: Integration tests against Prism mock
+      run: |
+        # Get the binary path
+        BINARY_PATH=$(nix path-info .#default)/bin/godon_cli
+
+        echo "Running integration tests against Prism mock..."
+
+        # Test breeder list
+        echo "Testing: breeder list"
+        $BINARY_PATH --hostname=localhost --port=4010 breeder list
+
+        # Create test YAML files using echo
+        echo 'name: "Test Breeder"' > test_breeder.yml
+        echo 'description: "Integration test breeder"' >> test_breeder.yml
+        echo 'config:' >> test_breeder.yml
+        echo '  setting1: "value1"' >> test_breeder.yml
+        echo '  setting2: 42' >> test_breeder.yml
+
+        echo "Testing: breeder create"
+        $BINARY_PATH --hostname=localhost --port=4010 breeder create --file=test_breeder.yml
+
+        # Test breeder show with a mock UUID
+        echo "Testing: breeder show"
+        $BINARY_PATH --hostname=localhost --port=4010 breeder show --uuid=123e4567-e89b-12d3-a456-426614174000
+
+        # Create update test file
+        echo 'name: "Updated Test Breeder"' > test_breeder_update.yml
+        echo 'description: "Updated integration test breeder"' >> test_breeder_update.yml
+        echo 'config:' >> test_breeder_update.yml
+        echo '  setting1: "updated_value1"' >> test_breeder_update.yml
+        echo '  setting2: 100' >> test_breeder_update.yml
+        echo '  new_setting: "new_value"' >> test_breeder_update.yml
+
+        echo "Testing: breeder update"
+        $BINARY_PATH --hostname=localhost --port=4010 breeder update --file=test_breeder_update.yml
+
+        # Test breeder purge
+        echo "Testing: breeder purge"
+        $BINARY_PATH --hostname=localhost --port=4010 breeder purge --uuid=123e4567-e89b-12d3-a456-426614174000
+
+        # Test help commands
+        echo "Testing: help commands"
+        $BINARY_PATH --help
+        $BINARY_PATH breeder --help || true  # May fail but tests argument parsing
+
+    - name: Cleanup Prism container
+      if: always()
+      run: |
+        docker stop prism || true
+        docker rm prism || true

src/godon/client.nim

@@ -33,7 +33,7 @@ proc baseUrl*(client: GodonClient): string =
 
 proc handleResponse*[T](client: GodonClient; response: Response): ApiResponse[T] =
   ## Handle HTTP response and convert to ApiResponse
-  let statusCode = parseInt(response.status)
+  let statusCode = parseInt(split(response.status, " ")[0])
   if statusCode >= 200 and statusCode < 300:
     try:
       let jsonData = parseJson(response.body)
@@ -50,7 +50,7 @@ proc handleResponse*[T](client: GodonClient; response: Response): ApiResponse[T]
 
 proc handleError*(client: GodonClient, response: Response): ref CatchableError =
   ## Convert HTTP error response to exception
-  let statusCode = parseInt(response.status)
+  let statusCode = parseInt(split(response.status, " ")[0])
   var errorMsg = "HTTP Error: " & $statusCode
   try:
     let errorJson = parseJson(response.body)

src/godon_cli.nim

@@ -41,6 +41,7 @@ proc parseArgs(): (string, string, int, string, seq[string]) =
   var p = initOptParser(commandLineParams())
 
   for kind, key, val in p.getopt():
+    echo "Parsed: kind=", kind, " key=", key, " val=", val
     case kind
     of cmdArgument:
       if command.len == 0:
@@ -53,6 +54,8 @@ proc parseArgs(): (string, string, int, string, seq[string]) =
       of "hostname":
         hostname = val
       of "port":
+        if val.len == 0:
+          writeError("Port option requires a value")
         try:
           port = parseInt(val)
         except ValueError: