Skip to content

Commit

Permalink
Add tests for reset_sticky_tasklist.go (cadence-workflow#6097)
Browse files Browse the repository at this point in the history
  • Loading branch information
natemort authored and timl3136 committed Jun 4, 2024
1 parent fe7fb4a commit 01e0e44
Showing 1 changed file with 138 additions and 0 deletions.
138 changes: 138 additions & 0 deletions service/history/engine/engineimpl/reset_sticky_tasklist_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,138 @@
// The MIT License (MIT)

// Copyright (c) 2017-2020 Uber Technologies Inc.

// Permission is hereby granted, free of charge, to any person obtaining a copy
// of this software and associated documentation files (the "Software"), to deal
// in the Software without restriction, including without limitation the rights
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
// copies of the Software, and to permit persons to whom the Software is
// furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included in all
// copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
// SOFTWARE.

package engineimpl

import (
ctx "context"
"testing"

"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/mock"

"github.com/uber/cadence/common/checksum"
"github.com/uber/cadence/common/persistence"
"github.com/uber/cadence/common/types"
"github.com/uber/cadence/service/history/constants"
"github.com/uber/cadence/service/history/engine/testdata"
"github.com/uber/cadence/service/history/workflow"
)

func TestResetStickyTaskList(t *testing.T) {
execution := &types.WorkflowExecution{
WorkflowID: constants.TestWorkflowID,
RunID: constants.TestRunID,
}
cases := []struct {
name string
request *types.HistoryResetStickyTaskListRequest
init func(engine *testdata.EngineForTest)
assertions func(engine *testdata.EngineForTest)
expectedErr error
}{
{
name: "Invalid Domain",
request: &types.HistoryResetStickyTaskListRequest{
DomainUUID: "",
Execution: execution,
},
expectedErr: &types.BadRequestError{Message: "Missing domain UUID."},
},
{
name: "Completed Workflow",
request: &types.HistoryResetStickyTaskListRequest{
DomainUUID: constants.TestDomainID,
Execution: execution,
},
init: func(engine *testdata.EngineForTest) {
engine.ShardCtx.Resource.ExecutionMgr.On("GetWorkflowExecution", mock.Anything, mock.MatchedBy(func(req *persistence.GetWorkflowExecutionRequest) bool {
return req.Execution == *execution
})).Return(&persistence.GetWorkflowExecutionResponse{
State: &persistence.WorkflowMutableState{
ExecutionInfo: &persistence.WorkflowExecutionInfo{
DomainID: constants.TestDomainID,
WorkflowID: execution.WorkflowID,
RunID: execution.RunID,
State: persistence.WorkflowStateCompleted,
},
ExecutionStats: &persistence.ExecutionStats{},
Checksum: checksum.Checksum{},
},
}, nil)
},
expectedErr: workflow.ErrAlreadyCompleted,
},
{
name: "Success",
request: &types.HistoryResetStickyTaskListRequest{
DomainUUID: constants.TestDomainID,
Execution: execution,
},
init: func(engine *testdata.EngineForTest) {
engine.ShardCtx.Resource.ExecutionMgr.On("GetWorkflowExecution", mock.Anything, mock.MatchedBy(func(req *persistence.GetWorkflowExecutionRequest) bool {
return req.Execution == *execution
})).Return(&persistence.GetWorkflowExecutionResponse{
State: &persistence.WorkflowMutableState{
ExecutionInfo: &persistence.WorkflowExecutionInfo{
DomainID: constants.TestDomainID,
WorkflowID: execution.WorkflowID,
RunID: execution.RunID,
StickyTaskList: "CLEAR ME PLEASE",
},
ExecutionStats: &persistence.ExecutionStats{},
Checksum: checksum.Checksum{},
},
}, nil)
engine.ShardCtx.Resource.ExecutionMgr.On("UpdateWorkflowExecution", mock.Anything, mock.MatchedBy(func(req *persistence.UpdateWorkflowExecutionRequest) bool {
return req.UpdateWorkflowMutation.ExecutionInfo.WorkflowID == execution.WorkflowID &&
req.UpdateWorkflowMutation.ExecutionInfo.RunID == execution.RunID &&
req.UpdateWorkflowMutation.ExecutionInfo.StickyTaskList == ""
})).Return(&persistence.UpdateWorkflowExecutionResponse{}, nil)
},
},
}

for _, testCase := range cases {
t.Run(testCase.name, func(t *testing.T) {
eft := testdata.NewEngineForTest(t, NewEngineWithShardContext)

if testCase.init != nil {
testCase.init(eft)
}
eft.Engine.Start()
result, err := eft.Engine.ResetStickyTaskList(ctx.Background(), testCase.request)

if testCase.assertions != nil {
testCase.assertions(eft)
}
eft.Engine.Stop()

if testCase.expectedErr == nil {
assert.NoError(t, err)
assert.Equal(t, &types.HistoryResetStickyTaskListResponse{}, result)
} else {
assert.Equal(t, testCase.expectedErr, err)
assert.Nil(t, result)
}
})
}
}

0 comments on commit 01e0e44

Please sign in to comment.