From 76dd06958a62596d44b0e23822e003f8dde2bd8e Mon Sep 17 00:00:00 2001
From: David Calhoun <david.b.calhoun@gmail.com>
Date: Sun, 20 Aug 2023 09:29:06 -0400
Subject: [PATCH 1/4] new tests for multiple testAttributes

---
 lib/rules/onClick.js         |  3 ++-
 tests/lib/rules/anchor.js    |  5 ++++-
 tests/lib/rules/button.js    |  7 +++++--
 tests/lib/rules/input.js     |  5 ++++-
 tests/lib/rules/onChange.js  | 21 ++++++++++++---------
 tests/lib/rules/onClick.js   |  9 ++++++++-
 tests/lib/rules/onKeyDown.js |  7 +++++--
 tests/lib/rules/onKeyUp.js   |  7 +++++--
 tests/lib/rules/onSubmit.js  |  7 +++++--
 9 files changed, 50 insertions(+), 21 deletions(-)

diff --git a/lib/rules/onClick.js b/lib/rules/onClick.js
index 1a93b7b..ada29cd 100644
--- a/lib/rules/onClick.js
+++ b/lib/rules/onClick.js
@@ -47,7 +47,8 @@ module.exports = {
                         const { nanoid } = require('nanoid');
                         const suggestedId = nanoid();
                         const namePositionEnd = node.name.range[1];
-                        const attributeText = `${ testAttribute }="${ suggestedId }"`;
+                        const singleTestAttribute = Array.isArray(testAttribute) ? testAttribute[0] : testAttribute;
+                        const attributeText = `${ singleTestAttribute }="${ suggestedId }"`;
                         const start = namePositionEnd - 1;
                         const end = start + 1;
 
diff --git a/tests/lib/rules/anchor.js b/tests/lib/rules/anchor.js
index cb4c116..0b16c09 100644
--- a/tests/lib/rules/anchor.js
+++ b/tests/lib/rules/anchor.js
@@ -28,7 +28,9 @@ ruleTester.run('anchor', rule, {
         { code: `<Foo href="bar" readonly>Bar</Foo>` },
         { code: `<link href="example.com" />` },
         { code: `<area href="example.com" />` },
-        { code: `<base href="example.com" />` }
+        { code: `<base href="example.com" />` },
+        { code: `<a testId={ bar }>Foo</a>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<a data-testid={ bar }>Foo</a>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -41,5 +43,6 @@ ruleTester.run('anchor', rule, {
         { code: '<Foo href="bar" disabled={ bar } />', errors: [anchorError] },
         { code: '<Foo href="bar" readonly={ bar } />', errors: [anchorError] },
         { code: '<Link href="example.com" />', errors: [anchorError] },
+        { code: `<a data-test-id={ bar }>Foo</a>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(anchor.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/button.js b/tests/lib/rules/button.js
index 25538a6..0c24b15 100644
--- a/tests/lib/rules/button.js
+++ b/tests/lib/rules/button.js
@@ -27,7 +27,9 @@ ruleTester.run('button', rule, {
         { code: `<DownloadButton data-test-id='foo' />` },
         { code: `<button disabled />` },
         { code: `<button readonly />` },
-        { code: `<Button>Foo</Button>`, options: ["warn", {htmlOnly: true}] }
+        { code: `<Button>Foo</Button>`, options: ["warn", {htmlOnly: true}] },
+        { code: `<button testId={ bar } />`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<button data-testid={ bar } />`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -42,6 +44,7 @@ ruleTester.run('button', rule, {
         { code: '<ButtonContainer />', errors: [buttonError] },
         { code: '<DownloadButton />', errors: [buttonError] },
         { code: '<button disabled={ foo } />', errors: [buttonError] },
-        { code: '<button readonly={ foo } />', errors: [buttonError] }
+        { code: '<button readonly={ foo } />', errors: [buttonError] },
+        { code: `<button data-test-id={ bar } />`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(button.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/input.js b/tests/lib/rules/input.js
index c1f3fd2..e6c6475 100644
--- a/tests/lib/rules/input.js
+++ b/tests/lib/rules/input.js
@@ -22,6 +22,8 @@ ruleTester.run('input', rule, {
         { code: `<input data-test-id='foo' />` },
         { code: `<input disabled />` },
         { code: `<input readonly />` },
+        { code: `<input testId={ bar }>Foo</input>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<input data-testid={ bar }>Foo</input>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -31,6 +33,7 @@ ruleTester.run('input', rule, {
         { code: '<input>Foo</input>', errors: [inputError] },
         { code: '<input />', errors: [inputError] },
         { code: '<input disabled={ foo } />', errors: [inputError] },
-        { code: '<input readonly={ foo } />', errors: [inputError] }
+        { code: '<input readonly={ foo } />', errors: [inputError] },
+        { code: `<input data-test-id={ bar }>Foo</input>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(input.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onChange.js b/tests/lib/rules/onChange.js
index b8294f2..b19533e 100644
--- a/tests/lib/rules/onChange.js
+++ b/tests/lib/rules/onChange.js
@@ -17,9 +17,9 @@ const onChangeError = getError(onChange.message, defaults.testAttribute);
 const ruleTester = new RuleTester();
 ruleTester.run('onChange', rule, {
     valid: [
-        { code: `<div onChange={ this.handleClick } data-test-id={ bar }>Foo</div>` },
-        { code: `<div onChange={ this.handleClick } data-test-id="bar">Foo</div>` },
-        { code: `<div onChange={ this.handleClick } data-test-id="bar" />` },
+        { code: `<div onChange={ this.handleChange } data-test-id={ bar }>Foo</div>` },
+        { code: `<div onChange={ this.handleChange } data-test-id="bar">Foo</div>` },
+        { code: `<div onChange={ this.handleChange } data-test-id="bar" />` },
         { code: `<div onChange={ () => {} } data-test-id={ bar }>Foo</div>` },
         { code: `<div onChange={ () => {} } data-test-id="bar">Foo</div>` },
         { code: `<div onChange={ () => {} } data-test-id="bar" />` },
@@ -27,16 +27,19 @@ ruleTester.run('onChange', rule, {
         { code: `<Bar onChange={ () => {} } data-test-id="bar">Foo</Bar>` },
         { code: `<Bar onChange={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onChange={ () => {} } disabled />` },
-        { code: `<Bar onChange={ () => {} } readonly />` }
+        { code: `<Bar onChange={ () => {} } readonly />` },
+        { code: `<div onChange={ this.handleChange } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<div onChange={ this.handleChange } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
-        { code: '<div onChange={ this.handleClick } />', errors: [onChangeError] },
-        { code: '<div onChange={ this.handleClick }>foo</div>', errors: [onChangeError] },
-        { code: '<Bar onChange={ this.handleClick } />', errors: [onChangeError] },
-        { code: '<Bar onChange={ this.handleClick }>foo</Bar>', errors: [onChangeError] },
+        { code: '<div onChange={ this.handleChange } />', errors: [onChangeError] },
+        { code: '<div onChange={ this.handleChange }>foo</div>', errors: [onChangeError] },
+        { code: '<Bar onChange={ this.handleChange } />', errors: [onChangeError] },
+        { code: '<Bar onChange={ this.handleChange }>foo</Bar>', errors: [onChangeError] },
         { code: '<Bar onChange={ () => handleChange() }>foo</Bar>', errors: [onChangeError] },
         { code: '<Bar onChange={ () => handleChange() } disabled={ foo }>foo</Bar>', errors: [onChangeError] },
-        { code: '<Bar onChange={ () => handleChange() } readonly={ foo }>foo</Bar>', errors: [onChangeError] }
+        { code: '<Bar onChange={ () => handleChange() } readonly={ foo }>foo</Bar>', errors: [onChangeError] },
+        { code: `<div onChange={ this.handleChange } data-test-id={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onChange.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onClick.js b/tests/lib/rules/onClick.js
index 7a78927..6104ead 100644
--- a/tests/lib/rules/onClick.js
+++ b/tests/lib/rules/onClick.js
@@ -37,7 +37,9 @@ ruleTester.run('onClick', rule, {
         { code: `<Bar onClick={ () => {} } data-test-id="bar">Foo</Bar>` },
         { code: `<Bar onClick={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onClick={ () => {} } disabled />` },
-        { code: `<Bar onClick={ () => {} } readonly />` }
+        { code: `<Bar onClick={ () => {} } readonly />` },
+        { code: `<div onClick={ this.handleClick } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<div onClick={ this.handleClick } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -82,5 +84,10 @@ ruleTester.run('onClick', rule, {
             output:
               `<Foo.Bar data-test-id="${ suggestedId }" onClick={ () => handleClick() } readonly={ foo }>foo</Foo.Bar>`,
         },
+        {
+            code: `<div onClick={ this.handleClick } data-test-id={ bar }>Foo</div>`,
+            options: ["always", {"testAttribute": ["testId", "data-testid"]}],
+            errors: [getError(onClick.message, ["testId", "data-testid"])],
+            output: `<div testId="${ suggestedId }" onClick={ this.handleClick } data-test-id={ bar }>Foo</div>` }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onKeyDown.js b/tests/lib/rules/onKeyDown.js
index 6e42f02..5f2506f 100644
--- a/tests/lib/rules/onKeyDown.js
+++ b/tests/lib/rules/onKeyDown.js
@@ -27,7 +27,9 @@ ruleTester.run('onKeyDown', rule, {
         { code: `<Bar onKeyDown={ () => {} } data-test-id="bar">Foo</Bar>` },
         { code: `<Bar onKeyDown={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onKeyDown={ () => {} } disabled />` },
-        { code: `<Bar onKeyDown={ () => {} } readonly />` }
+        { code: `<Bar onKeyDown={ () => {} } readonly />` },
+        { code: `<div onKeyDown={ this.handleKeyDown } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<div onKeyDown={ this.handleKeyDown } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -37,6 +39,7 @@ ruleTester.run('onKeyDown', rule, {
         { code: '<Bar onKeyDown={ this.handleKeyDown }>foo</Bar>', errors: [onKeyDownError] },
         { code: '<Bar onKeyDown={ () => handleKeyDown() }>foo</Bar>', errors: [onKeyDownError] },
         { code: '<Bar onKeyDown={ () => handleKeyDown() } disabled={ bar }>foo</Bar>', errors: [onKeyDownError] },
-        { code: '<Bar onKeyDown={ () => handleKeyDown() } readonly={ bar }>foo</Bar>', errors: [onKeyDownError] }
+        { code: '<Bar onKeyDown={ () => handleKeyDown() } readonly={ bar }>foo</Bar>', errors: [onKeyDownError] },
+        { code: `<div onKeyDown={ this.handleKeyDown } data-test-id={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onKeyDown.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onKeyUp.js b/tests/lib/rules/onKeyUp.js
index ea70f0b..9c9da19 100644
--- a/tests/lib/rules/onKeyUp.js
+++ b/tests/lib/rules/onKeyUp.js
@@ -27,7 +27,9 @@ ruleTester.run('onKeyUp', rule, {
         { code: `<Bar onKeyUp={ () => {} } data-test-id="bar">Foo</Bar>` },
         { code: `<Bar onKeyUp={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onKeyUp={ () => {} } disabled />` },
-        { code: `<Bar onKeyUp={ () => {} } readonly />` }
+        { code: `<Bar onKeyUp={ () => {} } readonly />` },
+        { code: `<div onKeyUp={ this.handleKeyUp } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<div onKeyUp={ this.handleKeyUp } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -37,6 +39,7 @@ ruleTester.run('onKeyUp', rule, {
         { code: '<Bar onKeyUp={ this.handleKeyUp }>foo</Bar>', errors: [onKeyUpError] },
         { code: '<Bar onKeyUp={ () => handleKeyUp() }>foo</Bar>', errors: [onKeyUpError] },
         { code: '<Bar onKeyUp={ () => handleKeyUp() } disabled={ bar }>foo</Bar>', errors: [onKeyUpError] },
-        { code: '<Bar onKeyUp={ () => handleKeyUp() } readonly={ bar }>foo</Bar>', errors: [onKeyUpError] }
+        { code: '<Bar onKeyUp={ () => handleKeyUp() } readonly={ bar }>foo</Bar>', errors: [onKeyUpError] },
+        { code: `<div onKeyUp={ this.handleKeyUp } data-test-id={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onKeyUp.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onSubmit.js b/tests/lib/rules/onSubmit.js
index 779b62b..e0bfea4 100644
--- a/tests/lib/rules/onSubmit.js
+++ b/tests/lib/rules/onSubmit.js
@@ -27,7 +27,9 @@ ruleTester.run('onSubmit', rule, {
         { code: `<Bar onSubmit={ () => {} } data-test-id="bar">Foo</Bar>` },
         { code: `<Bar onSubmit={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onSubmit={ () => {} } disabled />` },
-        { code: `<Bar onSubmit={ () => {} } readonly />` }
+        { code: `<Bar onSubmit={ () => {} } readonly />` },
+        { code: `<form onSubmit={ this.handleSubmit } testId={ bar }>Foo</form>`, options: ["always", {"testAttribute": "testId"}] },
+        { code: `<form onSubmit={ this.handleSubmit } data-testid={ bar }>Foo</form>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -37,6 +39,7 @@ ruleTester.run('onSubmit', rule, {
         { code: '<Bar onSubmit={ this.handleSubmit }>foo</Bar>', errors: [onSubmitError] },
         { code: '<Bar onSubmit={ () => handleSubmit() }>foo</Bar>', errors: [onSubmitError] },
         { code: '<Bar onSubmit={ () => handleSubmit() } disabled={ foo }>foo</Bar>', errors: [onSubmitError] },
-        { code: '<Bar onSubmit={ () => handleSubmit() } readonly={ foo }>foo</Bar>', errors: [onSubmitError] }
+        { code: '<Bar onSubmit={ () => handleSubmit() } readonly={ foo }>foo</Bar>', errors: [onSubmitError] },
+        { code: `<form onSubmit={ this.handleSubmit } data-test-id={ bar }>Foo</form>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onSubmit.message, ["testId", "data-testid"])] }
     ].map(parserOptionsMapper)
 });

From 1b56a18994a339bc987d1856988649a57d0c127d Mon Sep 17 00:00:00 2001
From: David Calhoun <david.b.calhoun@gmail.com>
Date: Sun, 20 Aug 2023 09:31:54 -0400
Subject: [PATCH 2/4] add Prettier config; formatting (trivial change)

---
 .prettierrc.js               | 11 +++++++++
 lib/constants.js             | 40 ++++++++++++++++----------------
 lib/index.js                 |  6 ++---
 lib/rules/anchor.js          | 13 +++--------
 lib/rules/button.js          | 13 +++--------
 lib/rules/input.js           | 13 +++--------
 lib/rules/onChange.js        | 13 +++--------
 lib/rules/onClick.js         | 22 +++++-------------
 lib/rules/onKeyDown.js       | 13 +++--------
 lib/rules/onKeyUp.js         | 13 +++--------
 lib/rules/onSubmit.js        | 13 +++--------
 lib/utils.js                 | 34 ++++++++++++----------------
 tests/lib/rules/anchor.js    | 15 ++++++------
 tests/lib/rules/button.js    | 17 +++++++-------
 tests/lib/rules/input.js     | 18 +++++++++------
 tests/lib/rules/onChange.js  | 21 +++++++++++------
 tests/lib/rules/onClick.js   | 41 +++++++++++++++++----------------
 tests/lib/rules/onKeyDown.js | 21 +++++++++++------
 tests/lib/rules/onKeyUp.js   | 21 +++++++++++------
 tests/lib/rules/onSubmit.js  | 21 +++++++++++------
 tests/parserOptionsMapper.js | 44 ++++++++++++++++--------------------
 21 files changed, 201 insertions(+), 222 deletions(-)
 create mode 100644 .prettierrc.js

diff --git a/.prettierrc.js b/.prettierrc.js
new file mode 100644
index 0000000..e31d6d1
--- /dev/null
+++ b/.prettierrc.js
@@ -0,0 +1,11 @@
+/**
+ * Config for Prettier, to automatically format code.  See https://prettier.io/
+ */
+
+module.exports = {
+    semi: true,
+    trailingComma: 'none',
+    singleQuote: true,
+    printWidth: 120,
+    tabWidth: 4
+};
diff --git a/lib/constants.js b/lib/constants.js
index 4d4bf38..c98100f 100644
--- a/lib/constants.js
+++ b/lib/constants.js
@@ -6,32 +6,32 @@ module.exports = {
     },
     defaultRuleSchema: [
         {
-            "enum": ["always", "never"]
+            enum: ['always', 'never']
         },
         {
-            "type": "object",
-            "properties": {
-                "testAttribute": {
-                    "oneOf": [
-                      {
-                        "type": "string",
-                      },
-                      {
-                        "type": "array",
-                        "items": {
-                          "type": "string",
+            type: 'object',
+            properties: {
+                testAttribute: {
+                    oneOf: [
+                        {
+                            type: 'string'
                         },
-                      },
-                    ],
-                  },
-                "ignoreDisabled": {
-                    "type": "boolean"
+                        {
+                            type: 'array',
+                            items: {
+                                type: 'string'
+                            }
+                        }
+                    ]
                 },
-                "ignoreReadonly": {
-                    "type": "boolean"
+                ignoreDisabled: {
+                    type: 'boolean'
+                },
+                ignoreReadonly: {
+                    type: 'boolean'
                 }
             },
-            "additionalProperties": false
+            additionalProperties: false
         }
     ],
     errors: {
diff --git a/lib/index.js b/lib/index.js
index 6b507ee..92c329a 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -2,12 +2,12 @@
  * @fileoverview Makes sure test attributes (e.g. data-test-id) are added to interactive DOM elements.
  * @author David Calhoun
  */
-"use strict";
+'use strict';
 
-const requireIndex = require("requireindex");
+const requireIndex = require('requireindex');
 
 module.exports = {
-    rules: requireIndex(`${ __dirname }/rules`),
+    rules: requireIndex(`${__dirname}/rules`),
     configs: {
         recommended: {
             parserOptions: {
diff --git a/lib/rules/anchor.js b/lib/rules/anchor.js
index 0f516bc..d7a6435 100644
--- a/lib/rules/anchor.js
+++ b/lib/rules/anchor.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attributes on anchors.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -25,7 +18,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/rules/button.js b/lib/rules/button.js
index 4ddac5e..86a5db5 100644
--- a/lib/rules/button.js
+++ b/lib/rules/button.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attributes on buttons.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -33,7 +26,7 @@ module.exports = {
         }
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/rules/input.js b/lib/rules/input.js
index 07ba4fe..b53c67f 100644
--- a/lib/rules/input.js
+++ b/lib/rules/input.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attributes on inputs.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -25,7 +18,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/rules/onChange.js b/lib/rules/onChange.js
index ae57320..388cfa0 100644
--- a/lib/rules/onChange.js
+++ b/lib/rules/onChange.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attribute data-test-id on elements with the onChange property.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -25,7 +18,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/rules/onClick.js b/lib/rules/onClick.js
index ada29cd..1f42cff 100644
--- a/lib/rules/onClick.js
+++ b/lib/rules/onClick.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attribute data-test-id on elements with the onClick property.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -25,7 +18,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
@@ -48,15 +41,12 @@ module.exports = {
                         const suggestedId = nanoid();
                         const namePositionEnd = node.name.range[1];
                         const singleTestAttribute = Array.isArray(testAttribute) ? testAttribute[0] : testAttribute;
-                        const attributeText = `${ singleTestAttribute }="${ suggestedId }"`;
+                        const attributeText = `${singleTestAttribute}="${suggestedId}"`;
                         const start = namePositionEnd - 1;
                         const end = start + 1;
 
-                        return fixer.insertTextAfterRange(
-                            [start, end],
-                            ` ${ attributeText }`
-                        );
-                    },
+                        return fixer.insertTextAfterRange([start, end], ` ${attributeText}`);
+                    }
                 });
             }
         };
diff --git a/lib/rules/onKeyDown.js b/lib/rules/onKeyDown.js
index 522bde6..f05fbf6 100644
--- a/lib/rules/onKeyDown.js
+++ b/lib/rules/onKeyDown.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attribute data-test-id on elements with the onKeyDown property.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -25,7 +18,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/rules/onKeyUp.js b/lib/rules/onKeyUp.js
index 357b342..e4f185b 100644
--- a/lib/rules/onKeyUp.js
+++ b/lib/rules/onKeyUp.js
@@ -2,16 +2,9 @@
  * @fileoverview Requires test attribute data-test-id on elements with the onKeyUp property.
  * @author David Calhoun
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -25,7 +18,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/rules/onSubmit.js b/lib/rules/onSubmit.js
index 1534eb3..bc45db0 100644
--- a/lib/rules/onSubmit.js
+++ b/lib/rules/onSubmit.js
@@ -1,16 +1,9 @@
 /**
  * @fileoverview Requires test attribute data-test-id on elements with the onSubmit property.
  */
-const {
-    errors,
-    defaultRuleSchema,
-    defaults
-} = require('../constants');
+const { errors, defaultRuleSchema, defaults } = require('../constants');
 
-const {
-    getError,
-    shouldBypass
-} = require('../utils');
+const { getError, shouldBypass } = require('../utils');
 
 module.exports = {
     meta: {
@@ -24,7 +17,7 @@ module.exports = {
         schema: defaultRuleSchema
     },
 
-    create: function(context) {
+    create: function (context) {
         const options = context.options[1] || {};
         const testAttribute = options.testAttribute || defaults.testAttribute;
 
diff --git a/lib/utils.js b/lib/utils.js
index 09e65ca..2cdb4c7 100644
--- a/lib/utils.js
+++ b/lib/utils.js
@@ -1,20 +1,12 @@
-const {
-    elementType,
-    getProp,
-    getPropValue,
-} = require('jsx-ast-utils');
+const { elementType, getProp, getPropValue } = require('jsx-ast-utils');
 
-const {
-    defaults
-} = require('./constants');
+const { defaults } = require('./constants');
 
 // Hacky use of the Function constructor here, not sure there's a better way, other than using
 // regex.
 // https://stackoverflow.com/a/37217166/214017
 const fillTemplate = (str, vars) => {
-    vars.attribute = Array.isArray(vars.attribute)
-        ? vars.attribute.join(", or ")
-        : vars.attribute;
+    vars.attribute = Array.isArray(vars.attribute) ? vars.attribute.join(', or ') : vars.attribute;
 
     return new Function(`return \`${str}\`;`).call(vars);
 };
@@ -26,11 +18,11 @@ const getValue = (node, prop) => getPropValue(getProp(node.attributes, prop));
 const attributeTest = (node, { attribute, test }) => {
     const attrs = Array.isArray(attribute) ? attribute : [attribute];
     const type = elementType(node);
-    
+
     // One of the attributes must be present on the node
     return attrs.some((attr) => {
-      const attributeValue = getValue(node, attr);
-      return test({ attributeValue, elementType: type });
+        const attributeValue = getValue(node, attr);
+        return test({ attributeValue, elementType: type });
     });
 };
 
@@ -40,10 +32,11 @@ module.exports = {
 
     /* Determines if a particular eslint rule should not apply to a DOM not. */
     shouldBypass: (node, options, rawTests) => {
-        const tests = [ ...rawTests ];
+        const tests = [...rawTests];
 
         // Ignore elements with a `disabled` attribute.
-        const shouldIgnoreDisabled = (typeof options.ignoreDisabled === 'boolean' && options.ignoreDisabled) ||
+        const shouldIgnoreDisabled =
+            (typeof options.ignoreDisabled === 'boolean' && options.ignoreDisabled) ||
             (typeof options.ignoreDisabled === 'undefined' && defaults.ignoreDisabled);
         if (shouldIgnoreDisabled) {
             tests.unshift({
@@ -53,7 +46,8 @@ module.exports = {
         }
 
         // Ignore elements with a `readonly` attribute.
-        const shouldIgnoreReadonly = (typeof options.ignoreReadonly === 'boolean' && options.ignoreReadonly) ||
+        const shouldIgnoreReadonly =
+            (typeof options.ignoreReadonly === 'boolean' && options.ignoreReadonly) ||
             (typeof options.ignoreReadonly === 'undefined' && defaults.ignoreReadonly);
         if (shouldIgnoreReadonly) {
             tests.unshift({
@@ -68,9 +62,11 @@ module.exports = {
         tests.push({
             attribute: testAttribute,
             test: ({ attributeValue }) => {
-                return typeof attributeValue !== 'undefined' &&
+                return (
+                    typeof attributeValue !== 'undefined' &&
                     typeof attributeValue !== 'boolean' &&
-                    attributeValue !== null;
+                    attributeValue !== null
+                );
             }
         });
 
diff --git a/tests/lib/rules/anchor.js b/tests/lib/rules/anchor.js
index 0b16c09..56a8d39 100644
--- a/tests/lib/rules/anchor.js
+++ b/tests/lib/rules/anchor.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/anchor');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { anchor } = errors;
@@ -29,8 +26,8 @@ ruleTester.run('anchor', rule, {
         { code: `<link href="example.com" />` },
         { code: `<area href="example.com" />` },
         { code: `<base href="example.com" />` },
-        { code: `<a testId={ bar }>Foo</a>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<a data-testid={ bar }>Foo</a>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        { code: `<a testId={ bar }>Foo</a>`, options: ['always', { testAttribute: 'testId' }] },
+        { code: `<a data-testid={ bar }>Foo</a>`, options: ['always', { testAttribute: ['testId', 'data-testid'] }] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -43,6 +40,10 @@ ruleTester.run('anchor', rule, {
         { code: '<Foo href="bar" disabled={ bar } />', errors: [anchorError] },
         { code: '<Foo href="bar" readonly={ bar } />', errors: [anchorError] },
         { code: '<Link href="example.com" />', errors: [anchorError] },
-        { code: `<a data-test-id={ bar }>Foo</a>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(anchor.message, ["testId", "data-testid"])] }
+        {
+            code: `<a data-test-id={ bar }>Foo</a>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(anchor.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/button.js b/tests/lib/rules/button.js
index 0c24b15..d5fc8be 100644
--- a/tests/lib/rules/button.js
+++ b/tests/lib/rules/button.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/button');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { button } = errors;
@@ -27,9 +24,9 @@ ruleTester.run('button', rule, {
         { code: `<DownloadButton data-test-id='foo' />` },
         { code: `<button disabled />` },
         { code: `<button readonly />` },
-        { code: `<Button>Foo</Button>`, options: ["warn", {htmlOnly: true}] },
-        { code: `<button testId={ bar } />`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<button data-testid={ bar } />`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        { code: `<Button>Foo</Button>`, options: ['warn', { htmlOnly: true }] },
+        { code: `<button testId={ bar } />`, options: ['always', { testAttribute: 'testId' }] },
+        { code: `<button data-testid={ bar } />`, options: ['always', { testAttribute: ['testId', 'data-testid'] }] }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -45,6 +42,10 @@ ruleTester.run('button', rule, {
         { code: '<DownloadButton />', errors: [buttonError] },
         { code: '<button disabled={ foo } />', errors: [buttonError] },
         { code: '<button readonly={ foo } />', errors: [buttonError] },
-        { code: `<button data-test-id={ bar } />`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(button.message, ["testId", "data-testid"])] }
+        {
+            code: `<button data-test-id={ bar } />`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(button.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/input.js b/tests/lib/rules/input.js
index e6c6475..8cc7566 100644
--- a/tests/lib/rules/input.js
+++ b/tests/lib/rules/input.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/input');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { input } = errors;
@@ -22,8 +19,11 @@ ruleTester.run('input', rule, {
         { code: `<input data-test-id='foo' />` },
         { code: `<input disabled />` },
         { code: `<input readonly />` },
-        { code: `<input testId={ bar }>Foo</input>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<input data-testid={ bar }>Foo</input>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        { code: `<input testId={ bar }>Foo</input>`, options: ['always', { testAttribute: 'testId' }] },
+        {
+            code: `<input data-testid={ bar }>Foo</input>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }]
+        }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -34,6 +34,10 @@ ruleTester.run('input', rule, {
         { code: '<input />', errors: [inputError] },
         { code: '<input disabled={ foo } />', errors: [inputError] },
         { code: '<input readonly={ foo } />', errors: [inputError] },
-        { code: `<input data-test-id={ bar }>Foo</input>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(input.message, ["testId", "data-testid"])] }
+        {
+            code: `<input data-test-id={ bar }>Foo</input>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(input.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onChange.js b/tests/lib/rules/onChange.js
index b19533e..d138891 100644
--- a/tests/lib/rules/onChange.js
+++ b/tests/lib/rules/onChange.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/onChange');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { onChange } = errors;
@@ -28,8 +25,14 @@ ruleTester.run('onChange', rule, {
         { code: `<Bar onChange={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onChange={ () => {} } disabled />` },
         { code: `<Bar onChange={ () => {} } readonly />` },
-        { code: `<div onChange={ this.handleChange } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<div onChange={ this.handleChange } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        {
+            code: `<div onChange={ this.handleChange } testId={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: 'testId' }]
+        },
+        {
+            code: `<div onChange={ this.handleChange } data-testid={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }]
+        }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -40,6 +43,10 @@ ruleTester.run('onChange', rule, {
         { code: '<Bar onChange={ () => handleChange() }>foo</Bar>', errors: [onChangeError] },
         { code: '<Bar onChange={ () => handleChange() } disabled={ foo }>foo</Bar>', errors: [onChangeError] },
         { code: '<Bar onChange={ () => handleChange() } readonly={ foo }>foo</Bar>', errors: [onChangeError] },
-        { code: `<div onChange={ this.handleChange } data-test-id={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onChange.message, ["testId", "data-testid"])] }
+        {
+            code: `<div onChange={ this.handleChange } data-test-id={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(onChange.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onClick.js b/tests/lib/rules/onClick.js
index 6104ead..d914550 100644
--- a/tests/lib/rules/onClick.js
+++ b/tests/lib/rules/onClick.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/onClick');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 const nanoidMock = require('mock-require');
 
@@ -18,7 +15,7 @@ const onClickError = getError(onClick.message, defaults.testAttribute);
 nanoidMock('nanoid', {
     nanoid: function () {
         return 'AbYK0YPm2OWYiMaFPKLbp';
-    },
+    }
 });
 
 const id = require('nanoid');
@@ -38,56 +35,62 @@ ruleTester.run('onClick', rule, {
         { code: `<Bar onClick={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onClick={ () => {} } disabled />` },
         { code: `<Bar onClick={ () => {} } readonly />` },
-        { code: `<div onClick={ this.handleClick } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<div onClick={ this.handleClick } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        {
+            code: `<div onClick={ this.handleClick } testId={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: 'testId' }]
+        },
+        {
+            code: `<div onClick={ this.handleClick } data-testid={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }]
+        }
     ].map(parserOptionsMapper),
 
     invalid: [
         {
             code: '<div onClick={ this.handleClick } />',
             errors: [onClickError],
-            output: `<div data-test-id="${ suggestedId }" onClick={ this.handleClick } />`,
+            output: `<div data-test-id="${suggestedId}" onClick={ this.handleClick } />`
         },
         {
             code: '<div onClick={ this.handleClick }>foo</div>',
             errors: [onClickError],
-            output: `<div data-test-id="${ suggestedId }" onClick={ this.handleClick }>foo</div>`,
+            output: `<div data-test-id="${suggestedId}" onClick={ this.handleClick }>foo</div>`
         },
         {
             code: '<Bar onClick={ this.handleClick } />',
             errors: [onClickError],
-            output: `<Bar data-test-id="${ suggestedId }" onClick={ this.handleClick } />`,
+            output: `<Bar data-test-id="${suggestedId}" onClick={ this.handleClick } />`
         },
         {
             code: '<Bar onClick={ this.handleClick }>foo</Bar>',
             errors: [onClickError],
-            output: `<Bar data-test-id="${ suggestedId }" onClick={ this.handleClick }>foo</Bar>`,
+            output: `<Bar data-test-id="${suggestedId}" onClick={ this.handleClick }>foo</Bar>`
         },
         {
             code: '<Bar onClick={ () => handleClick() }>foo</Bar>',
             errors: [onClickError],
-            output: `<Bar data-test-id="${ suggestedId }" onClick={ () => handleClick() }>foo</Bar>`,
+            output: `<Bar data-test-id="${suggestedId}" onClick={ () => handleClick() }>foo</Bar>`
         },
         {
             code: '<Bar onClick={ () => handleClick() } disabled={ foo }>foo</Bar>',
             errors: [onClickError],
-            output: `<Bar data-test-id="${ suggestedId }" onClick={ () => handleClick() } disabled={ foo }>foo</Bar>`,
+            output: `<Bar data-test-id="${suggestedId}" onClick={ () => handleClick() } disabled={ foo }>foo</Bar>`
         },
         {
             code: '<Bar onClick={ () => handleClick() } readonly={ foo }>foo</Bar>',
             errors: [onClickError],
-            output: `<Bar data-test-id="${ suggestedId }" onClick={ () => handleClick() } readonly={ foo }>foo</Bar>`,
+            output: `<Bar data-test-id="${suggestedId}" onClick={ () => handleClick() } readonly={ foo }>foo</Bar>`
         },
         {
             code: '<Foo.Bar onClick={ () => handleClick() } readonly={ foo }>foo</Foo.Bar>',
             errors: [onClickError],
-            output:
-              `<Foo.Bar data-test-id="${ suggestedId }" onClick={ () => handleClick() } readonly={ foo }>foo</Foo.Bar>`,
+            output: `<Foo.Bar data-test-id="${suggestedId}" onClick={ () => handleClick() } readonly={ foo }>foo</Foo.Bar>`
         },
         {
             code: `<div onClick={ this.handleClick } data-test-id={ bar }>Foo</div>`,
-            options: ["always", {"testAttribute": ["testId", "data-testid"]}],
-            errors: [getError(onClick.message, ["testId", "data-testid"])],
-            output: `<div testId="${ suggestedId }" onClick={ this.handleClick } data-test-id={ bar }>Foo</div>` }
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(onClick.message, ['testId', 'data-testid'])],
+            output: `<div testId="${suggestedId}" onClick={ this.handleClick } data-test-id={ bar }>Foo</div>`
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onKeyDown.js b/tests/lib/rules/onKeyDown.js
index 5f2506f..c696676 100644
--- a/tests/lib/rules/onKeyDown.js
+++ b/tests/lib/rules/onKeyDown.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/onKeyDown');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { onKeyDown } = errors;
@@ -28,8 +25,14 @@ ruleTester.run('onKeyDown', rule, {
         { code: `<Bar onKeyDown={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onKeyDown={ () => {} } disabled />` },
         { code: `<Bar onKeyDown={ () => {} } readonly />` },
-        { code: `<div onKeyDown={ this.handleKeyDown } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<div onKeyDown={ this.handleKeyDown } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        {
+            code: `<div onKeyDown={ this.handleKeyDown } testId={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: 'testId' }]
+        },
+        {
+            code: `<div onKeyDown={ this.handleKeyDown } data-testid={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }]
+        }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -40,6 +43,10 @@ ruleTester.run('onKeyDown', rule, {
         { code: '<Bar onKeyDown={ () => handleKeyDown() }>foo</Bar>', errors: [onKeyDownError] },
         { code: '<Bar onKeyDown={ () => handleKeyDown() } disabled={ bar }>foo</Bar>', errors: [onKeyDownError] },
         { code: '<Bar onKeyDown={ () => handleKeyDown() } readonly={ bar }>foo</Bar>', errors: [onKeyDownError] },
-        { code: `<div onKeyDown={ this.handleKeyDown } data-test-id={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onKeyDown.message, ["testId", "data-testid"])] }
+        {
+            code: `<div onKeyDown={ this.handleKeyDown } data-test-id={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(onKeyDown.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onKeyUp.js b/tests/lib/rules/onKeyUp.js
index 9c9da19..ca2e82f 100644
--- a/tests/lib/rules/onKeyUp.js
+++ b/tests/lib/rules/onKeyUp.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/onKeyUp');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { onKeyUp } = errors;
@@ -28,8 +25,14 @@ ruleTester.run('onKeyUp', rule, {
         { code: `<Bar onKeyUp={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onKeyUp={ () => {} } disabled />` },
         { code: `<Bar onKeyUp={ () => {} } readonly />` },
-        { code: `<div onKeyUp={ this.handleKeyUp } testId={ bar }>Foo</div>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<div onKeyUp={ this.handleKeyUp } data-testid={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        {
+            code: `<div onKeyUp={ this.handleKeyUp } testId={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: 'testId' }]
+        },
+        {
+            code: `<div onKeyUp={ this.handleKeyUp } data-testid={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }]
+        }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -40,6 +43,10 @@ ruleTester.run('onKeyUp', rule, {
         { code: '<Bar onKeyUp={ () => handleKeyUp() }>foo</Bar>', errors: [onKeyUpError] },
         { code: '<Bar onKeyUp={ () => handleKeyUp() } disabled={ bar }>foo</Bar>', errors: [onKeyUpError] },
         { code: '<Bar onKeyUp={ () => handleKeyUp() } readonly={ bar }>foo</Bar>', errors: [onKeyUpError] },
-        { code: `<div onKeyUp={ this.handleKeyUp } data-test-id={ bar }>Foo</div>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onKeyUp.message, ["testId", "data-testid"])] }
+        {
+            code: `<div onKeyUp={ this.handleKeyUp } data-test-id={ bar }>Foo</div>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(onKeyUp.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/lib/rules/onSubmit.js b/tests/lib/rules/onSubmit.js
index e0bfea4..b14cc18 100644
--- a/tests/lib/rules/onSubmit.js
+++ b/tests/lib/rules/onSubmit.js
@@ -4,10 +4,7 @@
 const rule = require('../../../lib/rules/onSubmit');
 const RuleTester = require('eslint').RuleTester;
 const parserOptionsMapper = require('../../parserOptionsMapper');
-const {
-    defaults,
-    errors
-} = require('../../../lib/constants');
+const { defaults, errors } = require('../../../lib/constants');
 const { getError } = require('../../../lib/utils');
 
 const { onSubmit } = errors;
@@ -28,8 +25,14 @@ ruleTester.run('onSubmit', rule, {
         { code: `<Bar onSubmit={ () => {} } data-test-id="bar" />` },
         { code: `<Bar onSubmit={ () => {} } disabled />` },
         { code: `<Bar onSubmit={ () => {} } readonly />` },
-        { code: `<form onSubmit={ this.handleSubmit } testId={ bar }>Foo</form>`, options: ["always", {"testAttribute": "testId"}] },
-        { code: `<form onSubmit={ this.handleSubmit } data-testid={ bar }>Foo</form>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}] }
+        {
+            code: `<form onSubmit={ this.handleSubmit } testId={ bar }>Foo</form>`,
+            options: ['always', { testAttribute: 'testId' }]
+        },
+        {
+            code: `<form onSubmit={ this.handleSubmit } data-testid={ bar }>Foo</form>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }]
+        }
     ].map(parserOptionsMapper),
 
     invalid: [
@@ -40,6 +43,10 @@ ruleTester.run('onSubmit', rule, {
         { code: '<Bar onSubmit={ () => handleSubmit() }>foo</Bar>', errors: [onSubmitError] },
         { code: '<Bar onSubmit={ () => handleSubmit() } disabled={ foo }>foo</Bar>', errors: [onSubmitError] },
         { code: '<Bar onSubmit={ () => handleSubmit() } readonly={ foo }>foo</Bar>', errors: [onSubmitError] },
-        { code: `<form onSubmit={ this.handleSubmit } data-test-id={ bar }>Foo</form>`, options: ["always", {"testAttribute": ["testId", "data-testid"]}], errors: [getError(onSubmit.message, ["testId", "data-testid"])] }
+        {
+            code: `<form onSubmit={ this.handleSubmit } data-test-id={ bar }>Foo</form>`,
+            options: ['always', { testAttribute: ['testId', 'data-testid'] }],
+            errors: [getError(onSubmit.message, ['testId', 'data-testid'])]
+        }
     ].map(parserOptionsMapper)
 });
diff --git a/tests/parserOptionsMapper.js b/tests/parserOptionsMapper.js
index 0ac2b9f..8f0e8e2 100644
--- a/tests/parserOptionsMapper.js
+++ b/tests/parserOptionsMapper.js
@@ -1,30 +1,24 @@
 const defaultParserOptions = {
-  ecmaVersion: 6,
-  ecmaFeatures: {
-    jsx: true,
-  }
+    ecmaVersion: 6,
+    ecmaFeatures: {
+        jsx: true
+    }
 };
 
-module.exports = function({
-  code,
-  errors,
-  output,
-  options = [],
-  parserOptions = {},
-}) {
-  const defaultOptions = {
-    code,
-    errors,
-    options,
-    parserOptions: {
-      ...defaultParserOptions,
-      ...parserOptions,
+module.exports = function ({ code, errors, output, options = [], parserOptions = {} }) {
+    const defaultOptions = {
+        code,
+        errors,
+        options,
+        parserOptions: {
+            ...defaultParserOptions,
+            ...parserOptions
+        }
+    };
+
+    if (output) {
+        defaultOptions.output = output;
     }
-  }
 
-  if (output) {
-    defaultOptions.output = output; 
-  }
-  
-  return defaultOptions;
-}
+    return defaultOptions;
+};

From 0774c67f1d9caeed0321e0729cc3b278ea69dea4 Mon Sep 17 00:00:00 2001
From: David Calhoun <david.b.calhoun@gmail.com>
Date: Sun, 20 Aug 2023 09:33:02 -0400
Subject: [PATCH 3/4] regenerate package-lock (fixes Dependabot 4)

---
 package-lock.json | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index a4882f4..38402de 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -2666,9 +2666,9 @@
       }
     },
     "node_modules/word-wrap": {
-      "version": "1.2.3",
-      "resolved": "https://registry.npmjs.org/word-wrap/-/word-wrap-1.2.3.tgz",
-      "integrity": "sha512-Hz/mrNwitNRh/HUAtM/VT/5VH+ygD6DV7mYKZAtHOrbs8U7lvPS6xf7EJKMF0uW1KJCl0H701g3ZGus+muE5vQ==",
+      "version": "1.2.5",
+      "resolved": "https://registry.npmjs.org/word-wrap/-/word-wrap-1.2.5.tgz",
+      "integrity": "sha512-BN22B5eaMMI9UMtjrGd5g5eCYPpCPDUy0FJXbYsaT5zYxjFOckS53SQDE3pWkVoWpHXVb3BrYcEN4Twa55B5cA==",
       "dev": true,
       "engines": {
         "node": ">=0.10.0"
@@ -4691,9 +4691,9 @@
       }
     },
     "word-wrap": {
-      "version": "1.2.3",
-      "resolved": "https://registry.npmjs.org/word-wrap/-/word-wrap-1.2.3.tgz",
-      "integrity": "sha512-Hz/mrNwitNRh/HUAtM/VT/5VH+ygD6DV7mYKZAtHOrbs8U7lvPS6xf7EJKMF0uW1KJCl0H701g3ZGus+muE5vQ==",
+      "version": "1.2.5",
+      "resolved": "https://registry.npmjs.org/word-wrap/-/word-wrap-1.2.5.tgz",
+      "integrity": "sha512-BN22B5eaMMI9UMtjrGd5g5eCYPpCPDUy0FJXbYsaT5zYxjFOckS53SQDE3pWkVoWpHXVb3BrYcEN4Twa55B5cA==",
       "dev": true
     },
     "workerpool": {

From e10cb05c139e6a9bff2eefa9a4bf3559bec7ac05 Mon Sep 17 00:00:00 2001
From: David Calhoun <david.b.calhoun@gmail.com>
Date: Sun, 20 Aug 2023 09:36:07 -0400
Subject: [PATCH 4/4] Github Actions for unit testing

---
 .github/workflows/units-tests.yaml | 31 ++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 .github/workflows/units-tests.yaml

diff --git a/.github/workflows/units-tests.yaml b/.github/workflows/units-tests.yaml
new file mode 100644
index 0000000..ced3922
--- /dev/null
+++ b/.github/workflows/units-tests.yaml
@@ -0,0 +1,31 @@
+name: unit testing
+on:
+    push:
+    pull_request:
+jobs:
+    build-and-test:
+        runs-on: ubuntu-latest
+        steps:
+            # Checks out code from Github.
+            - name: Checkout repo
+              uses: actions/checkout@v3
+            # Restore cache if available.
+            - name: Restore cached dependencies
+              id: dep-cache
+              uses: actions/cache@v3
+              env:
+                  cache-name: jstoxml-cache
+              with:
+                  path: node_modules
+                  key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
+                  restore-keys: |
+                      ${{ runner.os }}-build-${{ env.cache-name }}-
+                      ${{ runner.os }}-build-
+                      ${{ runner.os }}-
+            # Fully install from scratch when no cache is available.
+            - name: Install dependencies from scratch (cache miss only)
+              if: steps.dep-cache.outputs.cache-hit != 'true'
+              run: npm i
+            - name: Unit tests
+              run: npm test
+              shell: bash