test: add test for resolveGitDir behaviour outside a git directory

lint-staged · Jun 6, 2019 · ed9e586 · ed9e586
1 parent c87671f
commit ed9e586
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/resolveGitDir.js b/src/resolveGitDir.js
@@ -4,9 +4,9 @@ const execGit = require('./execGit')
 const path = require('path')
 const printErrors = require('./printErrors')
 
-module.exports = async function resolveGitDir() {
+module.exports = async function resolveGitDir(options) {
   try {
-    const gitDir = await execGit(['rev-parse', '--show-toplevel'])
+    const gitDir = await execGit(['rev-parse', '--show-toplevel'], options)
     return path.normalize(gitDir)
   } catch (error) {
     printErrors(error)

diff --git a/test/resolveGitDir.spec.js b/test/resolveGitDir.spec.js
@@ -20,4 +20,9 @@ describe('resolveGitDir', () => {
     expect(path.resolve(await resolveGitDir())).toEqual(expected)
     process.cwd = processCwdBkp
   })
+
+  it('should return null when not in a git directory', async () => {
+    const gitDir = await resolveGitDir({ cwd: '/' }) // assume root is not a git directory
+    expect(gitDir).toEqual(null)
+  })
 })