test: improve test coverage

test/__snapshots__/runAll.spec.js.snap

@@ -116,3 +116,31 @@ exports[`runAll should use an injected logger 1`] = `
 "
 LOG No staged files match any of provided globs."
 `;
+
+exports[`runAll should warn if the argument length is longer than what the platform can handle 1`] = `
+"
+WARN ‼  lint-staged generated an argument string of 999999 characters, and commands might not run correctly on your platform.
+It is recommended to use functions as linters and split your command based on the number of staged files. For more info, please visit:
+https://github.com/okonet/lint-staged#using-js-functions-to-customize-linter-commands
+LOG Stashing changes... [started]
+LOG Stashing changes... [skipped]
+LOG → No partially staged files found...
+LOG Running linters... [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 linters... [failed]
+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}
+}"
+`;

test/runAll.spec.js

@@ -110,6 +110,18 @@ describe('runAll', () => {
     expect(gitStashPop).toHaveBeenCalledTimes(1)
   })
 
+  it('should warn if the argument length is longer than what the platform can handle', async () => {
+    hasPartiallyStagedFiles.mockImplementationOnce(() => Promise.resolve(false))
+    getStagedFiles.mockImplementationOnce(async () => new Array(100000).fill('sample.js'))
+
+    try {
+      await runAll({ '*.js': () => 'echo "sample"' })
+    } catch (err) {
+      console.log(err)
+    }
+    expect(console.printHistory()).toMatchSnapshot()
+  })
+
   it('should skip linters and stash update but perform working copy restore if terminated', async () => {
     expect.assertions(4)
     hasPartiallyStagedFiles.mockImplementationOnce(() => Promise.resolve(true))