code review

Author: avivkeller

src/linter/reporters/__tests__/github.test.mjs

@@ -1,20 +1,14 @@
 import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
-import {
-  errorIssue,
-  infoIssue,
-  warnIssue,
-} from '../../__tests__/fixtures/issues.mjs';
+import * as issues from '../../__tests__/fixtures/issues.mjs';
 import github from '../github.mjs';
 
 describe('github', () => {
   it('should write to stdout with the correct fn based on the issue level', t => {
     t.mock.method(process.stdout, 'write');
 
-    github(infoIssue);
-    github(warnIssue);
-    github(errorIssue);
+    Object.values(issues).forEach(github);
 
     assert.equal(process.stdout.write.mock.callCount(), 3);
 
@@ -23,9 +17,9 @@ describe('github', () => {
     );
 
     assert.deepEqual(callsArgs, [
+      '::error file=doc/api/test.md,line=1,endLine=1::This is a ERROR issue',
       '::notice file=doc/api/test.md,line=1,endLine=1::This is a INFO issue',
       '::warning file=doc/api/test.md,line=1,endLine=1::This is a WARN issue',
-      '::error file=doc/api/test.md,line=1,endLine=1::This is a ERROR issue',
     ]);
   });
 });