Share manifest struct with integration tests (#88)

Author: casey

Cargo.toml

@@ -5,7 +5,7 @@ authors = ["Casey Rodarmor <[email protected]>"]
 autotests = false
 categories = ["authentication", "command-line-utilities", "cryptography"]
 description = "file verification utility"
-edition = "2021"
+edition = "2024"
 homepage = "https://github.com/casey/filepack"
 keywords = ["checksum", "verification"]
 license = "CC0-1.0"
@@ -42,6 +42,8 @@ all = { level = "deny", priority = -1 }
 arbitrary-source-item-ordering = "deny"
 float-cmp = "allow"
 large_enum_variant = "allow"
+missing-errors-doc = "allow"
+missing-panics-doc = "allow"
 needless-pass-by-value = "allow"
 pedantic = { level = "deny", priority = -1 }
 result-large-err = "allow"

rustfmt.toml

@@ -1,4 +1,4 @@
-edition = "2021"
+edition = "2024"
 max_width = 100
 newline_style = "Unix"
 tab_spaces = 2

src/arguments.rs

@@ -1,8 +1,8 @@
 use {
   super::*,
   clap::builder::{
-    styling::{AnsiColor, Effects},
     Styles,
+    styling::{AnsiColor, Effects},
   },
 };
 

src/display_path.rs

@@ -1,7 +1,7 @@
 use super::*;
 
 #[derive(Debug)]
-pub(crate) struct DisplayPath(PathBuf);
+pub struct DisplayPath(PathBuf);
 
 impl Display for DisplayPath {
   fn fmt(&self, f: &mut Formatter) -> fmt::Result {

src/error.rs

@@ -2,7 +2,7 @@ use super::*;
 
 #[derive(Debug, Snafu)]
 #[snafu(context(suffix(false)), visibility(pub(crate)))]
-pub(crate) enum Error {
+pub enum Error {
   #[snafu(display("failed to get current directory"))]
   CurrentDir { source: io::Error },
   #[snafu(display("failed to get local data directory"))]

src/lib.rs

@@ -1,11 +1,9 @@
 use {
   self::{
-    arguments::Arguments, display_path::DisplayPath, display_secret::DisplaySecret, error::Error,
-    lint::Lint, lint_group::LintGroup, list::List, manifest::Manifest, metadata::Metadata,
-    options::Options, owo_colorize_ext::OwoColorizeExt, private_key::PrivateKey,
-    public_key::PublicKey, relative_path::RelativePath, signature::Signature,
-    signature_error::SignatureError, style::Style, subcommand::Subcommand, template::Template,
-    utf8_path_ext::Utf8PathExt,
+    arguments::Arguments, display_path::DisplayPath, display_secret::DisplaySecret, lint::Lint,
+    lint_group::LintGroup, list::List, metadata::Metadata, options::Options,
+    owo_colorize_ext::OwoColorizeExt, private_key::PrivateKey, signature_error::SignatureError,
+    style::Style, subcommand::Subcommand, template::Template, utf8_path_ext::Utf8PathExt,
   },
   blake3::Hasher,
   camino::{Utf8Component, Utf8Path, Utf8PathBuf},
@@ -15,7 +13,7 @@ use {
   owo_colors::Styled,
   serde::{Deserialize, Deserializer, Serialize, Serializer},
   serde_with::{DeserializeFromStr, SerializeDisplay},
-  snafu::{ensure, ErrorCompat, OptionExt, ResultExt, Snafu},
+  snafu::{ErrorCompat, OptionExt, ResultExt, Snafu, ensure},
   std::{
     array::TryFromSliceError,
     backtrace::{Backtrace, BacktraceStatus},
@@ -32,7 +30,10 @@ use {
   walkdir::WalkDir,
 };
 
-pub use self::{entry::Entry, hash::Hash};
+pub use self::{
+  entry::Entry, error::Error, hash::Hash, manifest::Manifest, public_key::PublicKey,
+  relative_path::RelativePath, signature::Signature,
+};
 
 #[cfg(test)]
 use assert_fs::TempDir;
@@ -91,12 +92,12 @@ pub fn run() {
       eprintln!("       {}─ {err}", if i < causes - 1 { '├' } else { '└' });
     }
 
-    if let Some(backtrace) = err.backtrace() {
-      if backtrace.status() == BacktraceStatus::Captured {
-        eprintln!();
-        eprintln!("backtrace:");
-        eprintln!("{backtrace}");
-      }
+    if let Some(backtrace) = err.backtrace()
+      && backtrace.status() == BacktraceStatus::Captured
+    {
+      eprintln!();
+      eprintln!("backtrace:");
+      eprintln!("{backtrace}");
     }
 
     process::exit(1);

src/manifest.rs

@@ -2,11 +2,11 @@ use super::*;
 
 #[derive(Clone, Debug, Deserialize, PartialEq, Serialize)]
 #[serde(deny_unknown_fields, rename_all = "kebab-case")]
-pub(crate) struct Manifest {
+pub struct Manifest {
   #[serde(default, skip_serializing_if = "BTreeMap::is_empty")]
-  pub(crate) files: BTreeMap<RelativePath, Entry>,
+  pub files: BTreeMap<RelativePath, Entry>,
   #[serde(default, skip_serializing_if = "BTreeMap::is_empty")]
-  pub(crate) signatures: BTreeMap<PublicKey, Signature>,
+  pub signatures: BTreeMap<PublicKey, Signature>,
 }
 
 impl Manifest {
@@ -25,7 +25,7 @@ impl Manifest {
     hasher.finalize().into()
   }
 
-  pub(crate) fn load(path: Option<&Utf8Path>) -> Result<(Utf8PathBuf, Self)> {
+  pub fn load(path: Option<&Utf8Path>) -> Result<(Utf8PathBuf, Self)> {
     let path = if let Some(path) = path {
       if filesystem::metadata(path)?.is_dir() {
         path.join(Manifest::FILENAME)
@@ -45,7 +45,7 @@ impl Manifest {
     Ok((path, manifest))
   }
 
-  pub(crate) fn save(&self, path: &Utf8Path) -> Result<()> {
+  pub fn save(&self, path: &Utf8Path) -> Result<()> {
     filesystem::write(path, format!("{}\n", serde_json::to_string(self).unwrap()))
   }
 }

src/private_key.rs

@@ -2,7 +2,7 @@ use super::*;
 
 #[derive(Debug, Snafu)]
 #[snafu(context(suffix(Error)))]
-pub(crate) enum Error {
+pub enum Error {
   #[snafu(display("invalid private key hex"))]
   Hex { source: hex::FromHexError },
   #[snafu(display("invalid private key byte length {length}"))]

src/public_key.rs

@@ -2,7 +2,7 @@ use super::*;
 
 #[derive(Debug, Snafu)]
 #[snafu(context(suffix(Error)))]
-pub(crate) enum Error {
+pub enum Error {
   #[snafu(display("invalid public key hex: `{key}`"))]
   Hex {
     key: String,
@@ -21,7 +21,7 @@ pub(crate) enum Error {
 }
 
 #[derive(Clone, Debug, DeserializeFromStr, Eq, PartialEq, SerializeDisplay)]
-pub(crate) struct PublicKey(ed25519_dalek::VerifyingKey);
+pub struct PublicKey(ed25519_dalek::VerifyingKey);
 
 impl PublicKey {
   const LEN: usize = ed25519_dalek::PUBLIC_KEY_LENGTH;
@@ -38,7 +38,7 @@ impl PublicKey {
     Ok(public_key)
   }
 
-  pub(crate) fn verify(&self, message: &[u8], signature: &Signature) -> Result<()> {
+  pub fn verify(&self, message: &[u8], signature: &Signature) -> Result<()> {
     self
       .0
       .verify_strict(message, signature.as_ref())

src/relative_path.rs

@@ -5,7 +5,7 @@ pub(crate) use self::error::Error;
 mod error;
 
 #[derive(Clone, Debug, Eq, Hash, Ord, PartialEq, PartialOrd, Serialize)]
-pub(crate) struct RelativePath(String);
+pub struct RelativePath(String);
 
 impl RelativePath {
   const JUNK_NAMES: [&'static str; 2] = [".DS_Store", ".localized"];
@@ -140,10 +140,10 @@ impl FromStr for RelativePath {
     let mut chars = s.chars();
     let first = chars.next();
     let second = chars.next();
-    if let Some((first, second)) = first.zip(second) {
-      if second == ':' {
-        return Err(Error::WindowsDiskPrefix { letter: first });
-      }
+    if let Some((first, second)) = first.zip(second)
+      && second == ':'
+    {
+      return Err(Error::WindowsDiskPrefix { letter: first });
     }
 
     let mut path = String::new();