Skip to content

Commit 1665c6b

Browse files
committed
fix tests
1 parent cbd5e19 commit 1665c6b

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

internal/workflow_shadower_activities_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ func (s *workflowShadowerActivitiesSuite) TearDownTest() {
9090

9191
func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_Succeed() {
9292
numExecutions := 1000
93-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
93+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
9494
Executions: newTestWorkflowExecutions(numExecutions),
9595
NextPageToken: []byte{1, 2, 3},
9696
}, nil).Times(1)
@@ -113,7 +113,7 @@ func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_Succeed() {
113113

114114
func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_MinResultSize() {
115115
numExecutionsPerScan := 3
116-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
116+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
117117
Executions: newTestWorkflowExecutions(numExecutionsPerScan),
118118
NextPageToken: []byte{1, 2, 3},
119119
}, nil).Times(int(math.Ceil(float64(minScanWorkflowResultSize) / float64(numExecutionsPerScan))))
@@ -135,7 +135,7 @@ func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_MinResultSize
135135

136136
func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_CompletionTime() {
137137
activityTimeoutSeconds := int32(1)
138-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
138+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
139139
Executions: newTestWorkflowExecutions(1),
140140
NextPageToken: []byte{1, 2, 3},
141141
}, nil).MaxTimes(int(time.Duration(activityTimeoutSeconds) * time.Second / scanWorkflowWaitPeriod))
@@ -163,7 +163,7 @@ func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_CompletionTim
163163
}
164164

165165
func (s *workflowShadowerActivitiesSuite) TestScanWorkflowActivity_InvalidQuery() {
166-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(nil, &shared.BadRequestError{
166+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(nil, &shared.BadRequestError{
167167
Message: "invalid query",
168168
}).Times(1)
169169

internal/workflow_shadower_test.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ func (s *workflowShadowerSuite) TestShadowOptionsWithExcludeTypes() {
260260
shadower, err := NewWorkflowShadower(s.mockService, "testDomain", options, ReplayOptions{}, nil)
261261
s.NoError(err)
262262
s.mockService.EXPECT().
263-
ScanWorkflowExecutions(gomock.Any(), gomock.Any(), gomock.Any()).
263+
ListWorkflowExecutions(gomock.Any(), gomock.Any(), gomock.Any()).
264264
DoAndReturn(func(ctx context.Context, request *shared.ListWorkflowExecutionsRequest, opts ...interface{}) (*shared.ListWorkflowExecutionsResponse, error) {
265265
s.Equal(expectedQuery, *request.Query)
266266
return &shared.ListWorkflowExecutionsResponse{
@@ -280,7 +280,7 @@ func (s *workflowShadowerSuite) TestShadowWorkerExitCondition_ExpirationTime() {
280280
ExpirationInterval: expirationTime,
281281
}
282282

283-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
283+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
284284
Executions: newTestWorkflowExecutions(totalWorkflows),
285285
NextPageToken: nil,
286286
}, nil).Times(1)
@@ -301,7 +301,7 @@ func (s *workflowShadowerSuite) TestShadowWorkerExitCondition_MaxShadowingCount(
301301
ShadowCount: maxShadowCount,
302302
}
303303

304-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
304+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
305305
Executions: newTestWorkflowExecutions(maxShadowCount * 2),
306306
NextPageToken: []byte{1, 2, 3},
307307
}, nil).Times(1)
@@ -325,7 +325,7 @@ func (s *workflowShadowerSuite) TestShadowWorker_NormalMode() {
325325
if i == numScan-1 {
326326
scanResp.NextPageToken = nil
327327
}
328-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(scanResp, nil).Times(1)
328+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(scanResp, nil).Times(1)
329329
}
330330

331331
s.mockService.EXPECT().GetWorkflowExecutionHistory(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.GetWorkflowExecutionHistoryResponse{
@@ -349,7 +349,7 @@ func (s *workflowShadowerSuite) TestShadowWorker_ContinuousMode() {
349349
scanResp := &shared.ListWorkflowExecutionsResponse{
350350
Executions: workflowExecutions,
351351
}
352-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(scanResp, nil).Times(1)
352+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(scanResp, nil).Times(1)
353353
}
354354

355355
s.mockService.EXPECT().GetWorkflowExecutionHistory(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.GetWorkflowExecutionHistoryResponse{
@@ -379,7 +379,7 @@ func (s *workflowShadowerSuite) TestShadowWorker_ContinuousMode() {
379379

380380
func (s *workflowShadowerSuite) TestShadowWorker_ReplayFailed() {
381381
successfullyReplayed := 5
382-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
382+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
383383
Executions: newTestWorkflowExecutions(successfullyReplayed * 2),
384384
NextPageToken: []byte{1, 2, 3},
385385
}, nil).Times(1)
@@ -428,7 +428,7 @@ func (s *workflowShadowerSuite) TestShadowWorker_ExpectedReplayError() {
428428

429429
for _, test := range testCases {
430430
s.T().Run(test.msg, func(t *testing.T) {
431-
s.mockService.EXPECT().ScanWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
431+
s.mockService.EXPECT().ListWorkflowExecutions(gomock.Any(), gomock.Any(), callOptions()...).Return(&shared.ListWorkflowExecutionsResponse{
432432
Executions: newTestWorkflowExecutions(1),
433433
NextPageToken: nil,
434434
}, nil).Times(1)

0 commit comments

Comments
 (0)