fix: properly avoid bind mounts in tests

frantjc · frantjc · commit f3dafc118680 · 2026-06-01T01:10:48.000-04:00
diff --git a/helpers_test.go b/helpers_test.go
@@ -39,8 +39,8 @@ func StreamsCaptureStdout(t testing.TB) (*forge.Streams, *bytes.Buffer) {
 
 func MountShim(t testing.TB) forge.RunOpt {
 	t.Helper()
-	if os.Getenv("DAGGER_SESSION_TOKEN") == "" {
-		return MountShim(t)
+	if os.Getenv("DAGGER_SESSION_TOKEN") != "" {
+		return new(forge.RunOpts)
 	}
-	return new(forge.RunOpts)
+	return forge.WithMountShim()
 }

Original file line number	Diff line number	Diff line change
`@@ -39,8 +39,8 @@ func StreamsCaptureStdout(t testing.TB) (forge.Streams, bytes.Buffer) {`
`39`	`39`
`40`	`40`	`func MountShim(t testing.TB) forge.RunOpt {`
`41`	`41`	`t.Helper()`
`42`		`- if os.Getenv("DAGGER_SESSION_TOKEN") == "" {`
`43`		`- return MountShim(t)`
	`42`	`+ if os.Getenv("DAGGER_SESSION_TOKEN") != "" {`
	`43`	`+ return new(forge.RunOpts)`
`44`	`44`	`}`
`45`		`- return new(forge.RunOpts)`
	`45`	`+ return forge.WithMountShim()`
`46`	`46`	`}`