Fix status reporter initialization (#1341)

Author: AndersonQ

internal/pkg/core/status/reporter.go

@@ -88,10 +88,11 @@ type controller struct {
 // NewController creates a new reporter.
 func NewController(log *logger.Logger) Controller {
 	return &controller{
-		status:       Healthy,
-		reporters:    make(map[string]*reporter),
-		appReporters: make(map[string]*reporter),
-		log:          log,
+		status:         Healthy,
+		reporters:      make(map[string]*reporter),
+		localReporters: make(map[string]*reporter),
+		appReporters:   make(map[string]*reporter),
+		log:            log,
 	}
 }
 
@@ -154,12 +155,12 @@ func (r *controller) RegisterLocalComponent(componentIdentifier string) Reporter
 	return rep
 }
 
-// Register registers new component for status updates.
+// RegisterComponent registers new component for status updates.
 func (r *controller) RegisterComponent(componentIdentifier string) Reporter {
 	return r.RegisterComponentWithPersistance(componentIdentifier, false)
 }
 
-// Register registers new component for status updates.
+// RegisterComponentWithPersistance registers new component for status updates.
 func (r *controller) RegisterComponentWithPersistance(componentIdentifier string, persistent bool) Reporter {
 	id := componentIdentifier + "-" + uuid.New().String()[:8]
 	rep := &reporter{

internal/pkg/core/status/reporter_test.go

@@ -14,6 +14,24 @@ import (
 	"github.com/elastic/elastic-agent/pkg/core/logger"
 )
 
+func TestNewController_ensure_all_is_initialzed(t *testing.T) {
+	l, _ := logger.New("", false)
+
+	newController := NewController(l)
+
+	c, ok := newController.(*controller)
+	if !ok {
+		t.Fatalf("expected c %T, not c %T", controller{}, newController)
+	}
+
+	c.reporters["ignore"] = &reporter{}
+	c.localReporters["ignore"] = &reporter{}
+	c.appReporters["ignore"] = &reporter{}
+	if c.log == nil {
+		t.Error("logger shouldn't be nil, it was not correctly assigned")
+	}
+}
+
 func TestReporter(t *testing.T) {
 	l, _ := logger.New("", false)
 	t.Run("healthy by default", func(t *testing.T) {