Improve debuggability of TestRunnerServiceServerExecute

stateful · Feb 2, 2024 · c9607c2 · c9607c2
1 parent 1314454
commit c9607c2
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/internal/runnerv2service/service_execute_test.go b/internal/runnerv2service/service_execute_test.go
@@ -230,8 +230,6 @@ func TestRunnerServiceServerExecute_Input(t *testing.T) {
 	_, client := testCreateRunnerServiceClient(t, lis)
 
 	t.Run("ContinuousInput", func(t *testing.T) {
-		t.Parallel()
-
 		stream, err := client.Execute(context.Background())
 		require.NoError(t, err)
 
@@ -268,8 +266,6 @@ func TestRunnerServiceServerExecute_Input(t *testing.T) {
 	})
 
 	t.Run("SimulateCtrlC", func(t *testing.T) {
-		t.Parallel()
-
 		stream, err := client.Execute(context.Background())
 		require.NoError(t, err)
 
@@ -314,8 +310,6 @@ func TestRunnerServiceServerExecute_Input(t *testing.T) {
 	})
 
 	t.Run("CloseSendDirection", func(t *testing.T) {
-		t.Parallel()
-
 		stream, err := client.Execute(context.Background())
 		require.NoError(t, err)
 
@@ -341,6 +335,7 @@ func TestRunnerServiceServerExecute_Input(t *testing.T) {
 
 		result := <-execResult
 		// TODO(adamb): This should be a specific gRPC error rather than Unknown.
+		require.NotNil(t, result.Err)
 		assert.Contains(t, result.Err.Error(), "signal: interrupt")
 		assert.Equal(t, 130, result.ExitCode)
 	})