@reporters/github 1.4.0 → 1.4.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.4.1](https://github.com/MoLow/reporters/compare/github-v1.4.0...github-v1.4.1) (2023-07-19)
+
+
+### Bug Fixes
+
+* more accurate counting ([1949140](https://github.com/MoLow/reporters/commit/19491406b769f03555b3ae352ed9838e2154c855))
+
 ## [1.4.0](https://github.com/MoLow/reporters/compare/github-v1.3.0...github-v1.4.0) (2023-07-18)
 
 

package/index.js

@@ -75,7 +75,6 @@ module.exports = async function githubReporter(source) {
         break;
     }
   }
-  core.startGroup(`Test results (${counter.pass} passed, ${counter.fail} failed)`);
   const formatedDiagnostics = diagnostics.map((d) => {
     const [key, ...rest] = d.split(' ');
     const value = rest.join(' ');
@@ -84,6 +83,7 @@ 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.endGroup();
 

package/package.json

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

package/tests/index.test.js

@@ -6,8 +6,7 @@ const assert = require('assert');
 const path = require('path');
 const { readFileSync, writeFileSync } = require('fs');
 const { compareLines } = require('../../../tests/utils');
-// eslint-disable-next-line import/no-dynamic-require
-const output = require(`./output.${process.version.split('.')[0]}`);
+const output = require('./output');
 
 const GITHUB_STEP_SUMMARY = join(tmpdir(), 'github-actions-test-reporter');
 writeFileSync(GITHUB_STEP_SUMMARY, '');

package/tests/{output.v18.js → output.js}

@@ -16,7 +16,7 @@ module.exports = {
 ::debug::completed running is skipped
 ::debug::starting to run is a todo
 ::debug::completed running is a todo
-::group::Test results \\(6 passed, 2 failed\\)
+::group::Test results \\(3 passed, 1 failed\\)
 ::notice::Total Tests: 6%0ASuites 📂: 2%0APassed ✅: 3%0AFailed ❌: 1%0ACanceled 🚫: 0%0ASkipped ⏭️: 1%0ATodo 📝: 1%0ADuration 🕐: .*ms
 ::endgroup::
 `,

package/tests/output.v19.js

@@ -1,26 +0,0 @@
-module.exports = {
-  stdout: `::debug::starting to run tests
-::debug::starting to run is ok
-::debug::completed running is ok
-::debug::starting to run fails
-::error title=fails,file=tests/example.js::\\[Error \\[ERR_TEST_FAILURE\\]: this is an error\\] {%0A  failureType: 'testCodeFailure',%0A  cause: Error: this is an error%0A      at .*.<anonymous> (.*/example.js:6:11).*
-::debug::starting to run is a diagnostic
-::debug::completed running is a diagnostic
-::notice file=tests/example.js::this is a diagnostic
-::error title=tests,file=tests/example.js::\\[Error \\[ERR_TEST_FAILURE\\]: 1 subtest failed\\] { failureType: 'subtestsFailed', cause: '1 subtest failed', code: 'ERR_TEST_FAILURE' }
-::debug::starting to run more tests
-::debug::starting to run is ok
-::debug::completed running is ok
-::debug::completed running more tests
-::debug::starting to run is skipped
-::debug::completed running is skipped
-::debug::starting to run is a todo
-::debug::completed running is a todo
-::group::Test results \\(6 passed, 2 failed\\)
-::notice::Total Tests: 6%0ASuites 📂: 2%0APassed ✅: 3%0AFailed ❌: 1%0ACanceled 🚫: 0%0ASkipped ⏭️: 1%0ATodo 📝: 1%0ADuration 🕐: .*ms
-::endgroup::
-`,
-  summary: `<h1>Test Results</h1>
-<table><tr><td>Total Tests</td><td>6</td></tr><tr><td>Suites 📂</td><td>2</td></tr><tr><td>Passed ✅</td><td>3</td></tr><tr><td>Failed ❌</td><td>1</td></tr><tr><td>Canceled 🚫</td><td>0</td></tr><tr><td>Skipped ⏭️</td><td>1</td></tr><tr><td>Todo 📝</td><td>1</td></tr><tr><td>Duration 🕐</td><td>.*ms</td></tr></table>
-`,
-};

package/tests/output.v20.js

@@ -1,26 +0,0 @@
-module.exports = {
-  stdout: `::debug::starting to run tests
-::debug::starting to run is ok
-::debug::completed running is ok
-::debug::starting to run fails
-::error title=fails,file=tests/example.js::\\[Error \\[ERR_TEST_FAILURE\\]: this is an error\\] {%0A  failureType: 'testCodeFailure',%0A  cause: Error: this is an error%0A      at .*.<anonymous> (.*/example.js:6:11).*
-::debug::starting to run is a diagnostic
-::debug::completed running is a diagnostic
-::notice file=tests/example.js::this is a diagnostic
-::error title=tests,file=tests/example.js::\\[Error \\[ERR_TEST_FAILURE\\]: 1 subtest failed\\] { failureType: 'subtestsFailed', cause: '1 subtest failed', code: 'ERR_TEST_FAILURE' }
-::debug::starting to run more tests
-::debug::starting to run is ok
-::debug::completed running is ok
-::debug::completed running more tests
-::debug::starting to run is skipped
-::debug::completed running is skipped
-::debug::starting to run is a todo
-::debug::completed running is a todo
-::group::Test results \\(6 passed, 2 failed\\)
-::notice::Total Tests: 6%0ASuites 📂: 2%0APassed ✅: 3%0AFailed ❌: 1%0ACanceled 🚫: 0%0ASkipped ⏭️: 1%0ATodo 📝: 1%0ADuration 🕐: .*ms
-::endgroup::
-`,
-  summary: `<h1>Test Results</h1>
-<table><tr><td>Total Tests</td><td>6</td></tr><tr><td>Suites 📂</td><td>2</td></tr><tr><td>Passed ✅</td><td>3</td></tr><tr><td>Failed ❌</td><td>1</td></tr><tr><td>Canceled 🚫</td><td>0</td></tr><tr><td>Skipped ⏭️</td><td>1</td></tr><tr><td>Todo 📝</td><td>1</td></tr><tr><td>Duration 🕐</td><td>.*ms</td></tr></table>
-`,
-};