]> Cypherpunks repositories - gostls13.git/commitdiff
exp/inotify: remove use of _test
authorRob Pike <r@golang.org>
Thu, 16 Feb 2012 05:23:58 +0000 (21:23 -0800)
committerRob Pike <r@golang.org>
Thu, 16 Feb 2012 05:23:58 +0000 (21:23 -0800)
Fixes #2573.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5676063

src/pkg/exp/inotify/inotify_linux_test.go

index c2160fc65375d42198940fa55d9285848c3b435f..d41d66bfacd8d457ff4c7fa351c48baf55c0b160 100644 (file)
@@ -7,6 +7,7 @@
 package inotify
 
 import (
+       "io/ioutil"
        "os"
        "testing"
        "time"
@@ -16,16 +17,19 @@ func TestInotifyEvents(t *testing.T) {
        // Create an inotify watcher instance and initialize it
        watcher, err := NewWatcher()
        if err != nil {
-               t.Fatalf("NewWatcher() failed: %s", err)
+               t.Fatalf("NewWatcher failed: %s", err)
        }
 
-       t.Logf("NEEDS TO BE CONVERTED TO NEW GO TOOL") // TODO
-       return
+       dir, err := ioutil.TempDir("", "inotify")
+       if err != nil {
+               t.Fatalf("TempDir failed: %s", err)
+       }
+       defer os.RemoveAll(dir)
 
        // Add a watch for "_test"
-       err = watcher.Watch("_test")
+       err = watcher.Watch(dir)
        if err != nil {
-               t.Fatalf("Watcher.Watch() failed: %s", err)
+               t.Fatalf("Watch failed: %s", err)
        }
 
        // Receive errors on the error channel on a separate goroutine
@@ -35,7 +39,7 @@ func TestInotifyEvents(t *testing.T) {
                }
        }()
 
-       const testFile string = "_test/TestInotifyEvents.testfile"
+       testFile := dir + "/TestInotifyEvents.testfile"
 
        // Receive events on the event channel on a separate goroutine
        eventstream := watcher.Event
@@ -58,7 +62,7 @@ func TestInotifyEvents(t *testing.T) {
        // This should add at least one event to the inotify event queue
        _, err = os.OpenFile(testFile, os.O_WRONLY|os.O_CREATE, 0666)
        if err != nil {
-               t.Fatalf("creating test file failed: %s", err)
+               t.Fatalf("creating test file: %s", err)
        }
 
        // We expect this event to be received almost immediately, but let's wait 1 s to be sure
@@ -95,7 +99,7 @@ func TestInotifyClose(t *testing.T) {
                t.Fatal("double Close() test failed: second Close() call didn't return")
        }
 
-       err := watcher.Watch("_test")
+       err := watcher.Watch(os.TempDir())
        if err == nil {
                t.Fatal("expected error on Watch() after Close(), got nil")
        }