ci

Author: Hendler

.github/workflows/python.yml

@@ -4,14 +4,14 @@ on:
   push:
     branches: [ "main" ]
     paths: # Optional: Trigger only on changes within jacspy/ or relevant files
-      - 'JACS/jacspy/**'
-      - 'JACS/jacs/**' # jacspy depends on jacs
+      - 'jacspy/**'
+      - 'jacs/**' # jacspy depends on jacs
       - '.github/workflows/python.yml'
   pull_request:
     branches: [ "main" ]
     paths: # Optional: Trigger only on changes within jacspy/ or relevant files
-      - 'JACS/jacspy/**'
-      - 'JACS/jacs/**' # jacspy depends on jacs
+      - 'jacspy/**'
+      - 'jacs/**' # jacspy depends on jacs
       - '.github/workflows/python.yml'
   workflow_dispatch: # Allows manual triggering
 
@@ -22,15 +22,31 @@ jobs:
   # Job to run tests on every push/PR
   test-jacspy:
     name: Test jacspy crate
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-latest # Docker is available on ubuntu-latest runners
 
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Run jacspy tests
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v3
+
+      - name: Build Docker image for testing
         working-directory: jacspy  
-        run: cargo test --verbose
+        run: docker buildx build --tag "jacs-build-manylinux" -f Dockerfile .
+
+      - name: Run jacspy tests in Docker
+        working-directory: JACS/jacspy # To match the PWD context of your Makefile
+        env:
+          RUST_BACKTRACE: "1" # For more detailed Rust error messages
+        # This replicates the volume mount and command structure from your Makefile:
+        # $(pwd) will be JACS/jacspy, so $(pwd)/.. will be JACS (the project root relative to GITHUB_WORKSPACE/JACS)
+        # This mounts the JACS directory into /workspace in the container.
+        run: |
+          docker run --rm \
+            -v "$(pwd)/..:/workspace" \
+            jacs-build-manylinux \
+            bash -c "cd /workspace/jacspy && cargo test --verbose"
 
   # Job to build wheels, runs ONLY on push to main
   build-jacspy-wheels:

jacspy/src/lib.rs

@@ -529,7 +529,7 @@ fn verify_response(py: Python, document_string: String) -> PyResult<PyObject> {
             e
         ))
     })?;
-    
+
     agent
         .verify_external_document_signature(&document_key)
         .map_err(|e| {