upleb.uk

Public git repos — served from a NIP-34 GRASP relay at git.upleb.uk

summaryrefslogtreecommitdiff
path: root/grasp-audit/src/specs
diff options
context:
space:
mode:
authorDanConwayDev <DanConwayDev@protonmail.com>2026-01-08 11:20:35 +0000
committerDanConwayDev <DanConwayDev@protonmail.com>2026-01-08 11:23:28 +0000
commit5d02ad6b893f9059044914c115d77cf9d8e589c3 (patch)
treeb727f9c44d2f2d4e203dc2344e4c9bd5144a77dd /grasp-audit/src/specs
parent075307804bf66bba10f5bc55cb40e2e6a98a65ee (diff)
refactor: replace hardcoded Kind constants with rust-nostr variants
- Replace KIND_REPOSITORY_ANNOUNCEMENT with Kind::GitRepoAnnouncement - Replace KIND_REPOSITORY_STATE with Kind::RepoState - Replace KIND_PR with Kind::GitPullRequest - Replace KIND_PR_UPDATE with Kind::GitPullRequestUpdate - Replace KIND_USER_GRASP_LIST with Kind::GitUserGraspList - Replace KIND_PATCH with Kind::GitPatch - Replace KIND_ISSUE with Kind::GitIssue - Replace KIND_COMMENT with Kind::Comment - Replace all Kind::Custom(30617|30618|1617|1618|1619|1621|1111|10317) patterns - Remove all hardcoded KIND_* constants from events.rs - Update all match statements to use Kind enum directly - Update all filter builders to use Kind variants - Update all test helpers and assertions Benefits: - Type safety: compiler prevents wrong kind numbers - Readability: Kind::GitRepoAnnouncement is self-documenting - Maintainability: single source of truth (rust-nostr) - IDE support: full autocompletion and refactoring - Standards: aligns with rust-nostr best practices Files modified: 21 Constants removed: 9 Patterns replaced: 100+ Tests passing: 222/222
Diffstat (limited to 'grasp-audit/src/specs')
-rw-r--r--grasp-audit/src/specs/grasp01/event_acceptance_policy.rs8
-rw-r--r--grasp-audit/src/specs/grasp01/nip01_smoke.rs6
-rw-r--r--grasp-audit/src/specs/grasp01/push_authorization.rs4
3 files changed, 9 insertions, 9 deletions
diff --git a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs
index 3db1446..5b697d8 100644
--- a/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs
+++ b/grasp-audit/src/specs/grasp01/event_acceptance_policy.rs
@@ -659,7 +659,7 @@ impl EventAcceptancePolicyTests {
659 ]; 659 ];
660 660
661 let comment = client 661 let comment = client
662 .event_builder(Kind::Custom(1111), "Comment on repo") 662 .event_builder(Kind::Comment, "Comment on repo")
663 .tags(tags) 663 .tags(tags)
664 .build(client.keys()) 664 .build(client.keys())
665 .map_err(|e| format!("Failed to build comment: {}", e))?; 665 .map_err(|e| format!("Failed to build comment: {}", e))?;
@@ -935,7 +935,7 @@ impl EventAcceptancePolicyTests {
935 ]; 935 ];
936 936
937 let issue = client 937 let issue = client
938 .event_builder(Kind::Custom(1621), "issue content") 938 .event_builder(Kind::GitIssue, "issue content")
939 .tags(issue_tags) 939 .tags(issue_tags)
940 .build(client.keys()) 940 .build(client.keys())
941 .map_err(|e| format!("Failed to build issue: {}", e))?; 941 .map_err(|e| format!("Failed to build issue: {}", e))?;
@@ -995,7 +995,7 @@ impl EventAcceptancePolicyTests {
995 ]; 995 ];
996 996
997 let comment_b = client 997 let comment_b = client
998 .event_builder(Kind::Custom(1111), "Comment B quoting Comment A") 998 .event_builder(Kind::Comment, "Comment B quoting Comment A")
999 .tags(comment_b_tags) 999 .tags(comment_b_tags)
1000 .build(client.keys()) 1000 .build(client.keys())
1001 .map_err(|e| format!("Failed to build comment B: {}", e))?; 1001 .map_err(|e| format!("Failed to build comment B: {}", e))?;
@@ -1172,7 +1172,7 @@ impl EventAcceptancePolicyTests {
1172 ]; 1172 ];
1173 1173
1174 let comment = client 1174 let comment = client
1175 .event_builder(Kind::Custom(1111), "Comment on unaccepted repo") 1175 .event_builder(Kind::Comment, "Comment on unaccepted repo")
1176 .tags(tags) 1176 .tags(tags)
1177 .build(client.keys()) 1177 .build(client.keys())
1178 .map_err(|e| format!("Failed to build comment: {}", e))?; 1178 .map_err(|e| format!("Failed to build comment: {}", e))?;
diff --git a/grasp-audit/src/specs/grasp01/nip01_smoke.rs b/grasp-audit/src/specs/grasp01/nip01_smoke.rs
index bd45ea4..4d0b8a4 100644
--- a/grasp-audit/src/specs/grasp01/nip01_smoke.rs
+++ b/grasp-audit/src/specs/grasp01/nip01_smoke.rs
@@ -78,7 +78,7 @@ impl Nip01SmokeTests {
78 tokio::time::sleep(std::time::Duration::from_millis(100)).await; 78 tokio::time::sleep(std::time::Duration::from_millis(100)).await;
79 79
80 // Step 2: VERIFY - Query event back 80 // Step 2: VERIFY - Query event back
81 let filter = Filter::new().kind(Kind::Custom(30617)).id(event_id); 81 let filter = Filter::new().kind(Kind::GitRepoAnnouncement).id(event_id);
82 82
83 let events = client 83 let events = client
84 .query(filter) 84 .query(filter)
@@ -88,7 +88,7 @@ impl Nip01SmokeTests {
88 if events.is_empty() { 88 if events.is_empty() {
89 // Debug: try querying without audit client filtering 89 // Debug: try querying without audit client filtering
90 eprintln!("Event not found with audit client query, trying direct client query..."); 90 eprintln!("Event not found with audit client query, trying direct client query...");
91 let direct_filter = Filter::new().kind(Kind::Custom(30617)).id(event_id); 91 let direct_filter = Filter::new().kind(Kind::GitRepoAnnouncement).id(event_id);
92 let direct_events = client 92 let direct_events = client
93 .client() 93 .client()
94 .fetch_events(direct_filter, std::time::Duration::from_secs(5)) 94 .fetch_events(direct_filter, std::time::Duration::from_secs(5))
@@ -140,7 +140,7 @@ impl Nip01SmokeTests {
140 140
141 // Step 2: VERIFY - Subscribe to NIP-34 announcements from this author 141 // Step 2: VERIFY - Subscribe to NIP-34 announcements from this author
142 let filter = Filter::new() 142 let filter = Filter::new()
143 .kind(Kind::Custom(30617)) 143 .kind(Kind::GitRepoAnnouncement)
144 .author(client.public_key()); 144 .author(client.public_key());
145 145
146 let events = client 146 let events = client
diff --git a/grasp-audit/src/specs/grasp01/push_authorization.rs b/grasp-audit/src/specs/grasp01/push_authorization.rs
index 23eb735..4c7720b 100644
--- a/grasp-audit/src/specs/grasp01/push_authorization.rs
+++ b/grasp-audit/src/specs/grasp01/push_authorization.rs
@@ -877,7 +877,7 @@ impl PushAuthorizationTests {
877 // Create a rogue state event announcing the new commit 877 // Create a rogue state event announcing the new commit
878 // This event has the correct repo_id but is signed by a non-maintainer 878 // This event has the correct repo_id but is signed by a non-maintainer
879 let rogue_state = match client 879 let rogue_state = match client
880 .event_builder(Kind::Custom(30618), "") 880 .event_builder(Kind::RepoState, "")
881 .tag(Tag::identifier(&repo_id)) 881 .tag(Tag::identifier(&repo_id))
882 .tag(Tag::custom( 882 .tag(Tag::custom(
883 TagKind::custom("refs/heads/main"), 883 TagKind::custom("refs/heads/main"),
@@ -1591,7 +1591,7 @@ impl PushAuthorizationTests {
1591 // This references a commit that doesn't yet exist on the relay 1591 // This references a commit that doesn't yet exist on the relay
1592 // ============================================================ 1592 // ============================================================
1593 let state_event = match client 1593 let state_event = match client
1594 .event_builder(Kind::Custom(30618), "") 1594 .event_builder(Kind::RepoState, "")
1595 .tag(Tag::identifier(&repo_id)) 1595 .tag(Tag::identifier(&repo_id))
1596 .tag(Tag::custom( 1596 .tag(Tag::custom(
1597 TagKind::custom("HEAD"), 1597 TagKind::custom("HEAD"),