fix: correct misleading test title and add missing assertion

helizaga · helizaga · commit 02c8aeeeec54 · 2026-02-12T00:25:48.000-08:00
- Rename ui_color test to match what it actually tests (GTR_COLOR=never,
  not NO_COLOR empty string)
- Assert warning output in cmd_copy unknown branch test
diff --git a/tests/cmd_copy.bats b/tests/cmd_copy.bats
@@ -58,4 +58,5 @@ teardown() {
   # cmd_copy skips unknown targets with || continue, then warns
   run cmd_copy nonexistent -- ".env"
   [ "$status" -eq 0 ]
+  [[ "$output" == *"No files copied"* ]]
 }
diff --git a/tests/ui_color.bats b/tests/ui_color.bats
@@ -16,7 +16,7 @@ _source_ui() {
   [ "$status" -ne 0 ]
 }
 
-@test "_ui_should_color returns 1 when NO_COLOR is empty string (unset wins)" {
+@test "_ui_should_color returns 1 when GTR_COLOR=never" {
   _source_ui
   unset NO_COLOR
   GTR_COLOR=never run _ui_should_color 2

Original file line number	Diff line number	Diff line change
`@@ -58,4 +58,5 @@ teardown() {`
`58`	`58`	`# cmd_copy skips unknown targets with \|\| continue, then warns`
`59`	`59`	`run cmd_copy nonexistent -- ".env"`
`60`	`60`	`[ "$status" -eq 0 ]`
	`61`	`+ [[ "$output" == "No files copied" ]]`
`61`	`62`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ _source_ui() {`
`16`	`16`	`[ "$status" -ne 0 ]`
`17`	`17`	`}`
`18`	`18`
`19`		`-@test "_ui_should_color returns 1 when NO_COLOR is empty string (unset wins)" {`
	`19`	`+@test "_ui_should_color returns 1 when GTR_COLOR=never" {`
`20`	`20`	`_source_ui`
`21`	`21`	`unset NO_COLOR`
`22`	`22`	`GTR_COLOR=never run _ui_should_color 2`