From d07dc0e3b14b8464e47f5ab009552eacda568a36 Mon Sep 17 00:00:00 2001 From: DanConwayDev Date: Thu, 12 Feb 2026 11:23:48 +0000 Subject: fix: use consistent git identity for PR test commit hash The PR_TEST_COMMIT_HASH constant was incorrect because the discovery test used a different git identity (pr-test@example.com) than the actual create_pr_test_commit function (test@grasp-audit.local from fixtures.rs). This caused the same commit content to produce different hashes due to different author/committer info being embedded in the commit object. Fixed by updating the discovery test to use the same git identity as clone_repo() in fixtures.rs, ensuring consistent commit hashes. --- grasp-audit/src/specs/grasp01/push_authorization.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/grasp-audit/src/specs/grasp01/push_authorization.rs b/grasp-audit/src/specs/grasp01/push_authorization.rs index c1003b9..677af89 100644 --- a/grasp-audit/src/specs/grasp01/push_authorization.rs +++ b/grasp-audit/src/specs/grasp01/push_authorization.rs @@ -1701,16 +1701,16 @@ mod tests { String::from_utf8_lossy(&output.stderr) ); - // Configure git user - use PR Test Author identity + // Configure git user - use same identity as clone_repo in fixtures.rs let output = Command::new("git") - .args(["config", "user.email", "pr-test@example.com"]) + .args(["config", "user.email", "test@grasp-audit.local"]) .current_dir(path) .output() .expect("git config email failed"); assert!(output.status.success(), "git config email failed"); let output = Command::new("git") - .args(["config", "user.name", "PR Test Author"]) + .args(["config", "user.name", "GRASP Audit Test"]) .current_dir(path) .output() .expect("git config name failed"); -- cgit v1.2.3