Skip to content
This repository was archived by the owner on Jan 30, 2025. It is now read-only.

Commit f2cb59a

Browse files
committed
Refactor to work with t.Cleanup
Resolves: #991 (comment)
1 parent 43ff9a0 commit f2cb59a

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

tests/browser_test.go

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,12 @@ func TestTmpDirCleanup(t *testing.T) {
7070
t.Parallel()
7171

7272
tmpDirPath, err := os.MkdirTemp("./", "")
73-
defer func() {
74-
err := os.RemoveAll(tmpDirPath)
75-
require.NoError(t, err)
76-
}()
73+
t.Cleanup(
74+
func() {
75+
err := os.RemoveAll(tmpDirPath)
76+
require.NoError(t, err)
77+
},
78+
)
7779
require.NoError(t, err)
7880

7981
b := newTestBrowser(
@@ -100,10 +102,12 @@ func TestTmpDirCleanupOnContextClose(t *testing.T) {
100102
t.Parallel()
101103

102104
tmpDirPath, err := os.MkdirTemp("./", "")
103-
defer func() {
104-
err := os.RemoveAll(tmpDirPath)
105-
require.NoError(t, err)
106-
}()
105+
t.Cleanup(
106+
func() {
107+
err := os.RemoveAll(tmpDirPath)
108+
require.NoError(t, err)
109+
},
110+
)
107111
require.NoError(t, err)
108112

109113
b := newTestBrowser(

0 commit comments

Comments
 (0)