[Scanner] Use CurrentExecution instead of base one (cadence-workflow#…

…3421)
mkolodezny · Jul 31, 2020 · 42444dc · 42444dc
1 parent 121ef1e
commit 42444dc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/common/reconciliation/common/util.go b/common/reconciliation/common/util.go
@@ -164,8 +164,8 @@ func ExecutionOpen(execution interface{}) bool {
 // getExecution returns base Execution
 func getExecution(execution interface{}) *Execution {
 	switch e := execution.(type) {
-	case *Execution:
-		return e
+	case *CurrentExecution:
+		return &e.Execution
 	case *ConcreteExecution:
 		return &e.Execution
 	default:

diff --git a/common/reconciliation/common/util_test.go b/common/reconciliation/common/util_test.go
@@ -435,7 +435,7 @@ func (s *UtilSuite) TestDeleteExecution() {
 			execManager.On("DeleteCurrentWorkflowExecution", mock.Anything).Return(tc.deleteCurrentErr).Once()
 		}
 		pr := NewPersistenceRetryer(execManager, nil)
-		result := DeleteExecution(&Execution{}, pr)
+		result := DeleteExecution(&ConcreteExecution{}, pr)
 		s.Equal(tc.expectedFixResult, result)
 	}
 }