test: add some runAll tests

lint-staged · Nov 14, 2019 · bbfdbce · bbfdbce
1 parent 357934f
commit bbfdbce
Show file tree

Hide file tree

Showing 2 changed files with 130 additions and 0 deletions.
diff --git a/test/__snapshots__/runAll.spec.js.snap b/test/__snapshots__/runAll.spec.js.snap
@@ -1,10 +1,88 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`runAll should not skip tasks if there are files 1`] = `
+"
+LOG Preparing... [started]
+LOG Preparing... [completed]
+LOG Running tasks... [started]
+LOG Running tasks for *.js [started]
+LOG echo \\"sample\\" [started]
+LOG echo \\"sample\\" [completed]
+LOG Running tasks for *.js [completed]
+LOG Running tasks... [completed]
+LOG Applying unstaged changes... [started]
+LOG Applying unstaged changes... [completed]
+LOG Cleaning up... [started]
+LOG Cleaning up... [completed]"
+`;
+
 exports[`runAll should resolve the promise with no files 1`] = `
 "
 LOG No staged files match any of provided globs."
 `;
 
+exports[`runAll should skip applying unstaged modifications if there are errors during linting 1`] = `
+"
+LOG Preparing... [started]
+LOG Preparing... [completed]
+LOG Running tasks... [started]
+LOG Running tasks for *.js [started]
+LOG echo \\"sample\\" [started]
+LOG echo \\"sample\\" [failed]
+LOG → 
+LOG Running tasks for *.js [failed]
+LOG → 
+LOG Running tasks... [failed]
+LOG Applying unstaged changes... [started]
+LOG Applying unstaged changes... [skipped]
+LOG → Skipped because of errors from tasks
+LOG Restoring original state due to errors... [started]
+LOG Restoring original state due to errors... [completed]
+LOG Cleaning up... [started]
+LOG Cleaning up... [completed]
+LOG {
+  name: 'ListrError',
+  errors: [
+    {
+      privateMsg: '\\\\n\\\\n\\\\n× echo \\"sample\\" found some errors. Please fix them and try committing again.\\\\n\\\\nLinter finished with error',
+      context: {hasErrors: true}
+    }
+  ],
+  context: {hasErrors: true}
+}"
+`;
+
+exports[`runAll should skip tasks and restore state if terminated 1`] = `
+"
+LOG Preparing... [started]
+LOG Preparing... [completed]
+LOG Running tasks... [started]
+LOG Running tasks for *.js [started]
+LOG echo \\"sample\\" [started]
+LOG echo \\"sample\\" [failed]
+LOG → 
+LOG Running tasks for *.js [failed]
+LOG → 
+LOG Running tasks... [failed]
+LOG Applying unstaged changes... [started]
+LOG Applying unstaged changes... [skipped]
+LOG → Skipped because of errors from tasks
+LOG Restoring original state due to errors... [started]
+LOG Restoring original state due to errors... [completed]
+LOG Cleaning up... [started]
+LOG Cleaning up... [completed]
+LOG {
+  name: 'ListrError',
+  errors: [
+    {
+      privateMsg: '\\\\n\\\\n\\\\n‼ echo \\"sample\\" was terminated with SIGINT',
+      context: {hasErrors: true}
+    }
+  ],
+  context: {hasErrors: true}
+}"
+`;
+
 exports[`runAll should use an injected logger 1`] = `
 "
 LOG No staged files match any of provided globs."

diff --git a/test/runAll.spec.js b/test/runAll.spec.js
@@ -1,4 +1,5 @@
 import makeConsoleMock from 'consolemock'
+import execa from 'execa'
 import normalize from 'normalize-path'
 
 import resolveGitDir from '../src/resolveGitDir'
@@ -63,4 +64,55 @@ describe('runAll', () => {
     await runAll({ config: { '*.js': ['echo "sample"'] }, debug: true }, logger)
     expect(logger.printHistory()).toMatchSnapshot()
   })
+
+  it('should not skip tasks if there are files', async () => {
+    expect.assertions(1)
+    getStagedFiles.mockImplementationOnce(async () => ['sample.js'])
+    await runAll({ config: { '*.js': ['echo "sample"'] } })
+    expect(console.printHistory()).toMatchSnapshot()
+  })
+
+  it('should skip applying unstaged modifications if there are errors during linting', async () => {
+    expect.assertions(1)
+    getStagedFiles.mockImplementationOnce(async () => ['sample.js'])
+    execa.mockImplementation(() =>
+      Promise.resolve({
+        stdout: '',
+        stderr: 'Linter finished with error',
+        code: 1,
+        failed: true,
+        cmd: 'mock cmd'
+      })
+    )
+
+    try {
+      await runAll({ config: { '*.js': ['echo "sample"'] } })
+    } catch (err) {
+      console.log(err)
+    }
+    expect(console.printHistory()).toMatchSnapshot()
+  })
+
+  it('should skip tasks and restore state if terminated', async () => {
+    expect.assertions(1)
+    getStagedFiles.mockImplementationOnce(async () => ['sample.js'])
+    execa.mockImplementation(() =>
+      Promise.resolve({
+        stdout: '',
+        stderr: '',
+        code: 0,
+        failed: false,
+        killed: true,
+        signal: 'SIGINT',
+        cmd: 'mock cmd'
+      })
+    )
+
+    try {
+      await runAll({ config: { '*.js': ['echo "sample"'] } })
+    } catch (err) {
+      console.log(err)
+    }
+    expect(console.printHistory()).toMatchSnapshot()
+  })
 })