fix(jasmine): createSpyObj calls with missing (optional) baseName arg…

…ument should be handled (#608)

src/transformers/jasmine-globals.test.ts

@@ -355,6 +355,33 @@ describe('createSpyObj', () => {
     `
     )
   })
+
+  test('without base name and methods names only', () => {
+    expectTransformation(
+      `
+    const spyObj = jasmine.createSpyObj(['someMethod']);
+    `,
+      `
+    const spyObj = {
+        'someMethod': jest.fn()
+    };
+    `
+    )
+  })
+
+  test('without base name and methods names as well as properties object', () => {
+    expectTransformation(
+      `
+    const spyObj = jasmine.createSpyObj(['someMethod'], {property: true});
+    `,
+      `
+    const spyObj = {
+        'someMethod': jest.fn(),
+        'property': true
+    };
+    `
+    )
+  })
 })
 
 test('return value', () => {

src/transformers/jasmine-globals.ts

@@ -614,13 +614,21 @@ export default function jasmineGlobals(fileInfo, api, options) {
     })
     .filter((path) => {
       const args = path.node.arguments
+      const isArrayOrObjectExpression = (arg) =>
+        arg.type === 'ArrayExpression' || arg.type === 'ObjectExpression'
+
+      const firstArgumentIsMethodNames = isArrayOrObjectExpression(args[0])
+      if (firstArgumentIsMethodNames) {
+        // ensure that optional baseName is always filled, to simplify arguments handling
+        args.unshift(j.literal(''))
+      }
+
+      const [, spyObjMethods, spyObjProperties] = args
 
       return (
         (args.length === 2 || args.length === 3) &&
-        (args[1].type === 'ArrayExpression' || args[1].type === 'ObjectExpression') &&
-        (args[2] === undefined ||
-          args[2].type === 'ArrayExpression' ||
-          args[2].type === 'ObjectExpression')
+        isArrayOrObjectExpression(spyObjMethods) &&
+        (spyObjProperties === undefined || isArrayOrObjectExpression(spyObjProperties))
       )
     })
     .forEach((path) => {