Add support for using both the -c and -l flags

Author: titouanmathis

src/index.js

@@ -11,7 +11,6 @@ const {
   CI_CONFIG_PATH = '.gitlab-ci.yml',
   CI_JOB_NAME,
   CI_PROJECT_DIR = cwd(),
-  PRETTIER_CODE_QUALITY_REPORT,
 } = env;
 
 /**
@@ -39,9 +38,14 @@ function getOutputPath() {
  */
 function parse(results) {
   return results
-    .toString()
     .split('\n')
-    .filter((line) => line);
+    .filter(
+      (line) =>
+        Boolean(line) &&
+        !line.startsWith('Checking formatting...') &&
+        !line.includes('Code style issues found'),
+    )
+    .map((line) => line.replace('[warn] ', ''));
 }
 
 /**
@@ -50,6 +54,7 @@ function parse(results) {
  * @returns {Promise<void>}
  */
 export async function prettierFormatterGitLab(results) {
+  const { PRETTIER_CODE_QUALITY_REPORT } = env;
   if (CI_JOB_NAME || PRETTIER_CODE_QUALITY_REPORT) {
     const files = parse(results);
 

test/cli.spec.js

@@ -1,6 +1,7 @@
-import { describe, it, expect, afterEach } from 'bun:test';
+import { describe, it, expect, afterEach, spyOn } from 'bun:test';
 import { existsSync, unlinkSync, readFileSync } from 'node:fs';
 import { execSync } from 'node:child_process';
+import { prettierFormatterGitLab } from '../src/index.js';
 
 const CODE_QUALITY_FILENAME = 'gl-prettier-codequality.json';
 
@@ -24,4 +25,28 @@ describe('prettier-formatter-gitlab cli', () => {
     const content = readFileSync(CODE_QUALITY_FILENAME);
     expect(content.toString()).toMatchSnapshot();
   });
+
+  it('should work with the `-c` or `-l` flag', async () => {
+    const consoleSpy = spyOn(console, 'log');
+    consoleSpy.mockImplementation(() => null);
+    process.env.PRETTIER_CODE_QUALITY_REPORT = CODE_QUALITY_FILENAME;
+
+    // prettier -c path/to/folder/
+    const outputCheck = `Checking formatting...
+[warn] test/dirty-2.js
+[warn] test/dirty.js
+[warn] Code style issues found in 3 files. Run Prettier to fix.
+`;
+
+    // prettier -l path/to/folder/
+    const outputList = `test/dirty-2.js
+test/dirty.js
+`;
+
+    await prettierFormatterGitLab(outputCheck);
+    await prettierFormatterGitLab(outputList);
+    expect(consoleSpy).toHaveBeenCalledTimes(12);
+    expect(consoleSpy.mock.calls.slice(0, 6)).toEqual(consoleSpy.mock.calls.slice(6, 12));
+    consoleSpy.mockRestore();
+  });
 });