Drop Node 12 support (#301)

Author: edg2s

.github/workflows/nodejs.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x]
+        node-version: [14.x, 16.x, 18.x, 20.x]
 
     steps:
     - uses: actions/checkout@v1
@@ -23,7 +23,7 @@ jobs:
         npm run build --if-present
         npm test
         npm run report
-        npm i --no-save [email protected]
+        npm i --no-save [email protected] --force
         npm run onlytest
       env:
         CI: true

package.json

@@ -16,7 +16,7 @@
 	"scripts": {
 		"pretest": "eslint --report-unused-disable-directives .",
 		"test": "nyc mocha --reporter dot tests/** && git diff --exit-code docs/ src/ README.md",
-		"onlytest": "mocha --reporter dot tests/** --nofix",
+		"onlytest": "mocha --reporter dot tests/**",
 		"report": "nyc report --reporter=text-lcov > coverage.lcov",
 		"reporthtml": "nyc report --reporter=html",
 		"testpath": "mocha",

tools/rule-tester.js

@@ -1,20 +1,4 @@
 'use strict';
 
-const DocGenRuleTester = require( 'eslint-docgen' ).RuleTester;
-const noFix = process.argv.includes( '--nofix' );
-
-class RuleTester extends DocGenRuleTester {
-	run( name, rule, tests ) {
-		if ( noFix ) {
-			tests.valid.forEach( ( test ) => {
-				delete test.output;
-			} );
-			tests.invalid.forEach( ( test ) => {
-				delete test.output;
-			} );
-		}
-		return super.run.call( this, name, rule, tests );
-	}
-}
-
-module.exports = RuleTester;
+// TODO: This no longer overrides RuleTester so can be deleted and replaced
+module.exports = require( 'eslint-docgen' ).RuleTester;