From 72683beea066d066637e747c40dc859fb709babf Mon Sep 17 00:00:00 2001 From: DanConwayDev Date: Tue, 2 Dec 2025 21:20:17 +0000 Subject: refactor: rename AuditMode variants and change CLI default to shared Breaking change: Renamed AuditMode enum variants for clarity: - AuditMode::CI -> AuditMode::Isolated (fresh fixtures per test) - AuditMode::Production -> AuditMode::Shared (reuse fixtures across tests) Config constructors renamed (with deprecated aliases): - AuditConfig::ci() -> AuditConfig::isolated() - AuditConfig::production() -> AuditConfig::shared() CLI default changed from 'ci' to 'shared' mode, which enables fixture caching across tests. This fixes the issue where fixtures were being re-created for every test in CLI mode. Fixture caching behavior: - Shared mode (CLI default): Uses client's cache, fixtures reused - Isolated mode (for cargo test): Local cache per TestContext --- grasp-audit/src/specs/grasp01/cors.rs | 2 +- grasp-audit/src/specs/grasp01/event_acceptance_policy.rs | 2 +- grasp-audit/src/specs/grasp01/nip01_smoke.rs | 2 +- grasp-audit/src/specs/grasp01/nip11_document.rs | 2 +- grasp-audit/src/specs/grasp01/spec_requirements.rs | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) (limited to 'grasp-audit/src/specs') diff --git a/grasp-audit/src/specs/grasp01/cors.rs b/grasp-audit/src/specs/grasp01/cors.rs index c877c04..409f2d3 100644 --- a/grasp-audit/src/specs/grasp01/cors.rs +++ b/grasp-audit/src/specs/grasp01/cors.rs @@ -459,7 +459,7 @@ mod tests { .trim_end_matches('/') .to_string(); - let config = AuditConfig::ci(); + let config = AuditConfig::isolated(); let client = AuditClient::new(&relay_url, config) .await .unwrap_or_else(|_| { diff --git a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs index fee51db..89220ea 100644 --- a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs +++ b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs @@ -1139,7 +1139,7 @@ mod tests { "RELAY_URL environment variable must be set. Example: RELAY_URL=ws://localhost:18081", ); - let config = AuditConfig::ci(); + let config = AuditConfig::isolated(); let client = AuditClient::new(&relay_url, config) .await .unwrap_or_else(|_| { diff --git a/grasp-audit/src/specs/grasp01/nip01_smoke.rs b/grasp-audit/src/specs/grasp01/nip01_smoke.rs index 9a7d7d1..8a0a4d1 100644 --- a/grasp-audit/src/specs/grasp01/nip01_smoke.rs +++ b/grasp-audit/src/specs/grasp01/nip01_smoke.rs @@ -297,7 +297,7 @@ mod tests { let relay_url = std::env::var("RELAY_URL") .expect("RELAY_URL environment variable must be set for integration tests"); - let config = AuditConfig::ci(); + let config = AuditConfig::isolated(); let client = AuditClient::new(&relay_url, config) .await .expect("Failed to connect to relay"); diff --git a/grasp-audit/src/specs/grasp01/nip11_document.rs b/grasp-audit/src/specs/grasp01/nip11_document.rs index 33599b1..f4ca7b4 100644 --- a/grasp-audit/src/specs/grasp01/nip11_document.rs +++ b/grasp-audit/src/specs/grasp01/nip11_document.rs @@ -288,7 +288,7 @@ mod tests { "RELAY_URL environment variable must be set. Example: RELAY_URL=ws://localhost:18081", ); - let config = AuditConfig::ci(); + let config = AuditConfig::isolated(); let client = AuditClient::new(&relay_url, config) .await .unwrap_or_else(|_| { diff --git a/grasp-audit/src/specs/grasp01/spec_requirements.rs b/grasp-audit/src/specs/grasp01/spec_requirements.rs index 591fef1..9a833d8 100644 --- a/grasp-audit/src/specs/grasp01/spec_requirements.rs +++ b/grasp-audit/src/specs/grasp01/spec_requirements.rs @@ -206,4 +206,4 @@ mod tests { fn test_requirement_count() { assert_eq!(GRASP_01_REQUIREMENTS.len(), 19); } -} \ No newline at end of file +} -- cgit v1.2.3