Skip to content

Commit eafb345

Browse files
committed
Tests for AppendHistoryV2Events
1 parent ca70959 commit eafb345

File tree

1 file changed

+63
-0
lines changed

1 file changed

+63
-0
lines changed

service/history/shard/context_test.go

Lines changed: 63 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -486,6 +486,69 @@ func (s *contextTestSuite) TestCreateWorkflowExecution() {
486486
}
487487
}
488488

489+
func (s *contextTestSuite) TestAppendHistoryV2Events() {
490+
cases := []struct {
491+
name string
492+
err error
493+
domainLookupErr error
494+
response *persistence.AppendHistoryNodesResponse
495+
setup func()
496+
asserts func(*persistence.AppendHistoryNodesResponse, error)
497+
}{
498+
{
499+
name: "Success",
500+
response: &persistence.AppendHistoryNodesResponse{},
501+
asserts: func(response *persistence.AppendHistoryNodesResponse, err error) {
502+
s.NoError(err)
503+
s.NotNil(response)
504+
},
505+
},
506+
{
507+
name: "Domain lookup failed",
508+
domainLookupErr: assert.AnError,
509+
asserts: func(resp *persistence.AppendHistoryNodesResponse, err error) {
510+
s.ErrorIs(err, assert.AnError)
511+
},
512+
},
513+
{
514+
name: "History too big",
515+
response: &persistence.AppendHistoryNodesResponse{
516+
DataBlob: persistence.DataBlob{
517+
Data: make([]byte, historySizeLogThreshold+1),
518+
},
519+
},
520+
asserts: func(resp *persistence.AppendHistoryNodesResponse, err error) {
521+
// We do not err on history too big here, we just log it
522+
s.NoError(err)
523+
s.NotNil(resp)
524+
},
525+
},
526+
}
527+
528+
for _, tc := range cases {
529+
s.Run(tc.name, func() {
530+
// Need setup the suite manually, since we are in a subtest
531+
s.SetupTest()
532+
ctx := context.Background()
533+
request := &persistence.AppendHistoryNodesRequest{}
534+
workflowExecution := types.WorkflowExecution{
535+
WorkflowID: testWorkflowID,
536+
RunID: testWorkflowID,
537+
}
538+
539+
s.mockResource.DomainCache.EXPECT().GetDomainName(testDomainID).Return(testDomain, tc.domainLookupErr)
540+
if tc.setup != nil {
541+
tc.setup()
542+
}
543+
544+
s.mockResource.HistoryMgr.On("AppendHistoryNodes", ctx, mock.Anything).Once().Return(tc.response, tc.err)
545+
546+
resp, err := s.context.AppendHistoryV2Events(ctx, request, testDomainID, workflowExecution)
547+
tc.asserts(resp, err)
548+
})
549+
}
550+
}
551+
489552
func (s *contextTestSuite) TestValidateAndUpdateFailoverMarkers() {
490553
domainFailoverVersion := 100
491554
domainCacheEntryInactiveCluster := cache.NewGlobalDomainCacheEntryForTest(

0 commit comments

Comments
 (0)