courier: Don't hardcode path to sleep binary in the tests

ThinkChaos · albertito · commit eeb2deb7f608 · 2025-08-13T23:10:16.000+01:00
On NixOS, `/bin` is basically empty and this causes the courier tests (which invoke `/bin/sleep`) to fail. This patch fixes the tests by removing the hardcoded path. #73 Amended-by: Alberto Bertogli <albertito@blitiri.com.ar> Adjusted commit message.
diff --git a/internal/courier/mda_test.go b/internal/courier/mda_test.go
@@ -31,7 +31,7 @@ func TestMDA(t *testing.T) {
 }
 
 func TestMDATimeout(t *testing.T) {
-	p := MDA{"/bin/sleep", []string{"1"}, 100 * time.Millisecond}
+	p := MDA{"sleep", []string{"1"}, 100 * time.Millisecond}
 
 	err, permanent := p.Deliver("from", "to@local", []byte("data"))
 	if err != errTimeout {

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func TestMDA(t *testing.T) {`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`func TestMDATimeout(t *testing.T) {`
`34`		`- p := MDA{"/bin/sleep", []string{"1"}, 100 * time.Millisecond}`
	`34`	`+ p := MDA{"sleep", []string{"1"}, 100 * time.Millisecond}`
`35`	`35`
`36`	`36`	`err, permanent := p.Deliver("from", "to@local", []byte("data"))`
`37`	`37`	`if err != errTimeout {`