diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2026-02-12 12:57:44 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2026-02-12 12:57:44 +0000 |
| commit | dcaaa0c44c46f963929ab0baa91f63759ec702dc (patch) | |
| tree | 1808621ff43eb01acaabe091033672d347bfbf67 /grasp-audit/src/specs | |
| parent | 3fd6ce4149d567c67009b0332ca76c0cd6f51055 (diff) | |
refactor(grasp-audit): split ValidRepo into Sent/Served, add tolerant purgatory
- Rename ValidRepo to ValidRepoSent (announcement sent, may be in purgatory)
- Add ValidRepoServed (announcement queryable after git data pushed)
- Add send_event_and_note_purgatory() for tolerant purgatory detection
- Update fixtures to use tolerant method instead of strict assertion
- Update event_acceptance_policy tests to use ValidRepoServed
This enables tests to pass regardless of purgatory implementation status
while still having explicit purgatory tests that verify the behavior.
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 | 120 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/git_clone.rs | 6 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/git_filter.rs | 6 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/nip01_smoke.rs | 4 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/push_authorization.rs | 16 | ||||
| -rw-r--r-- | grasp-audit/src/specs/grasp01/repository_creation.rs | 6 |
7 files changed, 92 insertions, 68 deletions
diff --git a/grasp-audit/src/specs/grasp01/cors.rs b/grasp-audit/src/specs/grasp01/cors.rs index eba9e42..e5d9a27 100644 --- a/grasp-audit/src/specs/grasp01/cors.rs +++ b/grasp-audit/src/specs/grasp01/cors.rs | |||
| @@ -246,7 +246,7 @@ impl CorsTests { | |||
| 246 | let ctx = TestContext::new(client); | 246 | let ctx = TestContext::new(client); |
| 247 | 247 | ||
| 248 | // Create repository announcement to get a real repo path | 248 | // Create repository announcement to get a real repo path |
| 249 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 249 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 250 | Ok(r) => r, | 250 | Ok(r) => r, |
| 251 | Err(e) => { | 251 | Err(e) => { |
| 252 | return TestResult::new( | 252 | return TestResult::new( |
diff --git a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs index 8259283..3375c4d 100644 --- a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs +++ b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs | |||
| @@ -157,12 +157,15 @@ impl EventAcceptancePolicyTests { | |||
| 157 | let ctx = TestContext::new(client); | 157 | let ctx = TestContext::new(client); |
| 158 | 158 | ||
| 159 | // Request repository fixture - behavior depends on mode | 159 | // Request repository fixture - behavior depends on mode |
| 160 | let event = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 160 | let event = ctx |
| 161 | format!( | 161 | .get_fixture(FixtureKind::ValidRepoServed) |
| 162 | "Test setup failed: could not get valid repository fixture: {}", | 162 | .await |
| 163 | e | 163 | .map_err(|e| { |
| 164 | ) | 164 | format!( |
| 165 | })?; | 165 | "Test setup failed: could not get valid repository fixture: {}", |
| 166 | e | ||
| 167 | ) | ||
| 168 | })?; | ||
| 166 | 169 | ||
| 167 | // Get relay URL for validation | 170 | // Get relay URL for validation |
| 168 | let relay_url = client | 171 | let relay_url = client |
| @@ -602,12 +605,15 @@ impl EventAcceptancePolicyTests { | |||
| 602 | let ctx = TestContext::new(client); | 605 | let ctx = TestContext::new(client); |
| 603 | 606 | ||
| 604 | // NEW: Get repository fixture (mode-aware) | 607 | // NEW: Get repository fixture (mode-aware) |
| 605 | let repo = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 608 | let repo = ctx |
| 606 | format!( | 609 | .get_fixture(FixtureKind::ValidRepoServed) |
| 607 | "Test setup failed: could not get valid repository fixture: {}", | 610 | .await |
| 608 | e | 611 | .map_err(|e| { |
| 609 | ) | 612 | format!( |
| 610 | })?; | 613 | "Test setup failed: could not get valid repository fixture: {}", |
| 614 | e | ||
| 615 | ) | ||
| 616 | })?; | ||
| 611 | 617 | ||
| 612 | // 2. Create issue that references the repo | 618 | // 2. Create issue that references the repo |
| 613 | let issue = Self::create_issue_for_repo(client, &repo, "Test Issue 1")?; | 619 | let issue = Self::create_issue_for_repo(client, &repo, "Test Issue 1")?; |
| @@ -637,12 +643,15 @@ impl EventAcceptancePolicyTests { | |||
| 637 | let ctx = TestContext::new(client); | 643 | let ctx = TestContext::new(client); |
| 638 | 644 | ||
| 639 | // Get repository fixture (mode-aware) | 645 | // Get repository fixture (mode-aware) |
| 640 | let repo = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 646 | let repo = ctx |
| 641 | format!( | 647 | .get_fixture(FixtureKind::ValidRepoServed) |
| 642 | "Test setup failed: could not get valid repository fixture: {}", | 648 | .await |
| 643 | e | 649 | .map_err(|e| { |
| 644 | ) | 650 | format!( |
| 645 | })?; | 651 | "Test setup failed: could not get valid repository fixture: {}", |
| 652 | e | ||
| 653 | ) | ||
| 654 | })?; | ||
| 646 | 655 | ||
| 647 | // Extract repo_id and create `A` tag manually | 656 | // Extract repo_id and create `A` tag manually |
| 648 | let repo_id = | 657 | let repo_id = |
| @@ -690,12 +699,15 @@ impl EventAcceptancePolicyTests { | |||
| 690 | let ctx = TestContext::new(client); | 699 | let ctx = TestContext::new(client); |
| 691 | 700 | ||
| 692 | // Get repository fixture (mode-aware) | 701 | // Get repository fixture (mode-aware) |
| 693 | let repo = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 702 | let repo = ctx |
| 694 | format!( | 703 | .get_fixture(FixtureKind::ValidRepoServed) |
| 695 | "Test setup failed: could not get valid repository fixture: {}", | 704 | .await |
| 696 | e | 705 | .map_err(|e| { |
| 697 | ) | 706 | format!( |
| 698 | })?; | 707 | "Test setup failed: could not get valid repository fixture: {}", |
| 708 | e | ||
| 709 | ) | ||
| 710 | })?; | ||
| 699 | 711 | ||
| 700 | // Extract repo_id and create `q` tag | 712 | // Extract repo_id and create `q` tag |
| 701 | let repo_id = | 713 | let repo_id = |
| @@ -825,12 +837,15 @@ impl EventAcceptancePolicyTests { | |||
| 825 | let ctx = TestContext::new(client); | 837 | let ctx = TestContext::new(client); |
| 826 | 838 | ||
| 827 | // Get repository fixture (mode-aware) | 839 | // Get repository fixture (mode-aware) |
| 828 | let repo = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 840 | let repo = ctx |
| 829 | format!( | 841 | .get_fixture(FixtureKind::ValidRepoServed) |
| 830 | "Test setup failed: could not get valid repository fixture: {}", | 842 | .await |
| 831 | e | 843 | .map_err(|e| { |
| 832 | ) | 844 | format!( |
| 833 | })?; | 845 | "Test setup failed: could not get valid repository fixture: {}", |
| 846 | e | ||
| 847 | ) | ||
| 848 | })?; | ||
| 834 | 849 | ||
| 835 | // Create Kind 1 A that quotes the repo (makes it accepted) | 850 | // Create Kind 1 A that quotes the repo (makes it accepted) |
| 836 | let repo_id = Self::extract_d_tag(&repo).ok_or("Failed to extract repo_id")?; | 851 | let repo_id = Self::extract_d_tag(&repo).ok_or("Failed to extract repo_id")?; |
| @@ -881,12 +896,15 @@ impl EventAcceptancePolicyTests { | |||
| 881 | let ctx = TestContext::new(client); | 896 | let ctx = TestContext::new(client); |
| 882 | 897 | ||
| 883 | // Get repository fixture (mode-aware) | 898 | // Get repository fixture (mode-aware) |
| 884 | let repo = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 899 | let repo = ctx |
| 885 | format!( | 900 | .get_fixture(FixtureKind::ValidRepoServed) |
| 886 | "Test setup failed: could not get valid repository fixture: {}", | 901 | .await |
| 887 | e | 902 | .map_err(|e| { |
| 888 | ) | 903 | format!( |
| 889 | })?; | 904 | "Test setup failed: could not get valid repository fixture: {}", |
| 905 | e | ||
| 906 | ) | ||
| 907 | })?; | ||
| 890 | 908 | ||
| 891 | // Verify repo is queryable (ensures it's fully indexed before we reference it) | 909 | // Verify repo is queryable (ensures it's fully indexed before we reference it) |
| 892 | let repo_id = Self::extract_d_tag(&repo).ok_or("Failed to extract repo_id")?; | 910 | let repo_id = Self::extract_d_tag(&repo).ok_or("Failed to extract repo_id")?; |
| @@ -1034,12 +1052,15 @@ impl EventAcceptancePolicyTests { | |||
| 1034 | let ctx = TestContext::new(client); | 1052 | let ctx = TestContext::new(client); |
| 1035 | 1053 | ||
| 1036 | // Get repository fixture (mode-aware) | 1054 | // Get repository fixture (mode-aware) |
| 1037 | let repo = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 1055 | let repo = ctx |
| 1038 | format!( | 1056 | .get_fixture(FixtureKind::ValidRepoServed) |
| 1039 | "Test setup failed: could not get valid repository fixture: {}", | 1057 | .await |
| 1040 | e | 1058 | .map_err(|e| { |
| 1041 | ) | 1059 | format!( |
| 1042 | })?; | 1060 | "Test setup failed: could not get valid repository fixture: {}", |
| 1061 | e | ||
| 1062 | ) | ||
| 1063 | })?; | ||
| 1043 | 1064 | ||
| 1044 | // Create Kind 1 A locally but DON'T send it yet | 1065 | // Create Kind 1 A locally but DON'T send it yet |
| 1045 | let kind1_a = client | 1066 | let kind1_a = client |
| @@ -1148,12 +1169,15 @@ impl EventAcceptancePolicyTests { | |||
| 1148 | let ctx = TestContext::new(client); | 1169 | let ctx = TestContext::new(client); |
| 1149 | 1170 | ||
| 1150 | // Get accepted repo A fixture (mode-aware) | 1171 | // Get accepted repo A fixture (mode-aware) |
| 1151 | let _repo_a = ctx.get_fixture(FixtureKind::ValidRepo).await.map_err(|e| { | 1172 | let _repo_a = ctx |
| 1152 | format!( | 1173 | .get_fixture(FixtureKind::ValidRepoServed) |
| 1153 | "Test setup failed: could not get valid repository fixture: {}", | 1174 | .await |
| 1154 | e | 1175 | .map_err(|e| { |
| 1155 | ) | 1176 | format!( |
| 1156 | })?; | 1177 | "Test setup failed: could not get valid repository fixture: {}", |
| 1178 | e | ||
| 1179 | ) | ||
| 1180 | })?; | ||
| 1157 | 1181 | ||
| 1158 | // Create Repo B but DON'T send it (unaccepted) | 1182 | // Create Repo B but DON'T send it (unaccepted) |
| 1159 | let repo_b = Self::create_test_repo(client, "unaccepted-repo-b").await?; | 1183 | let repo_b = Self::create_test_repo(client, "unaccepted-repo-b").await?; |
diff --git a/grasp-audit/src/specs/grasp01/git_clone.rs b/grasp-audit/src/specs/grasp01/git_clone.rs index fda472b..0c223f4 100644 --- a/grasp-audit/src/specs/grasp01/git_clone.rs +++ b/grasp-audit/src/specs/grasp01/git_clone.rs | |||
| @@ -49,7 +49,7 @@ impl GitCloneTests { | |||
| 49 | let ctx = TestContext::new(client); | 49 | let ctx = TestContext::new(client); |
| 50 | 50 | ||
| 51 | // Create repository announcement | 51 | // Create repository announcement |
| 52 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 52 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 53 | Ok(r) => r, | 53 | Ok(r) => r, |
| 54 | Err(e) => { | 54 | Err(e) => { |
| 55 | return TestResult::new( | 55 | return TestResult::new( |
| @@ -171,7 +171,7 @@ impl GitCloneTests { | |||
| 171 | let ctx = TestContext::new(client); | 171 | let ctx = TestContext::new(client); |
| 172 | 172 | ||
| 173 | // Create repository announcement | 173 | // Create repository announcement |
| 174 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 174 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 175 | Ok(r) => r, | 175 | Ok(r) => r, |
| 176 | Err(e) => { | 176 | Err(e) => { |
| 177 | return TestResult::new( | 177 | return TestResult::new( |
| @@ -274,7 +274,7 @@ impl GitCloneTests { | |||
| 274 | let ctx = TestContext::new(client); | 274 | let ctx = TestContext::new(client); |
| 275 | 275 | ||
| 276 | // Create repository announcement | 276 | // Create repository announcement |
| 277 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 277 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 278 | Ok(r) => r, | 278 | Ok(r) => r, |
| 279 | Err(e) => { | 279 | Err(e) => { |
| 280 | return TestResult::new( | 280 | return TestResult::new( |
diff --git a/grasp-audit/src/specs/grasp01/git_filter.rs b/grasp-audit/src/specs/grasp01/git_filter.rs index 7f203a2..31d86aa 100644 --- a/grasp-audit/src/specs/grasp01/git_filter.rs +++ b/grasp-audit/src/specs/grasp01/git_filter.rs | |||
| @@ -62,7 +62,7 @@ impl GitFilterTests { | |||
| 62 | let ctx = TestContext::new(client); | 62 | let ctx = TestContext::new(client); |
| 63 | 63 | ||
| 64 | // Create repository announcement | 64 | // Create repository announcement |
| 65 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 65 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 66 | Ok(r) => r, | 66 | Ok(r) => r, |
| 67 | Err(e) => { | 67 | Err(e) => { |
| 68 | return TestResult::new( | 68 | return TestResult::new( |
| @@ -185,7 +185,7 @@ impl GitFilterTests { | |||
| 185 | let ctx = TestContext::new(client); | 185 | let ctx = TestContext::new(client); |
| 186 | 186 | ||
| 187 | // Create repository announcement | 187 | // Create repository announcement |
| 188 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 188 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 189 | Ok(r) => r, | 189 | Ok(r) => r, |
| 190 | Err(e) => { | 190 | Err(e) => { |
| 191 | return TestResult::new( | 191 | return TestResult::new( |
| @@ -296,7 +296,7 @@ impl GitFilterTests { | |||
| 296 | let ctx = TestContext::new(client); | 296 | let ctx = TestContext::new(client); |
| 297 | 297 | ||
| 298 | // Create repository announcement | 298 | // Create repository announcement |
| 299 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 299 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 300 | Ok(r) => r, | 300 | Ok(r) => r, |
| 301 | Err(e) => { | 301 | Err(e) => { |
| 302 | return TestResult::new( | 302 | return TestResult::new( |
diff --git a/grasp-audit/src/specs/grasp01/nip01_smoke.rs b/grasp-audit/src/specs/grasp01/nip01_smoke.rs index 5976252..8cb4166 100644 --- a/grasp-audit/src/specs/grasp01/nip01_smoke.rs +++ b/grasp-audit/src/specs/grasp01/nip01_smoke.rs | |||
| @@ -69,7 +69,7 @@ impl Nip01SmokeTests { | |||
| 69 | // Step 1: GENERATE - Create TestContext and get ValidRepo fixture | 69 | // Step 1: GENERATE - Create TestContext and get ValidRepo fixture |
| 70 | let ctx = TestContext::new(client); | 70 | let ctx = TestContext::new(client); |
| 71 | let event = ctx | 71 | let event = ctx |
| 72 | .get_fixture(FixtureKind::ValidRepo) | 72 | .get_fixture(FixtureKind::ValidRepoSent) |
| 73 | .await | 73 | .await |
| 74 | .map_err(|e| format!("Failed to create ValidRepo fixture: {}", e))?; | 74 | .map_err(|e| format!("Failed to create ValidRepo fixture: {}", e))?; |
| 75 | 75 | ||
| @@ -135,7 +135,7 @@ impl Nip01SmokeTests { | |||
| 135 | // Step 1: GENERATE - Create TestContext and get ValidRepo fixture | 135 | // Step 1: GENERATE - Create TestContext and get ValidRepo fixture |
| 136 | let ctx = TestContext::new(client); | 136 | let ctx = TestContext::new(client); |
| 137 | let _event = ctx | 137 | let _event = ctx |
| 138 | .get_fixture(FixtureKind::ValidRepo) | 138 | .get_fixture(FixtureKind::ValidRepoSent) |
| 139 | .await | 139 | .await |
| 140 | .map_err(|e| format!("Failed to create ValidRepo fixture: {}", e))?; | 140 | .map_err(|e| format!("Failed to create ValidRepo fixture: {}", e))?; |
| 141 | 141 | ||
diff --git a/grasp-audit/src/specs/grasp01/push_authorization.rs b/grasp-audit/src/specs/grasp01/push_authorization.rs index be354a0..78ef471 100644 --- a/grasp-audit/src/specs/grasp01/push_authorization.rs +++ b/grasp-audit/src/specs/grasp01/push_authorization.rs | |||
| @@ -208,7 +208,7 @@ async fn setup_pr_test_repo( | |||
| 208 | ) -> Result<(PathBuf, String, String, String), String> { | 208 | ) -> Result<(PathBuf, String, String, String), String> { |
| 209 | // Get fixtures | 209 | // Get fixtures |
| 210 | let repo_event = ctx | 210 | let repo_event = ctx |
| 211 | .get_fixture(FixtureKind::ValidRepo) | 211 | .get_fixture(FixtureKind::ValidRepoSent) |
| 212 | .await | 212 | .await |
| 213 | .map_err(|e| format!("Failed to get repo announcement: {}", e))?; | 213 | .map_err(|e| format!("Failed to get repo announcement: {}", e))?; |
| 214 | 214 | ||
| @@ -407,7 +407,7 @@ impl PushAuthorizationTests { | |||
| 407 | let ctx = TestContext::new(client); | 407 | let ctx = TestContext::new(client); |
| 408 | 408 | ||
| 409 | // Create repository (no state event) | 409 | // Create repository (no state event) |
| 410 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 410 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 411 | Ok(r) => r, | 411 | Ok(r) => r, |
| 412 | Err(e) => { | 412 | Err(e) => { |
| 413 | return TestResult::new( | 413 | return TestResult::new( |
| @@ -956,7 +956,7 @@ impl PushAuthorizationTests { | |||
| 956 | // ============================================================ | 956 | // ============================================================ |
| 957 | let ctx = TestContext::new(client); | 957 | let ctx = TestContext::new(client); |
| 958 | 958 | ||
| 959 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 959 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 960 | Ok(r) => r, | 960 | Ok(r) => r, |
| 961 | Err(e) => { | 961 | Err(e) => { |
| 962 | return TestResult::new( | 962 | return TestResult::new( |
| @@ -1110,7 +1110,7 @@ impl PushAuthorizationTests { | |||
| 1110 | let pr_event_id = pr_event.id.to_hex(); | 1110 | let pr_event_id = pr_event.id.to_hex(); |
| 1111 | 1111 | ||
| 1112 | // Get repo info for cloning (fresh clone for verification) | 1112 | // Get repo info for cloning (fresh clone for verification) |
| 1113 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 1113 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 1114 | Ok(r) => r, | 1114 | Ok(r) => r, |
| 1115 | Err(e) => { | 1115 | Err(e) => { |
| 1116 | return TestResult::new(test_name, SpecRef::GitAcceptRefsNostrEventId, desc) | 1116 | return TestResult::new(test_name, SpecRef::GitAcceptRefsNostrEventId, desc) |
| @@ -1198,7 +1198,7 @@ impl PushAuthorizationTests { | |||
| 1198 | let pr_event_id = pr_event.id.to_hex(); | 1198 | let pr_event_id = pr_event.id.to_hex(); |
| 1199 | 1199 | ||
| 1200 | // Get repo info for cloning (fresh clone for this test) | 1200 | // Get repo info for cloning (fresh clone for this test) |
| 1201 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 1201 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 1202 | Ok(r) => r, | 1202 | Ok(r) => r, |
| 1203 | Err(e) => { | 1203 | Err(e) => { |
| 1204 | return TestResult::new(test_name, SpecRef::GitAcceptRefsNostrEventId, desc) | 1204 | return TestResult::new(test_name, SpecRef::GitAcceptRefsNostrEventId, desc) |
| @@ -1289,7 +1289,7 @@ impl PushAuthorizationTests { | |||
| 1289 | let pr_event_id = pr_event.id.to_hex(); | 1289 | let pr_event_id = pr_event.id.to_hex(); |
| 1290 | 1290 | ||
| 1291 | // Get repo info for cloning (fresh clone for this test) | 1291 | // Get repo info for cloning (fresh clone for this test) |
| 1292 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 1292 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 1293 | Ok(r) => r, | 1293 | Ok(r) => r, |
| 1294 | Err(e) => { | 1294 | Err(e) => { |
| 1295 | return TestResult::new(test_name, SpecRef::GitAcceptRefsNostrEventId, desc) | 1295 | return TestResult::new(test_name, SpecRef::GitAcceptRefsNostrEventId, desc) |
| @@ -1425,7 +1425,7 @@ impl PushAuthorizationTests { | |||
| 1425 | // ============================================================ | 1425 | // ============================================================ |
| 1426 | // Step 2: Extract repo_id and owner npub from ValidRepo (cached by fixture) | 1426 | // Step 2: Extract repo_id and owner npub from ValidRepo (cached by fixture) |
| 1427 | // ============================================================ | 1427 | // ============================================================ |
| 1428 | let valid_repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 1428 | let valid_repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 1429 | Ok(e) => e, | 1429 | Ok(e) => e, |
| 1430 | Err(e) => { | 1430 | Err(e) => { |
| 1431 | return TestResult::new(test_name, SpecRef::GitSetHeadOnReceive, desc) | 1431 | return TestResult::new(test_name, SpecRef::GitSetHeadOnReceive, desc) |
| @@ -1528,7 +1528,7 @@ impl PushAuthorizationTests { | |||
| 1528 | // ============================================================ | 1528 | // ============================================================ |
| 1529 | // Step 2: Extract repo_id and owner npub from ValidRepo (cached by fixture) | 1529 | // Step 2: Extract repo_id and owner npub from ValidRepo (cached by fixture) |
| 1530 | // ============================================================ | 1530 | // ============================================================ |
| 1531 | let valid_repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 1531 | let valid_repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 1532 | Ok(e) => e, | 1532 | Ok(e) => e, |
| 1533 | Err(e) => { | 1533 | Err(e) => { |
| 1534 | return TestResult::new(test_name, SpecRef::GitSetHeadOnReceive, desc) | 1534 | return TestResult::new(test_name, SpecRef::GitSetHeadOnReceive, desc) |
diff --git a/grasp-audit/src/specs/grasp01/repository_creation.rs b/grasp-audit/src/specs/grasp01/repository_creation.rs index a702afe..5730f1c 100644 --- a/grasp-audit/src/specs/grasp01/repository_creation.rs +++ b/grasp-audit/src/specs/grasp01/repository_creation.rs | |||
| @@ -51,7 +51,7 @@ impl RepositoryCreationTests { | |||
| 51 | let ctx = TestContext::new(client); | 51 | let ctx = TestContext::new(client); |
| 52 | 52 | ||
| 53 | // Use TestContext to create and send repository announcement | 53 | // Use TestContext to create and send repository announcement |
| 54 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 54 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 55 | Ok(r) => r, | 55 | Ok(r) => r, |
| 56 | Err(e) => { | 56 | Err(e) => { |
| 57 | return TestResult::new( | 57 | return TestResult::new( |
| @@ -131,7 +131,7 @@ impl RepositoryCreationTests { | |||
| 131 | let ctx = TestContext::new(client); | 131 | let ctx = TestContext::new(client); |
| 132 | 132 | ||
| 133 | // Create a repository announcement | 133 | // Create a repository announcement |
| 134 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 134 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 135 | Ok(r) => r, | 135 | Ok(r) => r, |
| 136 | Err(e) => { | 136 | Err(e) => { |
| 137 | return TestResult::new( | 137 | return TestResult::new( |
| @@ -210,7 +210,7 @@ impl RepositoryCreationTests { | |||
| 210 | 210 | ||
| 211 | let ctx = TestContext::new(client); | 211 | let ctx = TestContext::new(client); |
| 212 | 212 | ||
| 213 | let repo = match ctx.get_fixture(FixtureKind::ValidRepo).await { | 213 | let repo = match ctx.get_fixture(FixtureKind::ValidRepoSent).await { |
| 214 | Ok(r) => r, | 214 | Ok(r) => r, |
| 215 | Err(e) => { | 215 | Err(e) => { |
| 216 | return TestResult::new( | 216 | return TestResult::new( |