codecov

Author: Shaddoll

internal/common/auth/service_wrapper_test.go

@@ -763,3 +763,19 @@ func (s *serviceWrapperSuite) TestFailoverDomainInvalidToken() {
 	_, err := sw.FailoverDomain(ctx, &shared.FailoverDomainRequest{})
 	s.EqualError(err, "error")
 }
+
+func (s *serviceWrapperSuite) TestListFailoverHistoryValidToken() {
+	s.Service.EXPECT().ListFailoverHistory(gomock.Any(), gomock.Any(), gomock.Any()).Times(1)
+	sw := NewWorkflowServiceWrapper(s.Service, s.AuthProvider)
+	ctx, _ := thrift.NewContext(time.Minute)
+	_, err := sw.ListFailoverHistory(ctx, &shared.ListFailoverHistoryRequest{})
+	s.NoError(err)
+}
+
+func (s *serviceWrapperSuite) TestListFailoverHistoryInvalidToken() {
+	s.AuthProvider = newJWTAuthIncorrect()
+	sw := NewWorkflowServiceWrapper(s.Service, s.AuthProvider)
+	ctx, _ := thrift.NewContext(time.Minute)
+	_, err := sw.ListFailoverHistory(ctx, &shared.ListFailoverHistoryRequest{})
+	s.EqualError(err, "error")
+}

internal/common/isolationgroup/service_wrapper_test.go

@@ -415,6 +415,15 @@ func TestAPICalls(t *testing.T) {
 			},
 			expectedResponse: &shared.FailoverDomainResponse{},
 		},
+		"ListFailoverHistory": {
+			action: func(ctx context.Context, sw workflowserviceclient.Interface) (interface{}, error) {
+				return sw.ListFailoverHistory(ctx, &shared.ListFailoverHistoryRequest{})
+			},
+			affordance: func(m *workflowservicetest.MockClient) {
+				m.EXPECT().ListFailoverHistory(gomock.Any(), &shared.ListFailoverHistoryRequest{}, gomock.Any()).Times(1).Return(&shared.ListFailoverHistoryResponse{}, nil)
+			},
+			expectedResponse: &shared.ListFailoverHistoryResponse{},
+		},
 	}
 
 	for name, td := range tests {

internal/convert_test.go

@@ -100,6 +100,11 @@ func TestConvertActiveClusterSelectionPolicy(t *testing.T) {
 			policy:       nil,
 			thriftPolicy: nil,
 		},
+		{
+			name:         "empty policy",
+			policy:       &ActiveClusterSelectionPolicy{},
+			thriftPolicy: &s.ActiveClusterSelectionPolicy{},
+		},
 		{
 			name: "valid policy",
 			policy: &ActiveClusterSelectionPolicy{