chore: address lint warnings

Author: NathanLovato

src/main.rs

@@ -19,7 +19,7 @@ mod cli;
 use std::{
     env, fs,
     io::{self, IsTerminal, Read, Write},
-    path::PathBuf,
+    path::{Path, PathBuf},
     thread,
 };
 
@@ -115,10 +115,12 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
         unreachable!();
     };
 
-    let mut config = FormatterConfiguration::default();
+    let mut config = FormatterConfiguration {
+        safe: use_safe_mode,
+        reorder_code: do_reorder_code,
+        ..Default::default()
+    };
 
-    config.safe = use_safe_mode;
-    config.reorder_code = do_reorder_code;
     if let Some(quote_style) = quote_style {
         config.quote_style = quote_style;
     }
@@ -317,7 +319,7 @@ fn format_one_file(
 /// incorrectly override `.editorconfig`.
 fn config_apply_editorconfig_then_cli_overrides(
     config: &mut FormatterConfiguration,
-    config_path: &PathBuf,
+    config_path: &Path,
     config_overrides: FormatterConfigOverrides,
 ) {
     gdscript_formatter::editorconfig::apply_editorconfig_to_formatter_config(config, config_path);

tests/cli_tests.rs

@@ -67,7 +67,10 @@ fn stdin_and_file_modes_apply_editorconfig_and_cli_overrides() {
         .wait_with_output()
         .expect("should collect formatter output");
     assert!(stdin_output.status.success());
-    assert_eq!(String::from_utf8(stdin_output.stdout).unwrap(), expected);
+    assert_eq!(
+        String::from_utf8(stdin_output.stdout).expect("stdin output should be valid UTF-8"),
+        expected,
+    );
 
     let input_path = directory.join("input.gd");
     fs::write(&input_path, input).expect("should write input file");
@@ -88,7 +91,10 @@ fn stdin_and_file_modes_apply_editorconfig_and_cli_overrides() {
     .output()
     .expect("should format file");
     assert!(file_output.status.success());
-    assert_eq!(String::from_utf8(file_output.stdout).unwrap(), expected);
+    assert_eq!(
+        String::from_utf8(file_output.stdout).expect("file output should be valid UTF-8"),
+        expected,
+    );
 
     fs::remove_dir_all(directory).expect("should remove temporary test directory");
 }