fix: js error when duplicating requests with no args

Author: gyoshev

src/MigrationManager.ts

@@ -97,6 +97,10 @@ export default class MigrationManager {
       return false
     }
 
+    if (!args) {
+      args = new ArgsBuilder()
+    }
+
     if (state === MigrationState.DUPLICATE) {
       args.add(Constants.PARAMS.CT, true)
     }

test/specs/MigrationManager.test.ts

@@ -77,7 +77,7 @@ describe(MigrationManager, () => {
     })
   })
 
-  describe('modifyRequest', () => {
+  describe('duplicateRequest', () => {
     it('adds ct parameter when duplicating requests', () => {
       setMigrationState(DUPLICATE)
 
@@ -100,6 +100,14 @@ describe(MigrationManager, () => {
       expect(suppress).toBe(false)
     })
 
+    it('does not crash when request has no args', () => {
+      setMigrationState(DUPLICATE)
+
+      const suppress = manager.duplicateRequest('getNewsfeedMessages', undefined, {})
+
+      expect(suppress).toBe(false)
+    })
+
     it('returns true if request should be suppressed', () => {
       setMigrationState(CLEVERTAP)