Fix pushing multiple experiments (#4028)

Author: mattseddon

extension/src/cli/dvc/executor.test.ts

@@ -318,6 +318,23 @@ describe('CliExecutor', () => {
         executable: 'dvc'
       })
     })
+
+    it('should call createProcess with the correct parameters to push multiple experiments to the remote', async () => {
+      const cwd = __dirname
+      const stdout = ''
+      mockedCreateProcess.mockReturnValueOnce(getMockedProcess(stdout))
+      const mockExperimentIds = ['toric-sail', 'couth-bean', 'arced-ibex']
+
+      const output = await dvcExecutor.expPush(cwd, ...mockExperimentIds)
+      expect(output).toStrictEqual(stdout)
+
+      expect(mockedCreateProcess).toHaveBeenCalledWith({
+        args: ['exp', 'push', 'origin', ...mockExperimentIds],
+        cwd,
+        env: mockedEnv,
+        executable: 'dvc'
+      })
+    })
   })
 
   describe('expRemove', () => {

extension/src/cli/dvc/executor.ts

@@ -80,12 +80,12 @@ export class DvcExecutor extends DvcCli {
     )
   }
 
-  public expPush(cwd: string, id: string) {
+  public expPush(cwd: string, ...ids: string[]) {
     return this.executeExperimentProcess(
       cwd,
       ExperimentSubCommand.PUSH,
       DEFAULT_REMOTE,
-      id
+      ...ids
     )
   }