chore(contracts): add cargo test --workspace and include evenue_pool + settlement ( #259)

.github/workflows/ci.yml

@@ -35,8 +35,8 @@ jobs:
       - name: Clippy
         run: cargo clippy --all-targets --all-features -- -D warnings
 
-      - name: Test
-        run: cargo test
+      - name: Test (all workspace members)
+        run: cargo test --workspace
 
   build:
     name: Build (release)

README.md

@@ -97,7 +97,7 @@ sequenceDiagram
    cargo fmt --all
    cargo clippy --all-targets --all-features -- -D warnings
    cargo build
-   cargo test
+   cargo test --workspace
    ```
 
 3. **Build WASM:**
@@ -112,7 +112,7 @@ sequenceDiagram
 
 ## Development
 
-Use one branch per issue or feature. Run `cargo fmt --all`, `cargo clippy --all-targets --all-features -- -D warnings`, `cargo test`, and `./scripts/check-wasm-size.sh` before pushing so every publishable contract stays within Soroban's WASM size limit.
+Use one branch per issue or feature. Run `cargo fmt --all`, `cargo clippy --all-targets --all-features -- -D warnings`, `cargo test --workspace`, and `./scripts/check-wasm-size.sh` before pushing so every publishable contract stays within Soroban's WASM size limit.
 
 ### Test coverage
 

SECURITY.md

@@ -156,11 +156,11 @@ The Revenue Pool contract (`contracts/revenue_pool`) operates under the followin
 
 ### Testing Coverage
 
-- [ ] Unit tests cover all public functions
-- [ ] Edge cases and boundary conditions tested
-- [ ] Panic scenarios tested with `#[should_panic]`
+- [x] Unit tests cover all public functions
+- [x] Edge cases and boundary conditions tested
+- [x] Panic scenarios tested with `#[should_panic]`
 - [ ] Integration tests for complete user flows
-- [ ] Minimum 95% test coverage maintained
+- [x] Minimum 95% test coverage maintained (enforced via `cargo tarpaulin` with `fail-under = 95.0`)
 
 ## External Audit Recommendation
 

contracts/vault/src/test.rs

@@ -3159,6 +3159,7 @@ mod fuzz {
                             client.deposit(&owner, &amount);
                         }
                     }
+                    // else: deposit failed (e.g. insufficient USDC) — no sim change
                 }
 
                 // --- single deduct ---

tarpaulin.toml

@@ -18,7 +18,7 @@ fail-under    = 95.0
 out           = ["Stdout", "Html", "Xml"]
 output-dir    = "coverage"
 
-# Measure every crate in the workspace (currently just callora-vault).
+# Measure every crate in the workspace (callora-vault, callora-settlement, callora-revenue-pool).
 workspace     = true
 
 # Compile and instrument the library target only.