fix(shipit-deploy): skip fetching git in case when repositoryUrl was not provided (closes #207) (#226)

Author: SleepWalker

packages/shipit-deploy/src/tasks/deploy/fetch.js

@@ -22,7 +22,7 @@ const fetchTask = shipit => {
         shipit.log('Create workspace...')
         /* eslint-disable no-param-reassign */
         if (shipit.config.shallowClone) {
-          const tmpDir = await tmp.dir({mode: "0755"})
+          const tmpDir = await tmp.dir({ mode: '0755' })
           shipit.workspace = tmpDir.path
         } else {
           shipit.workspace = shipit.config.workspace
@@ -174,14 +174,20 @@ const fetchTask = shipit => {
     }
 
     await createWorkspace()
-    await initRepository()
-    await setGitConfig()
-    await addRemote()
-    await fetch()
-    await checkout()
-    await reset()
-    await merge()
-    await updateSubmodules()
+
+    if (shipit.config.repositoryUrl) {
+      await initRepository()
+      await setGitConfig()
+      await addRemote()
+      await fetch()
+      await checkout()
+      await reset()
+      await merge()
+      await updateSubmodules()
+    } else {
+      shipit.log(chalk.yellow('Skip fetching repo. No repositoryUrl provided'))
+    }
+
     shipit.emit('fetched')
   })
 }

packages/shipit-deploy/src/tasks/deploy/fetch.test.js

@@ -6,11 +6,13 @@ jest.mock('tmp-promise')
 
 describe('deploy:fetch task', () => {
   let shipit
+  let log
 
   beforeEach(() => {
+    log = jest.fn()
     shipit = new Shipit({
       environment: 'test',
-      log: jest.fn(),
+      log,
     })
 
     fetchTask(shipit)
@@ -129,4 +131,15 @@ describe('deploy:fetch task', () => {
     expect(shipit.local).toBeCalledWith('git checkout master', opts)
     expect(shipit.local).toBeCalledWith('git branch --list master', opts)
   })
+
+  it('should skip fetching if no repositoryUrl provided', async () => {
+    delete shipit.config.repositoryUrl
+
+    await start(shipit, 'deploy:fetch')
+
+    expect(shipit.local).not.toHaveBeenCalled()
+    expect(log).toBeCalledWith(
+      expect.stringContaining('Skip fetching repo. No repositoryUrl provided'),
+    )
+  })
 })