test: update server logger behavior to use default logger when nil

Updates the server to use a default logger instead of panicking when the provided logger is nil. Adjusts corresponding tests to validate the new behavior, ensuring `nil logger should not panic`. Includes default configuration initialization in `server.go`.

Author: iagapie

server/server.go

@@ -35,6 +35,10 @@ func New(cfg Config, handler http.Handler, logger *slog.Logger) *Server {
 		logger = slog.New(slog.DiscardHandler)
 	}
 
+	logger = logger.WithGroup("server")
+
+	cfg.SetDefaults()
+
 	h2s := &http2.Server{
 		MaxConcurrentStreams: cfg.HTTP2.MaxConcurrentStreams,
 		MaxReadFrameSize:     cfg.HTTP2.MaxReadFrameSize,

server/server_test.go

@@ -85,8 +85,8 @@ func TestNewServer(t *testing.T) {
 		})
 	})
 
-	t.Run("nil logger should panic", func(t *testing.T) {
-		assert.Panics(t, func() {
+	t.Run("nil logger should not panic", func(t *testing.T) {
+		assert.NotPanics(t, func() {
 			New(Config{}, handler, nil)
 		})
 	})