fix: fix nil option

rotate.go

@@ -1,12 +1,14 @@
 package logh
 
 import (
-	"io"
 	"log/slog"
 )
 
-func NewRotateJSONHandler(rollSize int, flushInterval int, opts *slog.HandlerOptions) (slog.Handler, error) {
-	var w io.Writer
+func NewRotateJSONHandler(directory string, basename string, rollSize int, opts *slog.HandlerOptions, options ...Option) (slog.Handler, error) {
+	file, err := NewRotateFile(directory, basename, rollSize, options...)
+	if err != nil {
+		return nil, err
+	}
 
-	return slog.NewJSONHandler(w, opts), nil
+	return slog.NewJSONHandler(file, opts), nil
 }

rotate_file.go

@@ -33,7 +33,9 @@ func NewRotateFile(directory string, basename string, rotateSize int, opts ...Op
 	}
 
 	for _, opt := range opts {
-		opt(rf)
+		if opt != nil {
+			opt(rf)
+		}
 	}
 
 	rf.rotate()

rotate_file_test.go

@@ -14,7 +14,7 @@ func TestNewRotateFile(t *testing.T) {
 	defer os.RemoveAll(path)
 
 	path += "/TestNewRotateFile"
-	file, err := NewRotateFile(path, "test", 10)
+	file, err := NewRotateFile(path, "test", 10, nil)
 	require.NoError(t, err)
 	require.NotNil(t, file)
 

rotate_test.go

@@ -1,7 +1,16 @@
 package logh
 
-// func TestRotateJSONHandler(t *testing.T) {
-// 	handler, err := NewRotateJSONHandler(0, 0, nil)
-// 	require.NoError(t, err)
+import (
+	"log/slog"
+	"testing"
 
-// }
+	"github.com/stretchr/testify/require"
+)
+
+func TestRotateJSONHandler(t *testing.T) {
+	handler, err := NewRotateJSONHandler("test", "test", 10, nil, nil)
+	require.NoError(t, err)
+
+	logger := slog.New(handler)
+	logger.Info("test")
+}