move check for funcs to StartWorker so that they are not checked for …

…IE api starters/stoppers

redisconsistent/config.go

@@ -4,7 +4,6 @@ import (
 	"time"
 
 	"github.com/google/uuid"
-	"github.com/pkg/errors"
 	"github.com/redis/go-redis/v9"
 
 	"github.com/streemtech/divider"
@@ -152,18 +151,6 @@ func New(client redis.UniversalClient, rootKey string, Opts ...DividerOpt) (divi
 		v(conf)
 	}
 
-	if conf.starter == nil {
-		return nil, errors.New("missing starter func")
-	}
-
-	if conf.stopper == nil {
-		return nil, errors.New("missing stopper func")
-	}
-
-	if conf.workFetcher == nil {
-		return nil, errors.New("missing work fetcher func")
-	}
-
 	return &dividerWorker{
 		conf:      *conf,
 		client:    client,

redisconsistent/redis.go

@@ -24,6 +24,17 @@ func (d *dividerWorker) StartWorker(ctx context.Context) {
 		return
 	}
 
+	//can not start worker if expected functions are nil.
+	if d.conf.starter == nil {
+		panic("missing starter func")
+	}
+	if d.conf.stopper == nil {
+		panic("missing stopper func")
+	}
+	if d.conf.workFetcher == nil {
+		panic("missing work fetcher func")
+	}
+
 	d.ctx, d.cancel = context.WithCancel(context.WithoutCancel(ctx))
 
 	var logger divider.LoggerGen