Merge pull request #337 from adamantivm/jac/rust_cicd

Add Rust CI/CD workflow with comprehensive testing

Author: adamantivm

.github/workflows/python-app.yml

@@ -5,9 +5,19 @@ name: Python application
 
 on:
   push:
-    branches: ["main"]
+    paths:
+      - 'deep_quoridor/src/**'
+      - 'deep_quoridor/test/**'
+      - 'deep_quoridor/experiments/**'
+      - 'deep_quoridor/*requirements.txt'
+      - '.github/workflows/python-app.yml'
   pull_request:
-    branches: ["main"]
+    paths:
+      - 'deep_quoridor/src/**'
+      - 'deep_quoridor/test/**'
+      - 'deep_quoridor/experiments/**'
+      - 'deep_quoridor/*requirements.txt'
+      - '.github/workflows/python-app.yml'
   workflow_dispatch:
     inputs:
       debug_enabled:

.github/workflows/rust-ci.yml

@@ -0,0 +1,96 @@
+name: Rust CI
+
+on:
+  push:
+    paths:
+      - 'deep_quoridor/rust/**'
+      - '.github/workflows/rust-ci.yml'
+  pull_request:
+    paths:
+      - 'deep_quoridor/rust/**'
+      - '.github/workflows/rust-ci.yml'
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      
+      - name: Install Rust toolchain
+        uses: dtolnay/rust-toolchain@stable
+        with:
+          components: rustfmt, clippy
+      
+      - name: Cache cargo registry
+        uses: actions/cache@v4
+        with:
+          path: ~/.cargo/registry
+          key: ${{ runner.os }}-cargo-registry-${{ hashFiles('**/Cargo.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-cargo-registry-
+      
+      - name: Cache cargo index
+        uses: actions/cache@v4
+        with:
+          path: ~/.cargo/git
+          key: ${{ runner.os }}-cargo-git-${{ hashFiles('**/Cargo.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-cargo-git-
+      
+      - name: Check formatting
+        working-directory: deep_quoridor/rust
+        run: cargo fmt --all -- --check
+      
+      - name: Run clippy
+        working-directory: deep_quoridor/rust
+        run: cargo clippy --all-targets --all-features
+      
+      - name: Build
+        working-directory: deep_quoridor/rust
+        run: cargo build --verbose
+
+      - name: Run tests with all features
+        working-directory: deep_quoridor/rust
+        env:
+          RUST_BACKTRACE: 1
+        run: cargo test --all-features --verbose
+
+      - name: Build release
+        working-directory: deep_quoridor/rust
+        run: cargo build --release --verbose
+
+  build-python-extension:
+    name: Build Python Extension
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+      
+      - name: Install Rust toolchain
+        uses: dtolnay/rust-toolchain@stable
+      
+      - name: Install maturin
+        run: pip install maturin
+      
+      - name: Build Python wheel
+        working-directory: deep_quoridor/rust
+        run: maturin build --release
+      
+      - name: Test Python import
+        working-directory: deep_quoridor/rust
+        run: |
+          pip install target/wheels/*.whl
+          python -c "import quoridor_rs; print('Successfully imported quoridor_rs')"

deep_quoridor/rust/src/compact/q_game_mechanics.rs

@@ -685,8 +685,12 @@ mod tests {
         // This should be invalid if it blocks the only remaining path
         // (depends on whether there's still a way around)
 
-        // For now, just test that pathfinding completes without error
-        assert!(is_valid || !is_valid, "Pathfinding should complete");
+        // Log the result to verify pathfinding completed
+        eprintln!(
+            "Pathfinding completed successfully: wall at (2,1) is {}",
+            if is_valid { "valid" } else { "invalid" }
+        );
+        // Test passes as long as we get here without panic
     }
 
     #[test]

deep_quoridor/rust/src/validation.rs

@@ -199,7 +199,13 @@ pub fn is_wall_action_valid_mut(
     }
 
     // Check if wall cells are free
-    if !check_wall_cells(&grid.view(), wall_row, wall_col, wall_orientation, CELL_FREE) {
+    if !check_wall_cells(
+        &grid.view(),
+        wall_row,
+        wall_col,
+        wall_orientation,
+        CELL_FREE,
+    ) {
         return false;
     }