test(logging): remove deprecated observeroutput

internal/beatcmd/beat.go

@@ -97,6 +97,9 @@ type BeatParams struct {
 	// ElasticLicensed indicates whether this build of APM Server
 	// is licensed with the Elastic License v2.
 	ElasticLicensed bool
+
+	// Logger holds the logger used by the runner
+	Logger *logp.Logger
 }
 
 // NewBeat creates a new Beat.
@@ -144,6 +147,7 @@ func NewBeat(args BeatParams) (*Beat, error) {
 				Hostname:        hostname,
 				StartTime:       time.Now(),
 				EphemeralID:     ephemeralID,
+				Logger:          args.Logger,
 			},
 			Keystore:   keystore,
 			Config:     &beat.BeatConfig{Output: cfg.Output},
@@ -170,7 +174,9 @@ func (b *Beat) init() error {
 	if err := configureLogging(b.Config); err != nil {
 		return fmt.Errorf("failed to configure logging: %w", err)
 	}
-	b.Beat.Info.Logger = logp.NewLogger("")
+	if b.Info.Logger == nil {
+		b.Info.Logger = logp.NewLogger("")
+	}
 
 	// log paths values to help with troubleshooting
 	b.Info.Logger.Infof("%s", paths.Paths.String())

internal/beatcmd/beat_test.go

@@ -663,6 +663,7 @@ func newBeat(t testing.TB, configYAML string, newRunner NewRunnerFunc) *Beat {
 	beat, err := NewBeat(BeatParams{
 		NewRunner:       newRunner,
 		ElasticLicensed: true,
+		Logger:          logptest.NewTestingLogger(t, ""),
 	})
 	require.NoError(t, err)
 	return beat

internal/beatcmd/logging_test.go

@@ -27,18 +27,6 @@ import (
 	"github.com/elastic/elastic-agent-libs/logp"
 )
 
-func init() {
-	// Configure tests to log at debug level, and to send
-	// logs to logp.ObserverLogs(). It is important to not
-	// log to files by default in tests, as logp will keep
-	// the files open and prevent temporary directories from
-	// being removed on Windows.
-	logOptions = append(logOptions,
-		logp.ToObserverOutput(),
-		logp.WithLevel(logp.DebugLevel),
-	)
-}
-
 var environments = []logp.Environment{
 	logp.DefaultEnvironment,
 	logp.SystemdEnvironment,