summaryrefslogtreecommitdiffhomepage
path: root/atomicfile/atomicfile_test.go
diff options
context:
space:
mode:
Diffstat (limited to 'atomicfile/atomicfile_test.go')
-rw-r--r--atomicfile/atomicfile_test.go94
1 files changed, 47 insertions, 47 deletions
diff --git a/atomicfile/atomicfile_test.go b/atomicfile/atomicfile_test.go
index 78c93e664..b7a78765b 100644
--- a/atomicfile/atomicfile_test.go
+++ b/atomicfile/atomicfile_test.go
@@ -1,47 +1,47 @@
-// Copyright (c) Tailscale Inc & AUTHORS
-// SPDX-License-Identifier: BSD-3-Clause
-
-//go:build !js && !windows
-
-package atomicfile
-
-import (
- "net"
- "os"
- "path/filepath"
- "runtime"
- "strings"
- "testing"
-)
-
-func TestDoesNotOverwriteIrregularFiles(t *testing.T) {
- // Per tailscale/tailscale#7658 as one example, almost any imagined use of
- // atomicfile.Write should likely not attempt to overwrite an irregular file
- // such as a device node, socket, or named pipe.
-
- const filename = "TestDoesNotOverwriteIrregularFiles"
- var path string
- // macOS private temp does not allow unix socket creation, but /tmp does.
- if runtime.GOOS == "darwin" {
- path = filepath.Join("/tmp", filename)
- t.Cleanup(func() { os.Remove(path) })
- } else {
- path = filepath.Join(t.TempDir(), filename)
- }
-
- // The least troublesome thing to make that is not a file is a unix socket.
- // Making a null device sadly requires root.
- l, err := net.ListenUnix("unix", &net.UnixAddr{Name: path, Net: "unix"})
- if err != nil {
- t.Fatal(err)
- }
- defer l.Close()
-
- err = WriteFile(path, []byte("hello"), 0644)
- if err == nil {
- t.Fatal("expected error, got nil")
- }
- if !strings.Contains(err.Error(), "is not a regular file") {
- t.Fatalf("unexpected error: %v", err)
- }
-}
+// Copyright (c) Tailscale Inc & AUTHORS
+// SPDX-License-Identifier: BSD-3-Clause
+
+//go:build !js && !windows
+
+package atomicfile
+
+import (
+ "net"
+ "os"
+ "path/filepath"
+ "runtime"
+ "strings"
+ "testing"
+)
+
+func TestDoesNotOverwriteIrregularFiles(t *testing.T) {
+ // Per tailscale/tailscale#7658 as one example, almost any imagined use of
+ // atomicfile.Write should likely not attempt to overwrite an irregular file
+ // such as a device node, socket, or named pipe.
+
+ const filename = "TestDoesNotOverwriteIrregularFiles"
+ var path string
+ // macOS private temp does not allow unix socket creation, but /tmp does.
+ if runtime.GOOS == "darwin" {
+ path = filepath.Join("/tmp", filename)
+ t.Cleanup(func() { os.Remove(path) })
+ } else {
+ path = filepath.Join(t.TempDir(), filename)
+ }
+
+ // The least troublesome thing to make that is not a file is a unix socket.
+ // Making a null device sadly requires root.
+ l, err := net.ListenUnix("unix", &net.UnixAddr{Name: path, Net: "unix"})
+ if err != nil {
+ t.Fatal(err)
+ }
+ defer l.Close()
+
+ err = WriteFile(path, []byte("hello"), 0644)
+ if err == nil {
+ t.Fatal("expected error, got nil")
+ }
+ if !strings.Contains(err.Error(), "is not a regular file") {
+ t.Fatalf("unexpected error: %v", err)
+ }
+}