Don't throw error during uninstall if file doesn't exist (#708)

typicode · Apr 9, 2020 · b1d4c32 · b1d4c32
1 parent 4e1d440
commit b1d4c32
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 2 deletions.
diff --git a/src/installer/__tests__/index.ts b/src/installer/__tests__/index.ts
@@ -236,5 +236,14 @@ describe('install', (): void => {
     expectHookToExist('.git/hooks/pre-commit')
   })
 
+  it('should not throw error during uninstall if some files do not exist', (): void => {
+    writeFile('package.json', pkg)
+
+    install()
+    fs.unlinkSync(path.join(tempDir, '.git/hooks/pre-commit'))
+    fs.unlinkSync(path.join(tempDir, '.git/hooks/husky.sh'))
+
+    expect(uninstall).not.toThrow()
+  })
   hooksManagers.forEach(testMigration)
 })
diff --git a/src/installer/localScript.ts b/src/installer/localScript.ts
@@ -26,5 +26,8 @@ export function createLocalScript(
 }
 
 export function removeLocalScript(gitHooksDir: string): void {
-  fs.unlinkSync(path.join(gitHooksDir, 'husky.local.sh'))
+  const filename = path.join(gitHooksDir, 'husky.local.sh')
+  if (fs.existsSync(filename)) {
+    fs.unlinkSync(filename)
+  }
 }
diff --git a/src/installer/mainScript.ts b/src/installer/mainScript.ts
@@ -18,5 +18,8 @@ export function createMainScript(gitHooksDir: string): void {
 }
 
 export function removeMainScript(gitHooksDir: string): void {
-  fs.unlinkSync(path.join(gitHooksDir, 'husky.sh'))
+  const filename = path.join(gitHooksDir, 'husky.sh')
+  if (fs.existsSync(filename)) {
+    fs.unlinkSync(filename)
+  }
 }