[8.19] (backport #11539) Ensure we don't report output status in tests where it's irrelevant (#11565)

* Ensure we don't report output status in tests where it's irrelevant (#11539)

* Ensure we don't report output status in tests where it's irrelevant

* Extract common parameters in TestBeatDiagnostics

(cherry picked from commit b8a142f)

# Conflicts:
#	testing/integration/ess/beat_receivers_test.go

* Fix conflicts

Removed redundant configuration for elasticsearch in tests.

* Fix test

* Fix indentation for status_reporting in test file

---------

Co-authored-by: Mikołaj Świątek <[email protected]>

Author: mergify[bot]

testing/integration/ess/beat_receivers_test.go

@@ -726,6 +726,8 @@ outputs:
     type: elasticsearch
     hosts: [http://localhost:9200]
     api_key: placeholder
+    status_reporting:
+      enabled: false
 agent.monitoring.enabled: false
 `
 
@@ -866,7 +868,8 @@ outputs:
     hosts: [http://localhost:9200]
     api_key: placeholder
     indices: [] # not supported by the elasticsearch exporter
-
+    status_reporting:
+      enabled: false
   supported:
     type: elasticsearch
     hosts: [http://localhost:9200]
@@ -991,6 +994,8 @@ outputs:
     type: elasticsearch
     hosts: [http://localhost:9200]
     api_key: placeholder
+    status_reporting:
+      enabled: false
 agent.monitoring.enabled: false
 `
 
@@ -1078,6 +1083,8 @@ outputs:
     type: elasticsearch
     hosts: [http://localhost:9200]
     api_key: placeholder
+    status_reporting:
+      enabled: false
 agent.monitoring.enabled: false
 `
 	err = fixture.Configure(ctx, []byte(configNoComponents))

testing/integration/ess/diagnostics_test.go

@@ -347,6 +347,8 @@ outputs:
     type: elasticsearch
     hosts: [http://localhost:9200]
     api_key: placeholder
+    status_reporting:
+      enabled: false
 agent.monitoring.enabled: false
 agent.internal.runtime.filebeat.filestream: {{ .Runtime }}
 `
@@ -359,13 +361,25 @@ agent.internal.runtime.filebeat.filestream: {{ .Runtime }}
 
 	ctx, cancel := testcontext.WithDeadline(t, context.Background(), time.Now().Add(10*time.Minute))
 	defer cancel()
+	expectedComponentState := map[string]integrationtest.ComponentState{
+		"filestream-default": {
+			State: integrationtest.NewClientState(client.Healthy),
+			Units: map[integrationtest.ComponentUnitKey]integrationtest.ComponentUnitState{
+				integrationtest.ComponentUnitKey{UnitType: client.UnitTypeOutput, UnitID: "filestream-default"}: {
+					State: integrationtest.NewClientState(client.Healthy),
+				},
+				integrationtest.ComponentUnitKey{UnitType: client.UnitTypeInput, UnitID: "filestream-default-filestream-filebeat"}: {
+					State: integrationtest.NewClientState(client.Healthy),
+				},
+			},
+		},
+	}
+	expectedAgentState := integrationtest.NewClientState(client.Healthy)
 
 	testCases := []struct {
 		name                         string
 		runtime                      string
 		expectedCompDiagnosticsFiles []string
-		expectedAgentState           *client.State
-		expectedComponentState       map[string]integrationtest.ComponentState
 	}{
 		{
 			name:    "filebeat process",
@@ -377,20 +391,6 @@ agent.internal.runtime.filebeat.filestream: {{ .Runtime }}
 				"beat-rendered-config.yml",
 				"global_processors.txt",
 			),
-			expectedAgentState: integrationtest.NewClientState(client.Healthy),
-			expectedComponentState: map[string]integrationtest.ComponentState{
-				"filestream-default": {
-					State: integrationtest.NewClientState(client.Healthy),
-					Units: map[integrationtest.ComponentUnitKey]integrationtest.ComponentUnitState{
-						integrationtest.ComponentUnitKey{UnitType: client.UnitTypeOutput, UnitID: "filestream-default"}: {
-							State: integrationtest.NewClientState(client.Healthy),
-						},
-						integrationtest.ComponentUnitKey{UnitType: client.UnitTypeInput, UnitID: "filestream-default-filestream-filebeat"}: {
-							State: integrationtest.NewClientState(client.Healthy),
-						},
-					},
-				},
-			},
 		},
 		{
 			name:    "filebeat receiver",
@@ -400,7 +400,6 @@ agent.internal.runtime.filebeat.filestream: {{ .Runtime }}
 				"beat_metrics.json",
 				"input_metrics.json",
 			},
-			expectedAgentState: integrationtest.NewClientState(client.Degraded),
 		},
 	}
 
@@ -443,8 +442,8 @@ agent.internal.runtime.filebeat.filestream: {{ .Runtime }}
 			}
 			err = f.Run(ctx, integrationtest.State{
 				Configure:  configBuffer.String(),
-				AgentState: tc.expectedAgentState,
-				Components: tc.expectedComponentState,
+				AgentState: expectedAgentState,
+				Components: expectedComponentState,
 				After:      testDiagnosticsFactory(t, filebeatSetup, expDiagFiles, tc.expectedCompDiagnosticsFiles, f, []string{"diagnostics", "collect"}),
 			})
 			assert.NoError(t, err)
@@ -474,6 +473,8 @@ outputs:
     type: elasticsearch
     hosts: [http://localhost:9200]
     api_key: placeholder
+    status_reporting:
+      enabled: false
 agent.monitoring.enabled: false
 agent.internal.runtime.filebeat.filestream: otel
 `