Remove axe config obj in favor of callback

maintain options object
dequelabs · Dec 2, 2019 · bdba977 · bdba977
1 parent 08ba5e3
commit bdba977
Showing 1 changed file with 15 additions and 17 deletions.
diff --git a/src/AxeObserver.mjs b/src/AxeObserver.mjs
@@ -7,19 +7,20 @@ export default class AxeObserver {
   constructor(
     violationsCallback,
     {
-      axeCoreConfiguration = {
-        reporter: 'v2',
-        checks: [
-          {
-            id: 'color-contrast',
-            options: {
-              // Prevent axe from automatically scrolling
-              noScroll: true
+      axeCoreConfigurationCallback = function(axeInstance) {
+        axeInstance.configure({
+          reporter: 'v2',
+          checks: [
+            {
+              id: 'color-contrast',
+              options: {
+                // Prevent axe from automatically scrolling
+                noScroll: true
+              }
             }
-          }
-        ]
-      },
-      axeCoreInstanceCallback
+          ]
+        })
+      }
     } = {}
   ) {
     if (typeof violationsCallback !== 'function') {
@@ -44,13 +45,10 @@ export default class AxeObserver {
     // AuditQueue sequentially runs audits when the browser is idle.
     this._auditQueue = new AuditQueue()
 
-    // Allow for registering plugins etc
+    // Configure axe, register plugins etc
     if (typeof axeInstanceCallback === 'function') {
-      axeCoreInstanceCallback(axeCore)
+      axeCoreConfigurationCallback(axeCore)
     }
-
-    // Configure axe
-    axeCore.configure(axeCoreConfiguration)
   }
   observe(targetNode) {
     if (!targetNode) {