Update Rust toolchain to 1.92.0 and fix clippy (#23)

* Update Rust toolchain to 1.92.0

Fix new clippy lints

* Update assert_cmd and fix deprecation warnings

Author: martinemde

Cargo.toml

@@ -50,7 +50,7 @@ fnmatch-regex2 = "0.3.0"
 strip-ansi-escapes = "0.2.0"
 
 [dev-dependencies]
-assert_cmd = "2.0.10" # testing CLI
+assert_cmd = "2.1.1" # testing CLI
 rusty-hook = "^0.11.2" # git hooks
 predicates = "3.0.2" # kind of like rspec assertions
 pretty_assertions = "1.3.0" # Shows a more readable diff when comparing objects

rust-toolchain.toml

@@ -1,4 +1,4 @@
 [toolchain]
-channel = "1.83.0"
+channel = "1.92.0"
 components = ["clippy", "rustfmt"]
 targets = ["x86_64-apple-darwin", "aarch64-apple-darwin", "x86_64-unknown-linux-gnu"]

src/packs/checker.rs

@@ -517,8 +517,7 @@ mod tests {
     #[test]
     fn test_write_violations() {
         let chec_result = CheckAllResult {
-            reportable_violations: vec![
-                Violation {
+            reportable_violations: [Violation {
                     message: "foo/bar/file1.rb:10:5\nPrivacy violation: `::Foo::PrivateClass` is private to `foo`, but referenced from `bar`".to_string(),
                     identifier: ViolationIdentifier {
                         violation_type: "Privacy".to_string(),
@@ -539,8 +538,7 @@ mod tests {
                         referencing_pack_name: "foo".to_string(),
                         defining_pack_name: "bar".to_string(),
                     }
-                }
-            ].iter().cloned().collect(),
+                }].iter().cloned().collect(),
             stale_violations: Vec::new(),
             strict_mode_violations: HashSet::new(),
         };

src/packs/configuration.rs

@@ -86,7 +86,7 @@ impl Configuration {
 
     pub(crate) fn constant_resolver_configuration(
         &self,
-    ) -> ConstantResolverConfiguration {
+    ) -> ConstantResolverConfiguration<'_> {
         ConstantResolverConfiguration {
             absolute_root: &self.absolute_root,
             cache_directory: &self.cache_directory,

src/packs/creator.rs

@@ -35,8 +35,10 @@ pub fn create(
     )?;
 
     write_pack_to_disk(&new_pack)?;
-    let pack_name =
-        &name.split('/').last().context("unable to find pack name")?;
+    let pack_name = &name
+        .split('/')
+        .next_back()
+        .context("unable to find pack name")?;
     std::fs::create_dir_all(new_pack_path.join("app/public/").join(pack_name))
         .context(format!("failed to create app/public/{}", &name))?;
     std::fs::create_dir_all(

src/packs/file_utils.rs

@@ -27,10 +27,10 @@ pub fn get_file_type(path: &Path) -> Option<SupportedFileType> {
 
     let is_ruby_file = ruby_extensions
         .into_iter()
-        .any(|ext| extension.map_or(false, |e| e == ext))
+        .any(|ext| extension.is_some_and(|e| e == ext))
         || ruby_special_files.iter().any(|file| path.ends_with(file));
 
-    let is_erb_file = path.extension().map_or(false, |ext| ext == "erb");
+    let is_erb_file = path.extension().is_some_and(|ext| ext == "erb");
 
     if is_ruby_file {
         Some(SupportedFileType::Ruby)

src/packs/parsing/ruby/parse_utils.rs

@@ -116,9 +116,7 @@ pub fn get_reference_from_active_record_association(
         .chain(ASSOCIATION_METHOD_NAMES.iter().copied().map(String::from))
         .collect();
 
-    let is_association = combined_associations
-        .iter()
-        .any(|association_method| node.method_name == *association_method);
+    let is_association = combined_associations.contains(&node.method_name);
 
     if is_association {
         let first_arg: Option<&Node> = node.args.first();

src/packs/parsing/ruby/rails_utils.rs

@@ -16,9 +16,9 @@ pub(crate) fn get_acronyms_from_disk(
         let inflections_file =
             std::fs::read_to_string(inflections_path).unwrap();
         let inflections_lines = inflections_file.lines();
+        let re = Regex::new(r#"['\\"]"#).unwrap();
         for line in inflections_lines {
             if line.contains(".acronym") {
-                let re = Regex::new(r#"['\\"]"#).unwrap();
                 let acronym = re.split(line).nth(1).unwrap();
                 acronyms.insert(acronym.to_string());
             }

tests/add_constant_dependencies.rs

@@ -1,4 +1,4 @@
-use assert_cmd::Command;
+use assert_cmd::cargo::cargo_bin_cmd;
 use predicates::prelude::*;
 use pretty_assertions::assert_eq;
 use serial_test::serial;
@@ -9,7 +9,7 @@ mod common;
 #[test]
 #[serial]
 fn test_add_constant_dependencies() -> anyhow::Result<()> {
-    Command::cargo_bin("pks")?
+    cargo_bin_cmd!("pks")
         .arg("--project-root")
         .arg("tests/fixtures/app_with_missing_dependencies")
         .arg("update-dependencies-for-constant")
@@ -43,7 +43,7 @@ fn test_add_constant_dependencies() -> anyhow::Result<()> {
 #[test]
 #[serial]
 fn test_add_constant_dependencies_no_dependencies() -> anyhow::Result<()> {
-    Command::cargo_bin("pks")?
+    cargo_bin_cmd!("pks")
         .arg("--project-root")
         .arg("tests/fixtures/app_with_missing_dependencies")
         .arg("update-dependencies-for-constant")

tests/add_dependency_test.rs

@@ -1,4 +1,4 @@
-use assert_cmd::Command;
+use assert_cmd::cargo::cargo_bin_cmd;
 use predicates::prelude::*;
 use pretty_assertions::assert_eq;
 use serial_test::serial;
@@ -9,7 +9,7 @@ mod common;
 #[test]
 #[serial]
 fn test_add_dependency() -> Result<(), Box<dyn Error>> {
-    Command::cargo_bin("pks")?
+    cargo_bin_cmd!("pks")
         .arg("--project-root")
         .arg("tests/fixtures/app_with_missing_dependency")
         .arg("add-dependency")
@@ -41,7 +41,7 @@ fn test_add_dependency() -> Result<(), Box<dyn Error>> {
 #[test]
 #[serial]
 fn test_add_dependency_creating_cycle() -> Result<(), Box<dyn Error>> {
-    Command::cargo_bin("pks")?
+    cargo_bin_cmd!("pks")
         .arg("--project-root")
         .arg("tests/fixtures/app_with_missing_dependency")
         .arg("add-dependency")
@@ -80,7 +80,7 @@ fn test_add_dependency_creating_cycle() -> Result<(), Box<dyn Error>> {
 
 #[test]
 fn test_add_dependency_unnecessarily() -> Result<(), Box<dyn Error>> {
-    Command::cargo_bin("pks")?
+    cargo_bin_cmd!("pks")
         .arg("--project-root")
         .arg("tests/fixtures/app_with_missing_dependency")
         .arg("add-dependency")