Skip to content

Commit

Permalink
serde tests now run in CI, removed serde dependencies subversions
Browse files Browse the repository at this point in the history
  • Loading branch information
d-sonuga committed Dec 4, 2024
1 parent 393ade1 commit d32e9c5
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 11 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/dusk_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,10 @@ jobs:
name: Nightly std tests
uses: dusk-network/.github/.github/workflows/run-tests.yml@main
with:
test_flags: --features=rkyv-impl,rkyv/size_16
test_flags: --features=rkyv-impl,rkyv/size_16,serde

test_parallel:
name: Nightly std tests parallel
uses: dusk-network/.github/.github/workflows/run-tests.yml@main
with:
test_flags: --features=parallel,rkyv-impl,rkyv/size_16
test_flags: --features=parallel,rkyv-impl,rkyv/size_16,serde
6 changes: 3 additions & 3 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ zeroize = { version = "1", default-features = false, features = ["derive"] }
rkyv = { version = "0.7", optional = true, default-features = false }
bytecheck = { version = "0.6", optional = true, default-features = false }
rayon = { version = "1.8", optional = true }
serde = { version = "1.0.210", optional = true }
bs58 = { version = "0.4.0" , optional = true }
serde_json = { version = "1.0.128", optional = true }
serde = { version = "1.0", optional = true }
bs58 = { version = "0.4" , optional = true }
serde_json = { version = "1.0", optional = true }

[dev-dependencies]
rand = "0.8"
Expand Down
12 changes: 6 additions & 6 deletions src/serde_support.rs
Original file line number Diff line number Diff line change
Expand Up @@ -107,9 +107,9 @@ impl<'de> Deserialize<'de> for Signature {
}
Err(err) => return Err(de::Error::custom(format!("{err:?}"))),
}
let pubk = Signature::from_bytes(&bytes)
let sig = Signature::from_bytes(&bytes)
.map_err(|err| de::Error::custom(format!("{err:?}")))?;
Ok(pubk)
Ok(sig)
}
}

Expand Down Expand Up @@ -139,9 +139,9 @@ impl<'de> Deserialize<'de> for MultisigSignature {
}
Err(err) => return Err(de::Error::custom(format!("{err:?}"))),
}
let pubk = MultisigSignature::from_bytes(&bytes)
let sig = MultisigSignature::from_bytes(&bytes)
.map_err(|err| de::Error::custom(format!("{err:?}")))?;
Ok(pubk)
Ok(sig)
}
}

Expand Down Expand Up @@ -171,8 +171,8 @@ impl<'de> Deserialize<'de> for SecretKey {
}
Err(err) => return Err(de::Error::custom(format!("{err:?}"))),
}
let pubk = SecretKey::from_bytes(&bytes)
let sk = SecretKey::from_bytes(&bytes)
.map_err(|err| de::Error::custom(format!("{err:?}")))?;
Ok(pubk)
Ok(sk)
}
}

0 comments on commit d32e9c5

Please sign in to comment.