fix(require-jsdoc): with eslint 6, we can't use schema for defaults, so revert to old approach

chore(linting): update eslint devDep
testing: avoid timeout (not an issue anymore with eslint 6)
testing(check-examples): use absolute paths with `matchingFileName` (used with `getConfigForFile`/`Linter`) per eslint 6
testing(check-examples): use a simple `matchingFileName` file which doesn't have problem with loading `babel-eslint` in eslint 6 as does our config
testing(require-param): use absolute paths with `parser` per eslint 6

Author: brettz9

package.json

@@ -23,7 +23,7 @@
     "babel-plugin-add-module-exports": "^1.0.2",
     "babel-plugin-istanbul": "^5.1.4",
     "chai": "^4.2.0",
-    "eslint": "^5.14.1",
+    "eslint": "^6.0.0",
     "eslint-config-canonical": "^17.1.1",
     "gitdown": "^2.5.8",
     "glob": "^7.1.4",
@@ -64,8 +64,8 @@
     "build": "rm -fr ./dist && NODE_ENV=production babel ./src --out-dir ./dist --copy-files --source-maps",
     "create-readme": "gitdown ./.README/README.md --output-file ./README.md && npm run add-assertions",
     "lint": "eslint ./src ./test",
-    "test-cov": "BABEL_ENV=test nyc mocha --recursive --require @babel/register --reporter progress --timeout 7000",
-    "test": "BABEL_ENV=test nyc --reporter text-summary mocha --recursive --require @babel/register --reporter progress --timeout 7000"
+    "test-cov": "BABEL_ENV=test nyc mocha --recursive --require @babel/register --reporter progress",
+    "test": "BABEL_ENV=test nyc --reporter text-summary mocha --recursive --require @babel/register --reporter progress"
   },
   "nyc": {
     "require": [

src/rules/requireJsdoc.js

@@ -18,19 +18,15 @@ const OPTIONS_SCHEMA = {
           default: {},
           properties: {
             ancestorsOnly: {
-              default: false,
               type: 'boolean'
             },
             cjs: {
-              default: true,
               type: 'boolean'
             },
             esm: {
-              default: true,
               type: 'boolean'
             },
             window: {
-              default: false,
               type: 'boolean'
             }
           },
@@ -142,10 +138,10 @@ export default iterateJsdoc(null, {
 
       if (publicOnly) {
         const opt = {
-          ancestorsOnly: publicOnly.ancestorsOnly,
-          esm: publicOnly.esm,
-          initModuleExports: publicOnly.cjs,
-          initWindow: publicOnly.window
+          ancestorsOnly: Boolean(_.get(publicOnly, 'ancestorsOnly', false)),
+          esm: Boolean(_.get(publicOnly, 'esm', true)),
+          initModuleExports: Boolean(_.get(publicOnly, 'cjs', true)),
+          initWindow: Boolean(_.get(publicOnly, 'window', false))
         };
         const parseResult = exportParser.parse(sourceCode.ast, node, opt);
         const exported = exportParser.isExported(node, parseResult, opt);

test/rules/assertions/checkExamples.js

@@ -1,5 +1,12 @@
 /* eslint-disable no-restricted-syntax */
 
+// After `importMeta` no longer experimental, we can use this ESM
+//   approach over `__dirname`?
+// import {fileURLToPath} from 'url';
+// import {join, dirname} from 'path';
+// join(dirname(fileURLToPath(import.meta.url)), 'babel-eslint')
+import {join} from 'path';
+
 export default {
   invalid: [
     {
@@ -314,7 +321,7 @@ export default {
       ],
       settings: {
         jsdoc: {
-          matchingFileName: 'test/jsdocUtils.js'
+          matchingFileName: join(__dirname, '../', 'data/test.js')
         }
       }
     },

test/rules/assertions/requireParam.js

@@ -1,3 +1,10 @@
+// After `importMeta` no longer experimental, we can use this ESM
+//   approach over `__dirname`?
+// import {fileURLToPath} from 'url';
+// import {join, dirname} from 'path';
+// join(dirname(fileURLToPath(import.meta.url)), 'babel-eslint')
+import {join} from 'path';
+
 export default {
   invalid: [
     {
@@ -636,7 +643,7 @@ export default {
           /** @const {boolean} test */
           const test = something?.find(_ => _)
       `,
-      parser: 'babel-eslint'
+      parser: join(__dirname, '../../../node_modules', 'babel-eslint')
     },
     {
       code: `

test/rules/data/.eslintrc.json

@@ -0,0 +1,6 @@
+{
+  "root": true,
+  "rules": {
+    "semi": ["error", "always"]
+  }
+}

test/rules/data/test.js

@@ -0,0 +1,2 @@
+// Just a plain JavaScript file which can be targeted by a
+//   test for ESLint rule matching