don't call Fatal from a goroutine

Apparently, it's not thread-safe.
ipfs · Mar 18, 2018 · 9160bdc · 9160bdc
1 parent 49a26f3
commit 9160bdc
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/command_test.go b/command_test.go
@@ -383,9 +383,10 @@ func TestCancel(t *testing.T) {
 	go func() {
 		err := re.Emit("abc")
 		if err != context.Canceled {
-			t.Fatalf("re:  expected context.Canceled but got %v", err)
+			t.Errorf("re:  expected context.Canceled but got %v", err)
+		} else {
+			t.Log("re.Emit err:", err)
 		}
-		t.Log("re.Emit err:", err)
 		re.Close()
 		close(wait)
 	}()
@@ -394,8 +395,9 @@ func TestCancel(t *testing.T) {
 
 	_, err = res.Next()
 	if err != context.Canceled {
-		t.Fatalf("res: expected context.Canceled but got %v", err)
+		t.Errorf("res: expected context.Canceled but got %v", err)
+	} else {
+		t.Log("res.Emit err:", err)
 	}
-	t.Log("res.Emit err:", err)
 	<-wait
 }