refactor: make locations optional in QueueManagerConfig

Author: RomainLanz

src/queue_manager.ts

@@ -28,7 +28,9 @@ class QueueManagerSingleton {
       }
     }
 
-    await Locator.registerFromGlob(config.locations)
+    if (config.locations && config.locations.length > 0) {
+      await Locator.registerFromGlob(config.locations)
+    }
 
     return this
   }
@@ -91,10 +93,6 @@ class QueueManagerSingleton {
       throw new errors.E_CONFIGURATION_ERROR(['Default adapter must be specified'])
     }
 
-    if (!config.locations || config.locations.length === 0) {
-      throw new errors.E_CONFIGURATION_ERROR(['Job locations must be specified'])
-    }
-
     if (!config.adapters[config.default]) {
       throw new errors.E_CONFIGURATION_ERROR([
         `Default adapter "${config.default}" not found in adapters configuration`,

src/types/main.ts

@@ -102,5 +102,5 @@ export interface QueueManagerConfig {
   retry?: RetryConfig
   queues?: Record<string, QueueConfig>
   worker?: WorkerConfig
-  locations: string[]
+  locations?: string[]
 }

tests/queue_manager.spec.ts

@@ -38,21 +38,6 @@ test.group('QueueManager', () => {
     }
   })
 
-  test('should validate locations presence', async ({ assert }) => {
-    assert.plan(2)
-
-    try {
-      await QueueManager.init({
-        default: 'sync',
-        adapters: { sync: sync() },
-        locations: [],
-      })
-    } catch (error) {
-      assert.instanceOf(error, errors.E_CONFIGURATION_ERROR)
-      assert.equal(error.message, 'Configuration error. Reason: Job locations must be specified')
-    }
-  })
-
   test('should validate that adapter is a function', async ({ assert }) => {
     assert.plan(2)