grpctest: defer Teardown to guarantee it is run even after t.Fatal (#…

…2562)
grpc · Jan 14, 2019 · f647b6f · f647b6f
1 parent 9e7c146
commit f647b6f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/internal/grpctest/grpctest.go b/internal/grpctest/grpctest.go
@@ -61,8 +61,9 @@ func RunSubTests(t *testing.T, x interface{}) {
 		tfunc := getTestFunc(t, xv, methodName)
 		t.Run(strings.TrimPrefix(methodName, "Test"), func(t *testing.T) {
 			setup(t)
+			// defer teardown to guarantee it is run even if tfunc uses t.Fatal()
+			defer teardown(t)
 			tfunc(t)
-			teardown(t)
 		})
 	}
 }