refactor: move git test helpers to auths-test-utils, break dev-dep cycle

auths-id had a dev-dependency on auths-infra-git solely for init_test_repo(),
creating a cycle: auths-id → auths-infra-git → auths-sdk → auths-id.
Move git test helpers to auths-test-utils and fix crate publish order.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: bordumb

Cargo.lock

@@ -55,7 +55,7 @@ auths-infra-http = { workspace = true, optional = true }
 
 [dev-dependencies]
 auths-core = { workspace = true, features = ["test-utils"] }
-auths-infra-git = { workspace = true, features = ["test-utils"] }
+auths-test-utils = { path = "../auths-test-utils" }
 criterion = { version = "0.8.2", features = ["html_reports"] }
 rand = "0.10.0"
 base58 = "0.2"

crates/auths-id/Cargo.toml

@@ -29,7 +29,7 @@ fn make_test_attestation(issuer: &str, subject: &str) -> TestAttestation {
 /// Tests the full KERI lifecycle: inception -> rotation -> rotation -> resolution
 #[test]
 fn full_keri_lifecycle() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     // === Phase 1: Inception ===
     let init: InceptionResult = create_keri_identity(&repo, None).unwrap();
@@ -97,7 +97,7 @@ fn full_keri_lifecycle() {
 /// Tests device attestation anchoring via IXN events
 #[test]
 fn device_enrollment_with_anchoring() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     // Create identity
     let init = create_keri_identity(&repo, None).unwrap();
@@ -130,7 +130,7 @@ fn device_enrollment_with_anchoring() {
 /// Tests that multiple attestations can be anchored
 #[test]
 fn multiple_device_attestations() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let identity_did = format!("did:keri:{}", init.prefix);
@@ -172,7 +172,7 @@ fn multiple_device_attestations() {
 /// Tests that rotation invalidates without proper commitment
 #[test]
 fn rotation_requires_commitment() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
 
@@ -186,7 +186,7 @@ fn rotation_requires_commitment() {
 /// Tests KEL validation detects sequence tampering
 #[test]
 fn kel_validation_rejects_sequence_tampering() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let _rot = rotate_keys(&repo, &init.prefix, &init.next_keypair_pkcs8, None).unwrap();
@@ -208,7 +208,7 @@ fn kel_validation_rejects_sequence_tampering() {
 /// Tests that unanchored attestations are not found
 #[test]
 fn unanchored_attestation_not_found() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let identity_did = format!("did:keri:{}", init.prefix);
@@ -224,7 +224,7 @@ fn unanchored_attestation_not_found() {
 /// Tests that key state is correct after operations
 #[test]
 fn key_state_reflects_operations() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
 
@@ -244,7 +244,7 @@ fn key_state_reflects_operations() {
 /// Tests that did:keri parsing works correctly
 #[test]
 fn did_keri_parsing() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let did = format!("did:keri:{}", init.prefix);
@@ -261,7 +261,7 @@ fn did_keri_parsing() {
 /// Tests anchor verification by digest
 #[test]
 fn verify_anchor_by_digest_works() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let identity_did = format!("did:keri:{}", init.prefix);

crates/auths-id/tests/cases/keri.rs

@@ -171,7 +171,7 @@ fn rotate_identity(
 #[tokio::test(flavor = "multi_thread")]
 async fn test_full_identity_lifecycle() {
     let kc = IsolatedKeychainHandle::new();
-    let (_dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, _repo) = auths_test_utils::git::init_test_repo();
     let repo_path = _dir.path().to_path_buf();
     let passphrase = "Test-P@ss12345";
 
@@ -249,7 +249,7 @@ async fn test_full_identity_lifecycle() {
 #[tokio::test(flavor = "multi_thread")]
 async fn test_attestation_chain_after_rotation() {
     let kc = IsolatedKeychainHandle::new();
-    let (_dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, _repo) = auths_test_utils::git::init_test_repo();
     let repo_path = _dir.path().to_path_buf();
     let passphrase = "Test-P@ss12345";
 
@@ -308,7 +308,7 @@ async fn test_attestation_chain_after_rotation() {
 #[tokio::test(flavor = "multi_thread")]
 async fn test_verify_device_authorization_lifecycle() {
     let kc = IsolatedKeychainHandle::new();
-    let (_dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, _repo) = auths_test_utils::git::init_test_repo();
     let repo_path = _dir.path().to_path_buf();
     let passphrase = "Test-P@ss12345";
 
@@ -364,7 +364,7 @@ async fn test_verify_device_authorization_lifecycle() {
 #[tokio::test(flavor = "multi_thread")]
 async fn test_multiple_rotations_maintain_verification() {
     let kc = IsolatedKeychainHandle::new();
-    let (_dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, _repo) = auths_test_utils::git::init_test_repo();
     let repo_path = _dir.path().to_path_buf();
     let passphrase = "Test-P@ss12345";
 
@@ -437,7 +437,7 @@ async fn test_multiple_rotations_maintain_verification() {
 #[test]
 fn test_init_creates_keri_kel() {
     let kc = IsolatedKeychainHandle::new();
-    let (_dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, _repo) = auths_test_utils::git::init_test_repo();
     let repo_path = _dir.path().to_path_buf();
     let passphrase = "Test-P@ss12345";
 
@@ -469,7 +469,7 @@ fn test_init_creates_keri_kel() {
 #[test]
 fn test_rotation_appends_to_kel() {
     let kc = IsolatedKeychainHandle::new();
-    let (_dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, _repo) = auths_test_utils::git::init_test_repo();
     let repo_path = _dir.path().to_path_buf();
     let passphrase = "Test-P@ss12345";
 

crates/auths-id/tests/cases/lifecycle.rs

@@ -92,7 +92,7 @@ fn double_rotation_does_not_corrupt_kel() {
 
 #[test]
 fn rotation_after_interaction_events_preserves_kel_integrity() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let identity_did = format!("did:keri:{}", init.prefix);
@@ -127,7 +127,7 @@ fn rotation_after_interaction_events_preserves_kel_integrity() {
 
 #[test]
 fn anchoring_works_with_rotated_key_after_ixn() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let identity_did = format!("did:keri:{}", init.prefix);
@@ -160,7 +160,7 @@ fn anchoring_works_with_rotated_key_after_ixn() {
 
 #[test]
 fn multiple_rotations_interleaved_with_ixn() {
-    let (_dir, repo) = auths_infra_git::testing::init_test_repo();
+    let (_dir, repo) = auths_test_utils::git::init_test_repo();
 
     let init = create_keri_identity(&repo, None).unwrap();
     let identity_did = format!("did:keri:{}", init.prefix);

crates/auths-id/tests/cases/rotation_edge_cases.rs

@@ -19,12 +19,7 @@ thiserror.workspace = true
 chrono = "0.4"
 log = "0.4"
 
-[features]
-test-utils = ["dep:tempfile"]
-
-[dependencies.tempfile]
-version = "3"
-optional = true
-
 [dev-dependencies]
 auths-sdk = { workspace = true, features = ["test-utils"] }
+auths-test-utils = { path = "../auths-test-utils" }
+tempfile = "3"

crates/auths-infra-git/Cargo.toml

@@ -23,7 +23,3 @@ pub use blob_store::GitBlobStore;
 pub use event_log::GitEventLog;
 pub use ref_store::GitRefStore;
 pub use repo::GitRepo;
-
-/// Test utilities for auths-infra-git consumers (behind `test-utils` feature).
-#[cfg(any(test, feature = "test-utils"))]
-pub mod testing;

crates/auths-infra-git/src/lib.rs

@@ -2,7 +2,7 @@ use auths_core::ports::storage::{BlobReader, BlobWriter, StorageError};
 use auths_infra_git::{GitBlobStore, GitRepo};
 
 fn setup() -> (tempfile::TempDir, GitRepo) {
-    let (dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (dir, _repo) = auths_test_utils::git::init_test_repo();
     let git_repo = GitRepo::open(dir.path()).unwrap();
     (dir, git_repo)
 }

crates/auths-infra-git/tests/cases/blob_store.rs

@@ -3,7 +3,7 @@ use auths_infra_git::{GitEventLog, GitRepo};
 use auths_verifier::keri::Prefix;
 
 fn setup() -> (tempfile::TempDir, GitRepo) {
-    let (dir, _repo) = auths_infra_git::testing::init_test_repo();
+    let (dir, _repo) = auths_test_utils::git::init_test_repo();
     let git_repo = GitRepo::open(dir.path()).unwrap();
     (dir, git_repo)
 }

crates/auths-infra-git/tests/cases/event_log.rs

@@ -47,7 +47,7 @@ auths_sdk::git_log_provider_contract_tests!(
 auths_sdk::git_log_provider_contract_tests!(
     git2_provider,
     {
-        let (dir, repo) = auths_infra_git::testing::init_test_repo();
+        let (dir, repo) = auths_test_utils::git::init_test_repo();
         create_commit(&repo, "first");
         create_commit(&repo, "second");
         create_commit(&repo, "third");