fixed relaxed exit code

jakekaplan · jakekaplan · commit b9fda4fa9f64 · 2026-01-19T17:03:12.000-05:00
diff --git a/crates/loq_cli/src/relax.rs b/crates/loq_cli/src/relax.rs
@@ -40,7 +40,7 @@ pub fn run_relax<W1: WriteColor, W2: WriteColor>(
         Ok(report) => {
             if report.is_empty() {
                 let _ = writeln!(stdout, "No violations to relax");
-                return ExitStatus::Failure;
+                return ExitStatus::Success;
             }
             let _ = write_report(stdout, &report);
             ExitStatus::Success
diff --git a/crates/loq_cli/tests/relax.rs b/crates/loq_cli/tests/relax.rs
@@ -121,7 +121,7 @@ max_lines = 700
 }
 
 #[test]
-fn exits_one_when_no_violations() {
+fn exits_zero_when_no_violations() {
     let temp = TempDir::new().unwrap();
     write_file(&temp, "src/small.rs", &repeat_lines(10));
 
@@ -131,7 +131,7 @@ fn exits_one_when_no_violations() {
         .output()
         .unwrap();
 
-    assert_eq!(output.status.code(), Some(1));
+    assert!(output.status.success());
     let stdout = strip_ansi(&String::from_utf8_lossy(&output.stdout));
     assert!(stdout.contains("No violations to relax"));
 }

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ pub fn run_relax<W1: WriteColor, W2: WriteColor>(`
`40`	`40`	`Ok(report) => {`
`41`	`41`	`if report.is_empty() {`
`42`	`42`	`let _ = writeln!(stdout, "No violations to relax");`
`43`		`- return ExitStatus::Failure;`
	`43`	`+ return ExitStatus::Success;`
`44`	`44`	`}`
`45`	`45`	`let _ = write_report(stdout, &report);`
`46`	`46`	`ExitStatus::Success`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ max_lines = 700`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`#[test]`
`124`		`-fn exits_one_when_no_violations() {`
	`124`	`+fn exits_zero_when_no_violations() {`
`125`	`125`	`let temp = TempDir::new().unwrap();`
`126`	`126`	`write_file(&temp, "src/small.rs", &repeat_lines(10));`
`127`	`127`
`@@ -131,7 +131,7 @@ fn exits_one_when_no_violations() {`
`131`	`131`	`.output()`
`132`	`132`	`.unwrap();`
`133`	`133`
`134`		`- assert_eq!(output.status.code(), Some(1));`
	`134`	`+ assert!(output.status.success());`
`135`	`135`	`let stdout = strip_ansi(&String::from_utf8_lossy(&output.stdout));`
`136`	`136`	`assert!(stdout.contains("No violations to relax"));`
`137`	`137`	`}`