diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2025-12-02 21:20:17 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2025-12-02 21:20:17 +0000 |
| commit | 72683beea066d066637e747c40dc859fb709babf (patch) | |
| tree | 2db3462ebbcb7501e56491148cc3ffa7aa294071 /grasp-audit/src/specs | |
| parent | 5c10ca008413744b09136618eaa85275c997704c (diff) | |
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
Diffstat (limited to 'grasp-audit/src/specs')
| -rw-r--r-- | grasp-audit/src/specs/grasp01/cors.rs | 2 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/event_acceptance_policy.rs | 2 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/nip01_smoke.rs | 2 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/nip11_document.rs | 2 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/spec_requirements.rs | 2 |
5 files changed, 5 insertions, 5 deletions
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 { | |||
| 459 | .trim_end_matches('/') | 459 | .trim_end_matches('/') |
| 460 | .to_string(); | 460 | .to_string(); |
| 461 | 461 | ||
| 462 | let config = AuditConfig::ci(); | 462 | let config = AuditConfig::isolated(); |
| 463 | let client = AuditClient::new(&relay_url, config) | 463 | let client = AuditClient::new(&relay_url, config) |
| 464 | .await | 464 | .await |
| 465 | .unwrap_or_else(|_| { | 465 | .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 { | |||
| 1139 | "RELAY_URL environment variable must be set. Example: RELAY_URL=ws://localhost:18081", | 1139 | "RELAY_URL environment variable must be set. Example: RELAY_URL=ws://localhost:18081", |
| 1140 | ); | 1140 | ); |
| 1141 | 1141 | ||
| 1142 | let config = AuditConfig::ci(); | 1142 | let config = AuditConfig::isolated(); |
| 1143 | let client = AuditClient::new(&relay_url, config) | 1143 | let client = AuditClient::new(&relay_url, config) |
| 1144 | .await | 1144 | .await |
| 1145 | .unwrap_or_else(|_| { | 1145 | .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 { | |||
| 297 | let relay_url = std::env::var("RELAY_URL") | 297 | let relay_url = std::env::var("RELAY_URL") |
| 298 | .expect("RELAY_URL environment variable must be set for integration tests"); | 298 | .expect("RELAY_URL environment variable must be set for integration tests"); |
| 299 | 299 | ||
| 300 | let config = AuditConfig::ci(); | 300 | let config = AuditConfig::isolated(); |
| 301 | let client = AuditClient::new(&relay_url, config) | 301 | let client = AuditClient::new(&relay_url, config) |
| 302 | .await | 302 | .await |
| 303 | .expect("Failed to connect to relay"); | 303 | .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 { | |||
| 288 | "RELAY_URL environment variable must be set. Example: RELAY_URL=ws://localhost:18081", | 288 | "RELAY_URL environment variable must be set. Example: RELAY_URL=ws://localhost:18081", |
| 289 | ); | 289 | ); |
| 290 | 290 | ||
| 291 | let config = AuditConfig::ci(); | 291 | let config = AuditConfig::isolated(); |
| 292 | let client = AuditClient::new(&relay_url, config) | 292 | let client = AuditClient::new(&relay_url, config) |
| 293 | .await | 293 | .await |
| 294 | .unwrap_or_else(|_| { | 294 | .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 { | |||
| 206 | fn test_requirement_count() { | 206 | fn test_requirement_count() { |
| 207 | assert_eq!(GRASP_01_REQUIREMENTS.len(), 19); | 207 | assert_eq!(GRASP_01_REQUIREMENTS.len(), 19); |
| 208 | } | 208 | } |
| 209 | } \ No newline at end of file | 209 | } |