@reporters/github 1.7.1 → 1.8.0

package/index.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('node:path');
+const { fileURLToPath } = require('node:url');
 const util = require('node:util');
 const { EOL } = require('node:os');
 const core = require('@actions/core');
@@ -12,11 +13,14 @@ const stack = new StackUtils({ cwd: WORKSPACE, internals: StackUtils.nodeInterna
 
 const isFile = (name) => name?.startsWith(WORKSPACE);
 
-const getRelativeFilePath = (name) => (isFile(name) ? path.relative(WORKSPACE, require.resolve(name) ?? '') : null);
+const getRelativeFilePath = (name) => (isFile(name) ? path.relative(WORKSPACE, name) : null);
 
 function getFilePath(fileName) {
   if (fileName.startsWith('file://')) {
-    return getRelativeFilePath(new URL(fileName).pathname);
+    return getRelativeFilePath(fileURLToPath(fileName));
+  }
+  if (!path.isAbsolute(fileName)) {
+    return getRelativeFilePath(path.resolve(fileName) ?? '');
   }
   return getRelativeFilePath(fileName);
 }
@@ -91,9 +95,10 @@ module.exports = async function githubReporter(source) {
       } case 'test:diagnostic':
         if (event.data.file === undefined
           || event.data.line === undefined
-          || event.data.column === undefined) {
+          || event.data.column === undefined
+          || (event.data.line === 1 && event.data.column === 1)) {
           diagnostics.push(event.data.message);
-        } else {
+        } else if (process.env.GITHUB_ACTIONS_REPORTER_VERBOSE) {
           core.notice(event.data.message, extractLocation(event.data));
         }
         break;
@@ -101,7 +106,7 @@ module.exports = async function githubReporter(source) {
         break;
     }
   }
-  const formatedDiagnostics = diagnostics.map((d) => {
+  const formattedDiagnostics = diagnostics.map((d) => {
     const [key, ...rest] = d.split(' ');
     const value = rest.join(' ');
     return [
@@ -109,14 +114,14 @@ module.exports = async function githubReporter(source) {
       DIAGNOSTIC_VALUES[key] ? DIAGNOSTIC_VALUES[key](value) : value,
     ];
   });
-  core.startGroup(`Test results (${formatedDiagnostics.find(([key]) => key === DIAGNOSTIC_KEYS.pass)?.[1] ?? counter.pass} passed, ${formatedDiagnostics.find(([key]) => key === DIAGNOSTIC_KEYS.fail)?.[1] ?? counter.fail} failed)`);
-  core.notice(formatedDiagnostics.map((d) => d.join(': ')).join(EOL));
+  core.startGroup(`Test results (${formattedDiagnostics.find(([key]) => key === DIAGNOSTIC_KEYS.pass)?.[1] ?? counter.pass} passed, ${formattedDiagnostics.find(([key]) => key === DIAGNOSTIC_KEYS.fail)?.[1] ?? counter.fail} failed)`);
+  core.notice(formattedDiagnostics.map((d) => d.join(': ')).join(EOL));
   core.endGroup();
 
   if (process.env.GITHUB_STEP_SUMMARY) {
     await core.summary
       .addHeading('Test Results')
-      .addTable(formatedDiagnostics)
+      .addTable(formattedDiagnostics)
       .write();
   }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reporters/github",
-  "version": "1.7.1",
+  "version": "1.8.0",
   "description": "A github actions reporter for `node:test`",
   "type": "commonjs",
   "keywords": [