[extension/bearertokenauthextension] fix unittest by adding zaptest l…

…ogger

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>

extension/bearertokenauthextension/bearertokenauth.go

@@ -114,9 +114,13 @@ func (b *BearerTokenAuth) startWatcher(ctx context.Context, watcher *fsnotify.Wa
 			// SEE: https://martensson.io/go-fsnotify-and-kubernetes-configmaps/
 			if event.Op == fsnotify.Remove || event.Op == fsnotify.Chmod {
 				// remove the watcher since the file is removed
-				watcher.Remove(event.Name)
+				if err := watcher.Remove(event.Name); err != nil {
+					b.logger.Error(err.Error())
+				}
 				// add a new watcher pointing to the new symlink/file
-				watcher.Add(b.filename)
+				if err := watcher.Add(b.filename); err != nil {
+					b.logger.Error(err.Error())
+				}
 				b.refreshToken()
 			}
 			// also allow normal files to be modified and reloaded.

extension/bearertokenauthextension/bearertokenauth_test.go

@@ -24,6 +24,7 @@ import (
 
 	"github.com/stretchr/testify/assert"
 	"go.opentelemetry.io/collector/component/componenttest"
+	"go.uber.org/zap/zaptest"
 )
 
 func TestPerRPCAuth(t *testing.T) {
@@ -111,7 +112,7 @@ func TestBearerStartWatchStop(t *testing.T) {
 	cfg := createDefaultConfig().(*Config)
 	cfg.Filename = "test.token"
 
-	bauth := newBearerTokenAuth(cfg, nil)
+	bauth := newBearerTokenAuth(cfg, zaptest.NewLogger(t))
 	assert.NotNil(t, bauth)
 
 	assert.Nil(t, bauth.Start(context.Background(), componenttest.NewNopHost()))

extension/bearertokenauthextension/go.mod

@@ -11,6 +11,7 @@ require (
 )
 
 require (
+	github.com/benbjohnson/clock v1.3.0 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/gogo/protobuf v1.3.2 // indirect
 	github.com/golang/protobuf v1.5.2 // indirect