Skip to content

Commit a7cd293

Browse files
committed
fix tests
Signed-off-by: Dave Grantham <[email protected]>
1 parent e95d1df commit a7cd293

File tree

40 files changed

+661
-249
lines changed

40 files changed

+661
-249
lines changed

Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ serde = { version = "1.0.219", default-features = false, features = ["alloc", "d
4747
serde_cbor = { version = "0.11.2", features = ["tags"]}
4848
serde_json = { version = "1.0.104"}
4949
serde_test = { version = "1.0.104"}
50+
test-log = { version = "0.2.17", features = ["trace", "color"] }
5051
thiserror = "2.0.12"
5152
tracing = "0.1.41"
5253
tracing-subscriber = { version = "0.3.19", features = ["env-filter"] }

crates/multibase/Cargo.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ base-x = { version = "0.2.11", default-features = false }
1717
base256emoji = "1.0.2"
1818
data-encoding = { version = "2.9.0", default-features = false, features = ["alloc"] }
1919
data-encoding-macro = "0.1.18"
20+
test-log.workspace = true
21+
tracing.workspace = true
2022

2123
[dev-dependencies]
2224
criterion = "0.5.1"

crates/multibase/tests/lib.rs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
use multibase::{decode, encode, Base, Base::*};
2+
use test_log::test;
3+
use tracing::{span, Level};
24

35
fn encode_decode_assert(input: &[u8], test_cases: Vec<(Base, &str)>) {
46
for (base, output) in test_cases {
@@ -9,18 +11,21 @@ fn encode_decode_assert(input: &[u8], test_cases: Vec<(Base, &str)>) {
911

1012
#[test]
1113
fn test_bases_code() {
14+
let _ = span!(Level::INFO, "test_bases_code").entered();
1215
assert_eq!(Identity.code(), '\x00');
1316
assert_eq!(Base2.code(), '0');
1417
}
1518

1619
#[test]
1720
fn test_bases_from_code() {
21+
let _ = span!(Level::INFO, "test_bases_from_code").entered();
1822
assert_eq!(Base::from_code('\x00').unwrap(), Identity);
1923
assert_eq!(Base::from_code('0').unwrap(), Base2);
2024
}
2125

2226
#[test]
2327
fn test_round_trip() {
28+
let _ = span!(Level::INFO, "test_round_trip").entered();
2429
let test_cases: &[&str] = &[
2530
"helloworld",
2631
"we all want decentralization",
@@ -36,6 +41,7 @@ fn test_round_trip() {
3641

3742
#[test]
3843
fn test_basic() {
44+
let _ = span!(Level::INFO, "test_basic").entered();
3945
let input = b"yes mani !";
4046
let test_cases = vec![
4147
(Identity, "\x00yes mani !"),
@@ -71,6 +77,7 @@ fn test_basic() {
7177

7278
#[test]
7379
fn preserves_leading_zero() {
80+
let _ = span!(Level::INFO, "preserves_leading_zero").entered();
7481
let input = b"\x00yes mani !";
7582
let test_cases = vec![
7683
(Identity, "\x00\x00yes mani !"),
@@ -103,6 +110,7 @@ fn preserves_leading_zero() {
103110

104111
#[test]
105112
fn preserves_two_leading_zeroes() {
113+
let _ = span!(Level::INFO, "preserves_two_leading_zeroes").entered();
106114
let input = b"\x00\x00yes mani !";
107115
let test_cases = vec![
108116
(Identity, "\x00\x00\x00yes mani !"),
@@ -135,6 +143,7 @@ fn preserves_two_leading_zeroes() {
135143

136144
#[test]
137145
fn case_insensitivity() {
146+
let _ = span!(Level::INFO, "case_insensitivity").entered();
138147
let input = b"hello world";
139148
let test_cases = vec![
140149
(Base16Lower, "f68656c6c6f20776F726C64"),

crates/multicid/Cargo.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,9 @@ rand_core_6.workspace = true
2424
rng.workspace = true
2525
serde = { workspace = true, optional = true }
2626
serde_cbor = { workspace = true, optional = true }
27+
test-log.workspace = true
2728
thiserror.workspace = true
29+
tracing.workspace = true
2830

2931
[dev-dependencies]
3032
hex.workspace = true

crates/multicid/src/cid.rs

Lines changed: 21 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -210,8 +210,7 @@ impl Builder {
210210
return Err(CidError::LegacyCid.into());
211211
}
212212
Ok(EncodedCid::new(
213-
self.base_encoding
214-
.unwrap_or_else(Cid::preferred_encoding),
213+
self.base_encoding.unwrap_or_else(Cid::preferred_encoding),
215214
self.try_build()?,
216215
))
217216
}
@@ -240,9 +239,12 @@ impl Builder {
240239
mod tests {
241240
use super::*;
242241
use multihash::mh;
242+
use test_log::test;
243+
use tracing::{span, Level};
243244

244245
#[test]
245246
fn test_default() {
247+
let _ = span!(Level::INFO, "test_default").entered();
246248
let v1 = Cid::default();
247249
assert_eq!(Codec::Cidv1, v1.codec());
248250
assert_eq!(Codec::DagCbor, v1.target_codec);
@@ -251,6 +253,7 @@ mod tests {
251253

252254
#[test]
253255
fn test_v0() {
256+
let _ = span!(Level::INFO, "test_v0").entered();
254257
let v0 = Builder::default()
255258
.with_hash(
256259
&mh::Builder::new_from_bytes(Codec::Sha2256, b"for great justice, move every zig!")
@@ -268,24 +271,31 @@ mod tests {
268271

269272
#[test]
270273
fn test_unknown_decode() {
274+
let _ = span!(Level::INFO, "test_unknown_decode").entered();
271275
// this does not assume it is a legacy v0 encoded CID
272276
let v0_1 = EncodedCid::try_from("Qmdb16CztyugMSs5anEPrJ6bLeo39bTGcM13zNPqjqUidT").unwrap();
273277
assert_eq!(Codec::Identity, v0_1.codec());
274278
assert_eq!(Codec::DagPb, v0_1.target_codec);
275279
assert_eq!(Codec::Sha2256, v0_1.hash.codec());
276280

277281
// this does not assume a multibase encoded CID
278-
let v0_2 = EncodedCid::try_from("bafybeihcrr5owouhnms63areolshu2lp4jjbjqlhf4exegk7tnso5ja6py").unwrap();
282+
let v0_2 =
283+
EncodedCid::try_from("bafybeihcrr5owouhnms63areolshu2lp4jjbjqlhf4exegk7tnso5ja6py")
284+
.unwrap();
279285
assert_eq!(Codec::Cidv1, v0_2.codec());
280286
assert_eq!(Codec::DagPb, v0_2.target_codec);
281287
assert_eq!(Codec::Sha2256, v0_2.hash.codec());
282288

283-
let v0_3 = EncodedCid::try_from("f01701220e28c7aeb3a876b25ed822472e47a696fe25214c1672f0972195f9b64eea41e7e").unwrap();
289+
let v0_3 = EncodedCid::try_from(
290+
"f01701220e28c7aeb3a876b25ed822472e47a696fe25214c1672f0972195f9b64eea41e7e",
291+
)
292+
.unwrap();
284293
assert_eq!(Codec::Cidv1, v0_3.codec());
285294
assert_eq!(Codec::DagPb, v0_3.target_codec);
286295
assert_eq!(Codec::Sha2256, v0_3.hash.codec());
287296

288-
let v0_4 = EncodedCid::try_from("uAXASIOKMeus6h2sl7YIkcuR6aW_iUhTBZy8Jchlfm2TupB5-").unwrap();
297+
let v0_4 =
298+
EncodedCid::try_from("uAXASIOKMeus6h2sl7YIkcuR6aW_iUhTBZy8Jchlfm2TupB5-").unwrap();
289299
assert_eq!(Codec::Cidv1, v0_4.codec());
290300
assert_eq!(Codec::DagPb, v0_4.target_codec);
291301
assert_eq!(Codec::Sha2256, v0_4.hash.codec());
@@ -303,6 +313,7 @@ mod tests {
303313

304314
#[test]
305315
fn test_v0_binary_roundtrip() {
316+
let _ = span!(Level::INFO, "test_v0_binary_roundtrip").entered();
306317
let v0 = Builder::default()
307318
.with_hash(
308319
&mh::Builder::new_from_bytes(Codec::Sha2256, b"for great justice, move every zig!")
@@ -318,6 +329,7 @@ mod tests {
318329

319330
#[test]
320331
fn test_v0_encoded_roundtrip() {
332+
let _ = span!(Level::INFO, "test_v0_encoded_roundtrip").entered();
321333
let v0 = Builder::default()
322334
.with_hash(
323335
&mh::Builder::new_from_bytes(Codec::Sha2256, b"for great justice, move every zig!")
@@ -334,6 +346,7 @@ mod tests {
334346

335347
#[test]
336348
fn test_v1() {
349+
let _ = span!(Level::INFO, "test_v1").entered();
337350
let v1 = Builder::new(Codec::Cidv1)
338351
.with_target_codec(Codec::DagCbor)
339352
.with_hash(
@@ -352,6 +365,7 @@ mod tests {
352365

353366
#[test]
354367
fn test_v1_binary_roundtrip() {
368+
let _ = span!(Level::INFO, "test_v1_binary_roundtrip").entered();
355369
let v1 = Builder::new(Codec::Cidv1)
356370
.with_target_codec(Codec::DagCbor)
357371
.with_hash(
@@ -368,6 +382,7 @@ mod tests {
368382

369383
#[test]
370384
fn test_v1_encoded_roundtrip() {
385+
let _ = span!(Level::INFO, "test_v1_encoded_roundtrip").entered();
371386
let v1 = Builder::new(Codec::Cidv1)
372387
.with_target_codec(Codec::DagCbor)
373388
.with_base_encoding(Base::Base32Lower)
@@ -386,6 +401,7 @@ mod tests {
386401

387402
#[test]
388403
fn test_null() {
404+
let _ = span!(Level::INFO, "test_null").entered();
389405
let cid1 = Cid::null();
390406
assert!(cid1.is_null());
391407
let cid2 = Cid::default();

0 commit comments

Comments
 (0)