[CLI] fix nil pointer issue in domain migration command rendering (ub…

…er#5378)

What changed?

correct empty response
refactor long running checker a little and add long running definition in the output

Why?
Empty response should include Value field otherwise it's going to cause nil pointer error in the rendering.

How did you test it?
tested locally
---------

Co-authored-by: David Porter <david.porter@uber.com>

tools/cli/domainMigrationCommand.go

@@ -41,7 +41,13 @@ import (
 	"github.com/uber/cadence/common/types"
 )
 
-var domainMigrationTemplate = `Validation Check:
+const (
+	// workflows running longer than 14 days are considered long running workflows
+	longRunningDuration = 14 * 24 * time.Hour
+)
+
+var (
+	domainMigrationTemplate = `Validation Check:
 {{- range .}}
 - {{.ValidationCheck}}: {{.ValidationResult}}
 {{- with .ValidationDetails}}
@@ -59,7 +65,7 @@ var domainMigrationTemplate = `Validation Check:
     Mismatched Domain Meta Data: {{.MismatchedDomainMetaData}}
   {{- end }}
   {{- if .LongRunningWorkFlowNum}}
-    Long Running Workflow Num: {{.LongRunningWorkFlowNum}}
+    Long Running Workflow Num (> 14 days): {{.LongRunningWorkFlowNum}}
   {{- end}}
   {{- if .MissingCurrSearchAttributes}}
     Missing Search Attributes in Current Domain:
@@ -96,6 +102,12 @@ var domainMigrationTemplate = `Validation Check:
   {{- end}}
 {{- end}}
 `
+	emptyGetDynamicConfigRequest = &types.GetDynamicConfigResponse{
+		Value: &types.DataBlob{
+			EncodingType: types.EncodingTypeJSON.Ptr(),
+		},
+	}
+)
 
 type domainMigrationCLIImpl struct {
 	frontendClient, destinationClient           frontend.Client
@@ -196,11 +208,10 @@ func (d *domainMigrationCLIImpl) migrationDomainWorkFlowCheck(c *cli.Context) Do
 
 func (d *domainMigrationCLIImpl) countLongRunningWorkflow(c *cli.Context) int {
 	domain := c.GlobalString(FlagDomain)
-	now := time.Now()
-	past14Days := now.Add(-14 * 24 * time.Hour)
+	thresholdOfLongRunning := time.Now().Add(-longRunningDuration)
 	request := &types.CountWorkflowExecutionsRequest{
 		Domain: domain,
-		Query:  "CloseTime=missing AND StartTime < " + strconv.FormatInt(past14Days.UnixNano(), 10),
+		Query:  "CloseTime=missing AND StartTime < " + strconv.FormatInt(thresholdOfLongRunning.UnixNano(), 10),
 	}
 	ctx, cancel := newContextForLongPoll(c)
 	defer cancel()
@@ -332,12 +343,12 @@ func (d *domainMigrationCLIImpl) migrationDynamicConfigCheck(c *cli.Context) Dom
 			currResp, err := d.frontendAdminClient.GetDynamicConfig(ctx, currRequest)
 			if err != nil {
 				// empty to indicate N/A
-				currResp = &types.GetDynamicConfigResponse{}
+				currResp = emptyGetDynamicConfigRequest
 			}
 			newResp, err := d.destinationAdminClient.GetDynamicConfig(ctx, newRequest)
 			if err != nil {
 				// empty to indicate N/A
-				newResp = &types.GetDynamicConfigResponse{}
+				newResp = emptyGetDynamicConfigRequest
 			}
 
 			if !reflect.DeepEqual(currResp.Value, newResp.Value) {
@@ -370,12 +381,12 @@ func (d *domainMigrationCLIImpl) migrationDynamicConfigCheck(c *cli.Context) Dom
 			currResp, err := d.frontendAdminClient.GetDynamicConfig(ctx, currRequest)
 			if err != nil {
 				// empty to indicate N/A
-				currResp = &types.GetDynamicConfigResponse{}
+				currResp = emptyGetDynamicConfigRequest
 			}
 			newResp, err := d.destinationAdminClient.GetDynamicConfig(ctx, newRequest)
 			if err != nil {
 				// empty to indicate N/A
-				newResp = &types.GetDynamicConfigResponse{}
+				newResp = emptyGetDynamicConfigRequest
 			}
 
 			if !reflect.DeepEqual(currResp.Value, newResp.Value) {
@@ -415,12 +426,12 @@ func (d *domainMigrationCLIImpl) migrationDynamicConfigCheck(c *cli.Context) Dom
 				currResp, err := d.frontendAdminClient.GetDynamicConfig(ctx, currRequest)
 				if err != nil {
 					// empty to indicate N/A
-					currResp = &types.GetDynamicConfigResponse{}
+					currResp = emptyGetDynamicConfigRequest
 				}
 				newResp, err := d.destinationAdminClient.GetDynamicConfig(ctx, newRequest)
 				if err != nil {
 					// empty to indicate N/A
-					newResp = &types.GetDynamicConfigResponse{}
+					newResp = emptyGetDynamicConfigRequest
 				}
 
 				if !reflect.DeepEqual(currResp.Value, newResp.Value) {