Actually, just return the first error

Now I got thousands of:

    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor
    helpers_test.go:384: fsnotify.readEvents: bad file descriptor

That's not helpful.

diff --git a/helpers_test.go b/helpers_test.go
index 7f1de528dd3dc0c3b87072a6f1d4f57db9d63618..00f3bf468811b8c95b9c4746f002a95afd7f9158 100644 (file)
--- a/helpers_test.go
+++ b/helpers_test.go
@@ -382,6 +382,8 @@ func (w *eventCollector) collect(t *testing.T) {
                                        return
                                }
                                t.Error(e)
+                               w.done <- struct{}{}
+                               return
                        case e, ok := <-w.w.Events:
                                if !ok {
                                        w.done <- struct{}{}