[chore][cmd/opampsupervisor]: add precheck step in Supervisor bootstrap test (open-telemetry#39292)

mmorel-35 · Fiery-Fenix · commit 0d544528927b · 2025-04-24T12:28:02.000+03:00
#### Description add precheck step in Supervisor bootstrap test #### Link to tracking issue Fixes open-telemetry#39283 Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
diff --git a/cmd/opampsupervisor/e2e_test.go b/cmd/opampsupervisor/e2e_test.go
@@ -626,6 +626,8 @@ func TestSupervisorBootstrapsCollector(t *testing.T) {
 		{
 			name: "With service.AllowNoPipelines",
 			cfg:  "nocap",
+			precheck: func(t *testing.T) {
+			},
 		},
 		{
 			name: "Without service.AllowNoPipelines",
@@ -643,6 +645,7 @@ func TestSupervisorBootstrapsCollector(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
+			tt.precheck(t)
 			agentDescription := atomic.Value{}
 
 			// Load the Supervisor config so we can get the location of

Original file line number	Diff line number	Diff line change
`@@ -626,6 +626,8 @@ func TestSupervisorBootstrapsCollector(t *testing.T) {`
`626`	`626`	`{`
`627`	`627`	`name: "With service.AllowNoPipelines",`
`628`	`628`	`cfg: "nocap",`
	`629`	`+ precheck: func(t *testing.T) {`
	`630`	`+ },`
`629`	`631`	`},`
`630`	`632`	`{`
`631`	`633`	`name: "Without service.AllowNoPipelines",`
`@@ -643,6 +645,7 @@ func TestSupervisorBootstrapsCollector(t *testing.T) {`
`643`	`645`
`644`	`646`	`for _, tt := range tests {`
`645`	`647`	`t.Run(tt.name, func(t *testing.T) {`
	`648`	`+ tt.precheck(t)`
`646`	`649`	`agentDescription := atomic.Value{}`
`647`	`650`
`648`	`651`	`// Load the Supervisor config so we can get the location of`