refactor(BUX-411): defaultOptions moved right into NewTaskManager

taskmanager/options.go

@@ -9,21 +9,6 @@ import (
 // that overwrite default client options.
 type ClientOps func(c *options)
 
-// defaultClientOptions will return an clientOptions struct with the default settings
-//
-// Useful for starting with the default and then modifying as needed
-func defaultClientOptions() *options {
-	// Set the default options
-	return &options{
-		debug:           false,
-		newRelicEnabled: false,
-		taskq: &taskqOptions{
-			tasks:  make(map[string]*taskq.Task),
-			config: DefaultTaskQConfig("taskq"),
-		},
-	}
-}
-
 // WithNewRelic will enable the NewRelic wrapper
 func WithNewRelic() ClientOps {
 	return func(c *options) {

taskmanager/taskmanager.go

@@ -38,12 +38,18 @@ type (
 )
 
 // NewTaskManager creates a new client for all TaskManager functionality
-//
-// If no options are given, it will use the defaultClientOptions()
+// If no options are given, it will use local memory for the queue.
 // ctx may contain a NewRelic txn (or one will be created)
 func NewTaskManager(ctx context.Context, opts ...ClientOps) (Tasker, error) {
 	// Create a new tm with defaults
-	tm := &TaskManager{options: defaultClientOptions()}
+	tm := &TaskManager{options: &options{
+		debug:           false,
+		newRelicEnabled: false,
+		taskq: &taskqOptions{
+			tasks:  make(map[string]*taskq.Task),
+			config: DefaultTaskQConfig("taskq"),
+		},
+	}}
 
 	// Overwrite defaults with any set by user
 	for _, opt := range opts {