Skip to content

Commit 58e2d18

Browse files
committed
refactor
1 parent 0f20e0a commit 58e2d18

File tree

1 file changed

+2
-21
lines changed

1 file changed

+2
-21
lines changed

gix-hash/tests/hash/oid.rs

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -22,41 +22,22 @@ fn is_null() {
2222

2323
#[test]
2424
fn is_empty_blob() {
25-
// Test with ObjectId::empty_blob
2625
let empty_blob = gix_hash::ObjectId::empty_blob(gix_hash::Kind::Sha1);
2726
assert!(empty_blob.is_empty_blob());
2827
assert!(empty_blob.as_ref().is_empty_blob());
29-
30-
// Test that non-empty blob hash returns false
28+
3129
let non_empty = gix_hash::Kind::Sha1.null();
3230
assert!(!non_empty.is_empty_blob());
3331
assert!(!non_empty.as_ref().is_empty_blob());
3432
}
3533

3634
#[test]
3735
fn is_empty_tree() {
38-
// Test with ObjectId::empty_tree
3936
let empty_tree = gix_hash::ObjectId::empty_tree(gix_hash::Kind::Sha1);
4037
assert!(empty_tree.is_empty_tree());
4138
assert!(empty_tree.as_ref().is_empty_tree());
42-
43-
// Test that non-empty tree hash returns false
39+
4440
let non_empty = gix_hash::Kind::Sha1.null();
4541
assert!(!non_empty.is_empty_tree());
4642
assert!(!non_empty.as_ref().is_empty_tree());
4743
}
48-
49-
#[test]
50-
fn oid_methods_are_consistent_with_objectid() {
51-
// Verify that the oid methods return the same results as ObjectId methods
52-
let empty_blob = gix_hash::ObjectId::empty_blob(gix_hash::Kind::Sha1);
53-
let empty_tree = gix_hash::ObjectId::empty_tree(gix_hash::Kind::Sha1);
54-
55-
// Check that ObjectId and oid versions give same results
56-
assert_eq!(empty_blob.is_empty_blob(), empty_blob.as_ref().is_empty_blob());
57-
assert_eq!(empty_tree.is_empty_tree(), empty_tree.as_ref().is_empty_tree());
58-
59-
// Check cross-validation (empty blob is not empty tree and vice versa)
60-
assert!(!empty_blob.as_ref().is_empty_tree());
61-
assert!(!empty_tree.as_ref().is_empty_blob());
62-
}

0 commit comments

Comments
 (0)