a00e6a5f
Replace dummy_commit/dummy_tree with valid 40-char hex OIDs in tests
a73x 2026-03-21 19:14
Use valid git OID format strings ("aaa...a" for commits, "bbb...b" for
trees) instead of invalid "dummy_commit"/"dummy_tree" literals across
all test and bench files.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
diff --git a/benches/core_ops.rs b/benches/core_ops.rs index 180e8f8..02e3fce 100644 --- a/benches/core_ops.rs +++ b/benches/core_ops.rs @@ -76,8 +76,8 @@ fn setup_patches(n: usize) -> (Repository, TempDir) { base_ref: "main".to_string(), branch: format!("feature-{}", i), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; @@ -217,8 +217,8 @@ fn bench_patch_from_ref(c: &mut Criterion) { base_ref: "main".to_string(), branch: format!("feature-bench-{}", count), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; diff --git a/tests/adversarial_test.rs b/tests/adversarial_test.rs index c996ea2..8b43512 100644 --- a/tests/adversarial_test.rs +++ b/tests/adversarial_test.rs @@ -542,13 +542,13 @@ fn arb_action() -> impl Strategy<Value = Action> { base_ref, branch, fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), } }), proptest::option::of(".*").prop_map(|body| Action::PatchRevision { commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), body, }), (".*",).prop_map(|(body,)| Action::PatchComment { body }), diff --git a/tests/collab_test.rs b/tests/collab_test.rs index 24bbeac..3f4d915 100644 --- a/tests/collab_test.rs +++ b/tests/collab_test.rs @@ -333,8 +333,8 @@ fn test_patch_review_workflow() { timestamp: now(), author: alice(), action: Action::PatchRevision { commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), body: Some("Updated implementation".to_string()), }, clock: 0, @@ -641,8 +641,8 @@ fn create_branch_patch( base_ref: base_ref.to_string(), branch: branch.to_string(), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; @@ -913,8 +913,8 @@ fn test_resolve_head_with_oid_string() { base_ref: "main".to_string(), branch: tip.to_string(), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; diff --git a/tests/crdt_test.rs b/tests/crdt_test.rs index fb39350..4046c94 100644 --- a/tests/crdt_test.rs +++ b/tests/crdt_test.rs @@ -361,8 +361,8 @@ fn concurrent_patch_close_merge_higher_clock_wins() { base_ref: "main".to_string(), branch: "test-branch".to_string(), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; diff --git a/tests/signing_test.rs b/tests/signing_test.rs index cc6b452..01bbc67 100644 --- a/tests/signing_test.rs +++ b/tests/signing_test.rs @@ -210,8 +210,8 @@ fn event_json_uses_namespaced_action_types() { base_ref: "main".to_string(), branch: "feature/fix-bug".to_string(), fixes: Some("deadbeef".to_string()), commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; diff --git a/tests/sort_test.rs b/tests/sort_test.rs index ca4f800..bd46aa5 100644 --- a/tests/sort_test.rs +++ b/tests/sort_test.rs @@ -87,8 +87,8 @@ fn create_patch_at( base_ref: "main".to_string(), branch: format!("branch-{}", title.replace(' ', "-")), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; diff --git a/tests/sync_test.rs b/tests/sync_test.rs index ced4c59..e398705 100644 --- a/tests/sync_test.rs +++ b/tests/sync_test.rs @@ -291,8 +291,8 @@ fn test_patch_review_across_repos() { base_ref: "main".to_string(), branch: "feature/x".to_string(), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; @@ -343,8 +343,8 @@ fn test_concurrent_review_and_revise() { base_ref: "main".to_string(), branch: "feature/wip".to_string(), fixes: None, commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), }, clock: 0, }; @@ -362,8 +362,8 @@ fn test_concurrent_review_and_revise() { timestamp: now(), author: alice(), action: Action::PatchRevision { commit: "dummy_commit".to_string(), tree: "dummy_tree".to_string(), commit: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa".to_string(), tree: "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb".to_string(), body: Some("Updated description".to_string()), }, clock: 0,