aboutsummaryrefslogtreecommitdiffstats
path: root/browserenv_unix_test.go
diff options
context:
space:
mode:
authorTeddy Wing2020-12-20 19:15:15 +0100
committerTeddy Wing2020-12-20 19:18:15 +0100
commit5933a5ae5fd8decf70e1e49bc2d877441f0e4479 (patch)
tree147ce1cd4981ddb4825accc2c3dbfaf7023a091b /browserenv_unix_test.go
parent8d4c2f66ce0998f39f0dda1986afd0cc641a7dc1 (diff)
downloadbrowserenv-5933a5ae5fd8decf70e1e49bc2d877441f0e4479.tar.bz2
Set `pkg/browser`'s `Stderr` and `Stdout` to ours
We need to connect our `Stderr` and `Stdout` to those in `github.com/pkg/browser` so that custom writers get linked correctly in the underlying library. Previously, customising these writers only worked for the command specified in the `BROWSER` environment variable, not when `BROWSER` was unset.
Diffstat (limited to 'browserenv_unix_test.go')
-rw-r--r--browserenv_unix_test.go28
1 files changed, 28 insertions, 0 deletions
diff --git a/browserenv_unix_test.go b/browserenv_unix_test.go
index 069c322..d328290 100644
--- a/browserenv_unix_test.go
+++ b/browserenv_unix_test.go
@@ -131,3 +131,31 @@ func TestOpenURLMultipleBrowserCommands(t *testing.T) {
unsetEnvBrowser(t)
}
+
+func TestOpenFilePkgBrowserUsesStderr(t *testing.T) {
+ var stderr strings.Builder
+ Stderr = &stderr
+
+ OpenFile("file:///tmp/does-not-exist")
+
+ got := stderr.String()
+ if got == "" {
+ t.Errorf("got empty stderr want an error message")
+ }
+
+ unsetEnvBrowser(t)
+}
+
+func TestOpenURLPkgBrowserUsesStderr(t *testing.T) {
+ var stderr strings.Builder
+ Stderr = &stderr
+
+ OpenURL("file:///tmp/does-not-exist")
+
+ got := stderr.String()
+ if got == "" {
+ t.Errorf("got empty stderr want an error message")
+ }
+
+ unsetEnvBrowser(t)
+}