Add more tests (#5)

Author: MatthewMckee4

.github/codecov.yml

@@ -8,3 +8,6 @@ coverage:
     patch:
       default:
         enabled: false
+
+ignore:
+  - "crates/action-format/**/*"

crates/action-format/src/main.rs

@@ -43,7 +43,22 @@ fn main() -> ExitCode {
     match run(&cli, printer) {
         Ok(status) => status.into(),
         Err(err) => {
-            let _ = writeln!(printer.stderr(), "{}: {}", "error".red().bold(), err);
+            #[allow(clippy::print_stderr)]
+            {
+                let mut causes = err.chain();
+                eprintln!(
+                    "{}: {}",
+                    "error".red().bold(),
+                    causes.next().unwrap().to_string().trim()
+                );
+                for cause in causes {
+                    eprintln!(
+                        "  {}: {}",
+                        "Caused by".red().bold(),
+                        cause.to_string().trim()
+                    );
+                }
+            }
             ExitStatus::Error.into()
         }
     }
@@ -60,11 +75,13 @@ fn run(cli: &Cli, printer: Printer) -> Result<ExitStatus> {
     let mut any_changed = false;
     let mut any_error = false;
 
-    for entry in walkdir::WalkDir::new(workflows_path)
+    let walker = walkdir::WalkDir::new(workflows_path)
+        .sort_by_file_name()
         .into_iter()
-        .filter_map(std::result::Result::ok)
-        .filter(|e| is_workflow_file(e.path()))
-    {
+        .filter_map(Result::ok)
+        .filter(|e| is_workflow_file(e.path()));
+
+    for entry in walker {
         match process_file(entry.path(), &config, cli, printer) {
             Ok(changed) => any_changed |= changed,
             Err(e) => {
@@ -121,16 +138,18 @@ fn process_file(
 
     if cli.diff {
         print_diff(path, &content, &formatted, printer);
+        return Ok(true);
     }
 
+    fs_err::write(path, &formatted)?;
+
     let _ = writeln!(
         printer.stdout(),
         "{}: {}",
         "Reformatted".green(),
         path.display()
     );
 
-    fs_err::write(path, &formatted)?;
     Ok(true)
 }
 

crates/action-format/tests/it/.format.rs.pending-snap

@@ -83,10 +83,10 @@ impl TestContext {
     }
 
     /// Create an action-format command for testing.
-    #[allow(clippy::unused_self)]
     pub fn command(&self) -> Command {
         let mut command = Command::new(get_bin());
         command.current_dir(self.root.path());
+        command.env("NO_COLOR", "1");
         command
     }
 }
@@ -101,18 +101,18 @@ pub fn get_bin() -> PathBuf {
     PathBuf::from(env!("CARGO_BIN_EXE_action-format"))
 }
 
+/// Common filters for snapshot testing.
 pub static INSTA_FILTERS: &[(&str, &str)] = &[
+    // Normalize Windows line endings
     (r"\r\n", "\n"),
+    // Normalize Windows paths
     (r"\\", "/"),
+    // Rewrite Windows output to Unix output
     (r"\\([\w\d]|\.)", "/$1"),
-    (r"action-format\.exe", "action-format"),
-    (
-        r"action-format(-.*)? \d+\.\d+\.\d+(-(alpha|beta|rc)\.\d+)?",
-        r"action-format [VERSION]",
-    ),
+    (r"seal\.exe", "seal"),
+    // Strip ANSI color codes (match ESC character using character class)
     (r"[\x1b]\[[0-9;]*m", ""),
-    // Normalize .github/workflows path
-    (r"\.github/workflows/", ""),
+    (r"\d+(?:;\d+)*m", ""),
 ];
 
 pub fn apply_filters<T: AsRef<str>>(mut snapshot: String, filters: impl AsRef<[(T, T)]>) -> String {