Infrastructure: add Ava ESLint plug-in with recommended rules (pull #1566)

* add ava ESLint plug-in
* fix: Ava ESLint recommended cleanups
   * No strict declaration in Ava module files
   * Ignore the "index" test file as it isn't directly called
   * Fix async/await functions
   * `after.always` hook must come before `beforeEach`

Author: nschonni

package-lock.json

@@ -35,6 +35,7 @@
     "cspell": "^4.1.2",
     "eslint": "^7.11.0",
     "eslint-config-prettier": "^6.12.0",
+    "eslint-plugin-ava": "^11.0.0",
     "eslint-plugin-prettier": "^3.1.4",
     "geckodriver": "^1.20.0",
     "glob": "^7.1.6",

package.json

@@ -1,5 +1,5 @@
 {
-  "extends": "../.eslintrc.json",
+  "extends": ["../.eslintrc.json", "plugin:ava/recommended"],
   "parserOptions": {
     "ecmaVersion": 8
   }

test/.eslintrc.json

@@ -1,4 +1,4 @@
-'use strict';
+/* eslint-disable ava/no-ignored-test-files */
 
 const path = require('path');
 const test = require('ava');
@@ -29,18 +29,22 @@ if (!coverageReportRun) {
     await session;
   });
 
+  test.after.always(async () => {
+    if (session) {
+      await session.close();
+    }
+
+    if (geckodriver) {
+      await geckodriver.stop();
+    }
+  });
+
   test.beforeEach((t) => {
     t.context.session = session;
     t.context.waitTime = testWaitTime;
     t.context.queryElement = queryElement;
     t.context.queryElements = queryElements;
   });
-
-  test.after.always(() => {
-    return Promise.resolve(session && session.close()).then(
-      () => geckodriver && geckodriver.stop()
-    );
-  });
 }
 
 /**
@@ -75,7 +79,7 @@ const _ariaTest = (desc, page, testId, body, failing) => {
   const testName = page + ' ' + selector + ': ' + desc;
 
   if (coverageReportRun) {
-    test(testName, async function (t) {
+    test(testName, function (t) {
       t.fail('All tests expect to fail. Running in coverage mode.');
     });
     return;

test/index.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const { ariaTest } = require('..');
 const { By, Key } = require('selenium-webdriver');
 const assertAriaControls = require('../util/assertAriaControls');

test/tests/accordion_accordion.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const { ariaTest } = require('..');
 const { By } = require('selenium-webdriver');
 

test/tests/alert_alert.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const { ariaTest } = require('..');
 const { By } = require('selenium-webdriver');
 const assertAriaLabelExists = require('../util/assertAriaLabelExists');

test/tests/breadcrumb_index.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const { ariaTest } = require('..');
 const { By, Key } = require('selenium-webdriver');
 

test/tests/button_button.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const { ariaTest } = require('..');
 const { By, Key } = require('selenium-webdriver');
 const assertAttributeValues = require('../util/assertAttributeValues');

test/tests/carousel_carousel-1-prev-next.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const { ariaTest } = require('..');
 const { By, Key } = require('selenium-webdriver');
 const assertAttributeValues = require('../util/assertAttributeValues');