Fix assertion in otel shutdown test

swiatekm · swiatekm · commit 0165a9500242 · 2025-11-04T12:44:53.000+01:00
diff --git a/testing/integration/ess/otel_test.go b/testing/integration/ess/otel_test.go
@@ -140,7 +140,9 @@ service:
 
 	// stop the collector and check that it emitted logs indicating a graceful shutdown
 	require.NoError(t, cmd.Process.Signal(os.Interrupt))
-	require.NoError(t, cmd.Wait())
+	if waitErr := cmd.Wait(); waitErr != nil {
+		assert.ErrorContains(t, waitErr, "signal: interrupt")
+	}
 	assert.Contains(t, output.String(), "Shutdown complete")
 }
 

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,9 @@ service:`
`140`	`140`
`141`	`141`	`// stop the collector and check that it emitted logs indicating a graceful shutdown`
`142`	`142`	`require.NoError(t, cmd.Process.Signal(os.Interrupt))`
`143`		`- require.NoError(t, cmd.Wait())`
	`143`	`+ if waitErr := cmd.Wait(); waitErr != nil {`
	`144`	`+ assert.ErrorContains(t, waitErr, "signal: interrupt")`
	`145`	`+ }`
`144`	`146`	`assert.Contains(t, output.String(), "Shutdown complete")`
`145`	`147`	`}`
`146`	`148`