Apply suggestions from code review

tabito-hara · ewbankkit · web-flow · commit 1db74e3bc6a5 · 2025-09-03T06:00:49.000+09:00
Co-authored-by: Kit Ewbank &lt;Kit_Ewbank@hotmail.com&gt;
diff --git a/internal/service/synthetics/canary.go b/internal/service/synthetics/canary.go
@@ -396,6 +396,8 @@ func resourceCanaryRead(ctx context.Context, d *schema.ResourceData, meta any) d
 	d.Set("artifact_s3_location", canary.ArtifactS3Location)
 	if len(canary.EngineConfigs) > 0 {
 		d.Set("engine_arn", canary.EngineConfigs[0].EngineArn)
+	} else {
+		d.Set("engine_arn", canary.EngineArn)
 	}
 	d.Set(names.AttrExecutionRoleARN, canary.ExecutionRoleArn)
 	d.Set("failure_retention_period", canary.FailureRetentionPeriodInDays)

Original file line number	Diff line number	Diff line change
`@@ -396,6 +396,8 @@ func resourceCanaryRead(ctx context.Context, d *schema.ResourceData, meta any) d`
`396`	`396`	`d.Set("artifact_s3_location", canary.ArtifactS3Location)`
`397`	`397`	`if len(canary.EngineConfigs) > 0 {`
`398`	`398`	`d.Set("engine_arn", canary.EngineConfigs[0].EngineArn)`
	`399`	`+ } else {`
	`400`	`+ d.Set("engine_arn", canary.EngineArn)`
`399`	`401`	`}`
`400`	`402`	`d.Set(names.AttrExecutionRoleARN, canary.ExecutionRoleArn)`
`401`	`403`	`d.Set("failure_retention_period", canary.FailureRetentionPeriodInDays)`