Use jest with eslint

.eslintignore

@@ -1,4 +0,0 @@
-dist/
-lib/
-node_modules/
-jest.config.js

.github/workflows/check-dist.yml

@@ -24,7 +24,7 @@ jobs:
       - uses: actions/checkout@v5
 
       - name: Set Node.js
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v5
         with:
           node-version-file: '.nvmrc'
 
@@ -46,7 +46,7 @@ jobs:
         id: diff
 
       # If index.js was different than expected, upload the expected version as an artifact
-      - uses: actions/upload-artifact@v5
+      - uses: actions/upload-artifact@v4
         if: ${{ failure() && steps.diff.conclusion == 'failure' }}
         with:
           name: dist

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
-      - uses: actions/setup-node@v6
+      - uses: actions/setup-node@v5
         with:
           node-version-file: '.nvmrc'
       - run: npm ci
@@ -25,7 +25,7 @@ jobs:
 
       - name: Upload test results
         if: ${{ !cancelled() }}
-        uses: actions/upload-artifact@v5
+        uses: actions/upload-artifact@v4
         with:
           name: test-results
           path: __tests__/__results__/*.xml

__tests__/report/get-report.test.ts

@@ -1,120 +0,0 @@
-import {getBadge, DEFAULT_OPTIONS, ReportOptions} from '../../src/report/get-report'
-
-describe('getBadge', () => {
-  describe('URI encoding with special characters', () => {
-    it('generates correct URI with simple badge title', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'tests'
-      }
-      const badge = getBadge(5, 0, 1, options)
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/tests-5%20passed%2C%201%20skipped-success)')
-    })
-
-    it('handles badge title with single hyphen', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'unit-tests'
-      }
-      const badge = getBadge(3, 0, 0, options)
-      // The hyphen in the badge title should be encoded as --
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/unit--tests-3%20passed-success)')
-    })
-
-    it('handles badge title with multiple hyphens', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'integration-api-tests'
-      }
-      const badge = getBadge(10, 0, 0, options)
-      // All hyphens in the title should be encoded as --
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/integration--api--tests-10%20passed-success)')
-    })
-
-    it('handles badge title with multiple underscores', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'my_integration_test'
-      }
-      const badge = getBadge(10, 0, 0, options)
-      // All underscores in the title should be encoded as __
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/my__integration__test-10%20passed-success)')
-    })
-
-    it('handles badge title with version format containing hyphen', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'MariaDb 12.0-ubi database tests'
-      }
-      const badge = getBadge(1, 0, 0, options)
-      // The hyphen in "12.0-ubi" should be encoded as --
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/MariaDb%2012.0--ubi%20database%20tests-1%20passed-success)')
-    })
-
-    it('handles badge title with dots and hyphens', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'v1.2.3-beta-test'
-      }
-      const badge = getBadge(4, 1, 0, options)
-      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/v1.2.3--beta--test-4%20passed%2C%201%20failed-critical)')
-    })
-
-    it('preserves structural hyphens between label and message', () => {
-      const options: ReportOptions = {
-        ...DEFAULT_OPTIONS,
-        badgeTitle: 'test-suite'
-      }
-      const badge = getBadge(2, 3, 1, options)
-      // The URI should have literal hyphens separating title-message-color
-      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/test--suite-2%20passed%2C%203%20failed%2C%201%20skipped-critical)')
-    })
-  })
-
-  describe('generates test outcome as color name for imgshields', () => {
-    it('uses success color when all tests pass', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(5, 0, 0, options)
-      expect(badge).toContain('-success)')
-    })
-
-    it('uses critical color when tests fail', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(5, 2, 0, options)
-      expect(badge).toContain('-critical)')
-    })
-
-    it('uses yellow color when no tests found', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(0, 0, 0, options)
-      expect(badge).toContain('-yellow)')
-    })
-  })
-
-  describe('badge message composition', () => {
-    it('includes only passed count when no failures or skips', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(5, 0, 0, options)
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/tests-5%20passed-success)')
-    })
-
-    it('includes passed and failed counts', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(5, 2, 0, options)
-      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/tests-5%20passed%2C%202%20failed-critical)')
-    })
-
-    it('includes passed, failed and skipped counts', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(5, 2, 1, options)
-      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/tests-5%20passed%2C%202%20failed%2C%201%20skipped-critical)')
-    })
-
-    it('uses "none" message when no tests', () => {
-      const options: ReportOptions = {...DEFAULT_OPTIONS}
-      const badge = getBadge(0, 0, 0, options)
-      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/tests-none-yellow)')
-    })
-  })
-})
-

dist/index.js

@@ -1909,7 +1909,6 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DEFAULT_OPTIONS = void 0;
 exports.getReport = getReport;
-exports.getBadge = getBadge;
 const core = __importStar(__nccwpck_require__(7484));
 const markdown_utils_1 = __nccwpck_require__(5129);
 const node_utils_1 = __nccwpck_require__(5384);
@@ -2023,10 +2022,8 @@ function getBadge(passed, failed, skipped, options) {
         color = 'yellow';
     }
     const hint = failed > 0 ? 'Tests failed' : 'Tests passed successfully';
-    const encodedBadgeTitle = encodeImgShieldsURIComponent(options.badgeTitle);
-    const encodedMessage = encodeImgShieldsURIComponent(message);
-    const encodedColor = encodeImgShieldsURIComponent(color);
-    return `![${hint}](https://img.shields.io/badge/${encodedBadgeTitle}-${encodedMessage}-${encodedColor})`;
+    const uri = encodeURIComponent(`${options.badgeTitle}-${message}-${color}`);
+    return `![${hint}](https://img.shields.io/badge/${uri})`;
 }
 function getTestRunsReport(testRuns, options) {
     const sections = [];
@@ -2156,9 +2153,6 @@ function getResultIcon(result) {
             return '';
     }
 }
-function encodeImgShieldsURIComponent(component) {
-    return encodeURIComponent(component).replace(/-/g, '--').replace(/_/g, '__');
-}
 
 
 /***/ }),

eslint.config.mjs

@@ -0,0 +1,52 @@
+import github from 'eslint-plugin-github'
+import prettier from 'eslint-plugin-prettier'
+import jest from 'eslint-plugin-jest'
+
+export default [
+  github.getFlatConfigs().recommended,
+  ...github.getFlatConfigs().typescript,
+  {
+    files: ['src/**/*.ts'],
+    plugins: {
+      prettier,
+    },
+    rules: {
+      'no-shadow': 'off',
+      'import/no-namespace': 'off',
+      'i18n-text/no-en': 'off',
+      'prefer-template': 'off',
+      'prettier/prettier': 'error',
+      "@typescript-eslint/array-type": ['error', {default: 'array'}],
+      '@typescript-eslint/no-unused-vars': ['error', {varsIgnorePattern: '^_'}],
+      '@typescript-eslint/no-shadow': ['error'],
+    },
+  },
+  {
+    files: ['__tests__/**/*.test.ts'],
+    ...jest.configs['flat/recommended'],
+    plugins: {
+      jest,
+    },
+    languageOptions: {
+      globals: jest.environments.globals.globals,
+    },
+    rules: {
+      'i18n-text/no-en': 'off',
+      'import/no-namespace': 'off',
+      "@typescript-eslint/array-type": ['error', {default: 'array'}],
+    },
+  },
+  {
+    ignores: [
+      'dist/**',
+      'lib/**',
+      'node_modules/**',
+      '__tests__/__snapshots__/**',
+      '__tests__/__results__/**',
+      'assets/**',
+      'reports/**',
+      'eslint.config.mjs',
+      'jest.config.js',
+    ],
+  },
+]

package.json

@@ -8,7 +8,7 @@
     "build": "tsc",
     "format": "prettier --write **/*.ts",
     "format-check": "prettier --check **/*.ts",
-    "lint": "eslint src/**/*.ts",
+    "lint": "eslint",
     "package": "ncc build --license licenses.txt && eolConverter lf 'dist/*'",
     "version": "npm run build && npm run package && git add dist/*",
     "test": "jest --ci --reporters=default --reporters=jest-junit",
@@ -49,25 +49,25 @@
     "@octokit/webhooks-types": "^7.6.1",
     "@types/adm-zip": "^0.5.7",
     "@types/jest": "^30.0.0",
-    "@types/node": "^20.19.23",
+    "@types/node": "^20.19.13",
     "@types/picomatch": "^4.0.2",
     "@types/xml2js": "^0.4.14",
-    "@typescript-eslint/eslint-plugin": "^7.18.0",
-    "@typescript-eslint/parser": "^7.18.0",
-    "@vercel/ncc": "^0.38.4",
+    "@typescript-eslint/eslint-plugin": "^8.43.0",
+    "@typescript-eslint/parser": "^8.43.0",
+    "@vercel/ncc": "^0.38.3",
     "eol-converter-cli": "^1.1.0",
-    "eslint": "^8.57.1",
-    "eslint-import-resolver-typescript": "^3.10.1",
-    "eslint-plugin-github": "^4.10.2",
+    "eslint": "^9.35.0",
+    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-plugin-github": "^6.0.0",
     "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-jest": "^28.14.0",
+    "eslint-plugin-jest": "^29.0.1",
     "eslint-plugin-prettier": "^5.5.4",
-    "jest": "^30.2.0",
+    "jest": "^30.1.3",
     "jest-junit": "^16.0.0",
     "js-yaml": "^4.1.0",
     "prettier": "^3.6.2",
-    "ts-jest": "^29.4.5",
-    "typescript": "^5.9.3"
+    "ts-jest": "^29.4.1",
+    "typescript": "^5.9.2"
   },
   "jest-junit": {
     "suiteName": "jest tests",

src/report/get-report.ts

@@ -125,7 +125,7 @@ function getReportBadge(results: TestRunResult[], options: ReportOptions): strin
   return getBadge(passed, failed, skipped, options)
 }
 
-export function getBadge(passed: number, failed: number, skipped: number, options: ReportOptions): string {
+function getBadge(passed: number, failed: number, skipped: number, options: ReportOptions): string {
   const text = []
   if (passed > 0) {
     text.push(`${passed} passed`)
@@ -145,10 +145,8 @@ export function getBadge(passed: number, failed: number, skipped: number, option
     color = 'yellow'
   }
   const hint = failed > 0 ? 'Tests failed' : 'Tests passed successfully'
-  const encodedBadgeTitle = encodeImgShieldsURIComponent(options.badgeTitle)
-  const encodedMessage = encodeImgShieldsURIComponent(message)
-  const encodedColor = encodeImgShieldsURIComponent(color)
-  return `![${hint}](https://img.shields.io/badge/${encodedBadgeTitle}-${encodedMessage}-${encodedColor})`
+  const uri = encodeURIComponent(`${options.badgeTitle}-${message}-${color}`)
+  return `![${hint}](https://img.shields.io/badge/${uri})`
 }
 
 function getTestRunsReport(testRuns: TestRunResult[], options: ReportOptions): string[] {
@@ -307,7 +305,3 @@ function getResultIcon(result: TestExecutionResult): string {
       return ''
   }
 }
-
-function encodeImgShieldsURIComponent(component: string): string {
-  return encodeURIComponent(component).replace(/-/g, '--').replace(/_/g, '__')
-}