fix: misc

huber-common/src/log.rs

@@ -1,5 +1,4 @@
 use std::str::FromStr;
-
 use log::LevelFilter;
 
 use crate::model::config::Config;

huber/Cargo.toml

@@ -20,8 +20,6 @@ chrono.workspace = true
 
 [dev-dependencies]
 assert_cmd = "2.0.16"
-pretty_assertions = "1.4.1"
-rstest = "0.24.0"
 whoami = "1.5.2"
 
 [dependencies]

huber/src/bin/huber.rs

@@ -9,6 +9,7 @@ use anyhow::anyhow;
 use clap::{CommandFactory, Parser, ValueHint};
 use clap_complete::Generator;
 use huber::cmd::config::ConfigCommands;
+use huber::cmd::lock::LockCommands;
 use huber::cmd::repo::RepoCommands;
 use huber::cmd::CommandTrait;
 use huber::cmd::Commands;
@@ -19,11 +20,10 @@ use huber_common::fs::dir;
 use huber_common::log::Logger;
 use huber_common::model::config::Config;
 use libcli_rs::output::OutputFormat;
-use log::{error, warn, LevelFilter};
+use log::{error, LevelFilter};
 use scopeguard::defer;
 use simpledi_rs::di::{DIContainer, DIContainerTrait, DependencyInjectTrait};
 use simpledi_rs::inject_dep;
-use huber::cmd::lock::LockCommands;
 
 #[derive(Parser)]
 #[command(version, bin_name = env!("CARGO_PKG_NAME"), about, long_about = None)]
@@ -149,7 +149,7 @@ async fn main() {
 
         if let Some(e) = e.downcast_ref::<HuberError>() {
             let source_err = e.source().map(|e| format!(": {}", e)).unwrap_or_default();
-            warn!("{}{}", e, source_err);
+            error!("{}{}", e, source_err);
         } else {
             error!("{}", e);
         }

huber/tests/config.rs

@@ -14,7 +14,7 @@ fn test_config_not_found() {
     }
 
     huber_cmd!(arg("config").arg("show").assert().failure().stderr(format!(
-        "[WARN ] Config not found: \"/home/{}/.huber/config.yaml\"\n",
+        "[ERROR] Config not found: \"/home/{}/.huber/config.yaml\"\n",
         username()
     )));
 }

huber/tests/current.rs

@@ -36,7 +36,7 @@ fn test_current_fail() {
         .assert()
         .failure()
         .stderr(format!(
-            "[WARN ] Package not installed: \"{}\"\n",
+            "[ERROR] Package not installed: \"{}\"\n",
             INVALID_PKG
         )));
 }

huber/tests/info.rs

@@ -27,5 +27,5 @@ fn test_info_fail() {
         .arg(INVALID_PKG)
         .assert()
         .failure()
-        .stderr(format!("[WARN ] Package not found: \"{}\"\n", INVALID_PKG)));
+        .stderr(format!("[ERROR] Package not found: \"{}\"\n", INVALID_PKG)));
 }

huber/tests/install.rs

@@ -30,5 +30,5 @@ fn test_install_fail() {
         .arg(INVALID_PKG_VERSION)
         .assert()
         .failure()
-        .stderr("[WARN ] Package not found: \"pkg_notfound\"\n"));
+        .stderr("[ERROR] Package not found: \"pkg_notfound\"\n"));
 }

huber/tests/lock.rs

@@ -29,7 +29,7 @@ fn test_lock_fail() {
 
     assert_eq_last_line!(
         assert.get_output().stderr,
-        "[WARN ] Package unable to lock: Package not installed: \"k9s\""
+        "[ERROR] Package unable to lock: Package not installed: \"k9s\""
     );
 }
 

huber/tests/uninstall.rs

@@ -28,5 +28,5 @@ fn test_uninstall_fail() {
         .arg(INVALID_PKG)
         .assert()
         .failure()
-        .stderr(format!("[WARN ] Package not found: \"{}\"\n", INVALID_PKG)));
+        .stderr(format!("[ERROR] Package not found: \"{}\"\n", INVALID_PKG)));
 }

justfile

@@ -26,6 +26,8 @@ fmt:
 
 # Fix code
 fix:
+    @cargo install cargo-udeps
+    @cargo +nightly udeps --all-targets
     @cargo {{ cargo_opts }} fix --allow-dirty --allow-staged
 
 # Release binaries