Return empty WorkflowContextList from GetWorkflowContextList

Fixes #391

Signed-off-by: Steve Wills <steve@mouf.net>

grpc-server/tinkerbell.go

@@ -55,20 +55,17 @@ func (s *server) GetWorkflowContextList(ctx context.Context, req *pb.WorkflowCon
 		return nil, err
 	}
 
-	if wfs != nil {
-		wfContexts := []*pb.WorkflowContext{}
-		for _, wf := range wfs {
-			wfContext, err := s.db.GetWorkflowContexts(ctx, wf)
-			if err != nil {
-				return nil, status.Errorf(codes.Aborted, err.Error())
-			}
-			wfContexts = append(wfContexts, wfContext)
+	wfContexts := []*pb.WorkflowContext{}
+	for _, wf := range wfs {
+		wfContext, err := s.db.GetWorkflowContexts(ctx, wf)
+		if err != nil {
+			return nil, status.Errorf(codes.Aborted, err.Error())
 		}
-		return &pb.WorkflowContextList{
-			WorkflowContexts: wfContexts,
-		}, nil
+		wfContexts = append(wfContexts, wfContext)
 	}
-	return nil, nil
+	return &pb.WorkflowContextList{
+		WorkflowContexts: wfContexts,
+	}, nil
 }
 
 // GetWorkflowActions implements tinkerbell.GetWorkflowActions.

grpc-server/tinkerbell_test.go

@@ -135,13 +135,11 @@ func TestGetWorkflowContextList(t *testing.T) {
 				assert.True(t, tc.want.expectedError)
 				return
 			}
-			if err == nil && res == nil {
-				assert.False(t, tc.want.expectedError)
-				return
-			}
 			assert.NoError(t, err)
 			assert.NotNil(t, res)
-			assert.Len(t, res.WorkflowContexts, 1)
+			assert.False(t, tc.want.expectedError)
+			assert.IsType(t, &pb.WorkflowContextList{}, res)
+			assert.IsType(t, []*pb.WorkflowContext{}, res.WorkflowContexts)
 		})
 	}
 }