Remove unnecessary check in the integration test

djeer · djeer · commit 31a487789268 · 2025-12-08T08:24:12.000Z
diff --git a/test/integration_test.go b/test/integration_test.go
@@ -7810,9 +7810,7 @@ func (ts *IntegrationTestSuite) TestSideEffectSummary() {
 		ts.NoError(err)
 		attributes := event.GetMarkerRecordedEventAttributes()
 		if event.EventType == enumspb.EVENT_TYPE_MARKER_RECORDED && attributes.MarkerName == "SideEffect" {
-			if event.UserMetadata != nil && event.UserMetadata.Summary != nil {
-				ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))
-			}
+			ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))
 		}
 	}
 	ts.Equal(summaryStr, summary)
@@ -7858,9 +7856,7 @@ func (ts *IntegrationTestSuite) TestMutableSideEffectSummary() {
 		ts.NoError(err)
 		attributes := event.GetMarkerRecordedEventAttributes()
 		if event.EventType == enumspb.EVENT_TYPE_MARKER_RECORDED && attributes.MarkerName == "MutableSideEffect" {
-			if event.UserMetadata != nil && event.UserMetadata.Summary != nil {
-				ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))
-			}
+			ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))
 		}
 	}
 	ts.Equal(summaryStr, summary)

Original file line number	Diff line number	Diff line change
`@@ -7810,9 +7810,7 @@ func (ts *IntegrationTestSuite) TestSideEffectSummary() {`
`7810`	`7810`	`ts.NoError(err)`
`7811`	`7811`	`attributes := event.GetMarkerRecordedEventAttributes()`
`7812`	`7812`	`if event.EventType == enumspb.EVENT_TYPE_MARKER_RECORDED && attributes.MarkerName == "SideEffect" {`
`7813`		`- if event.UserMetadata != nil && event.UserMetadata.Summary != nil {`
`7814`		`- ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))`
`7815`		`- }`
	`7813`	`+ ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))`
`7816`	`7814`	`}`
`7817`	`7815`	`}`
`7818`	`7816`	`ts.Equal(summaryStr, summary)`
`@@ -7858,9 +7856,7 @@ func (ts *IntegrationTestSuite) TestMutableSideEffectSummary() {`
`7858`	`7856`	`ts.NoError(err)`
`7859`	`7857`	`attributes := event.GetMarkerRecordedEventAttributes()`
`7860`	`7858`	`if event.EventType == enumspb.EVENT_TYPE_MARKER_RECORDED && attributes.MarkerName == "MutableSideEffect" {`
`7861`		`- if event.UserMetadata != nil && event.UserMetadata.Summary != nil {`
`7862`		`- ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))`
`7863`		`- }`
	`7859`	`+ ts.NoError(converter.GetDefaultDataConverter().FromPayload(event.UserMetadata.Summary, &summary))`
`7864`	`7860`	`}`
`7865`	`7861`	`}`
`7866`	`7862`	`ts.Equal(summaryStr, summary)`