Add more test coverage

auth0 · Oct 5, 2023 · 0eaa213 · 0eaa213
1 parent 7e82a73
commit 0eaa213
Show file tree

Hide file tree

Showing 9 changed files with 30 additions and 11 deletions.
diff --git a/dist/auth0.js b/dist/auth0.js
@@ -1,7 +1,7 @@
 /**
  * auth0-js v9.22.1
  * Author: Auth0
- * Date: 2023-10-04
+ * Date: 2023-10-05
  * License: MIT
  */
 
@@ -7572,7 +7572,6 @@
 	var AUTH0_PROVIDER = 'auth0';
 	var TIMEOUT_MS = 500;
 	var MAX_RETRY = 3;
-	var retryCount = 0;
 
 	var defaults$2 = {
 	  lang: 'en',
@@ -7722,6 +7721,7 @@
 	    opts.siteKey
 	  );
 	  if (opts.provider === ARKOSE_PROVIDER) {
+	    var retryCount = 0;
 	    attributes['data-callback'] = callbackName;
 	    attributes['onerror'] = function () {
 	      if (retryCount < MAX_RETRY) {
@@ -7811,6 +7811,7 @@
 	    function (arkose) {
 	      var global = globalForCaptchaProvider(challenge.provider);
 	      if (challenge.provider === ARKOSE_PROVIDER) {
+	        var retryCount = 0;
 	        arkose.setConfig({
 	          onReady: function () {
 	            if (arkoseConfig && arkoseConfig.onReady) {

diff --git a/dist/auth0.min.esm.js b/dist/auth0.min.esm.js
diff --git a/dist/auth0.min.esm.js.map b/dist/auth0.min.esm.js.map
diff --git a/dist/auth0.min.js b/dist/auth0.min.js
diff --git a/dist/auth0.min.js.map b/dist/auth0.min.js.map
diff --git a/dist/cordova-auth0-plugin.js b/dist/cordova-auth0-plugin.js
@@ -1,7 +1,7 @@
 /**
  * auth0-js v9.22.1
  * Author: Auth0
- * Date: 2023-10-04
+ * Date: 2023-10-05
  * License: MIT
  */
 

diff --git a/dist/cordova-auth0-plugin.min.js b/dist/cordova-auth0-plugin.min.js
diff --git a/src/web-auth/captcha.js b/src/web-auth/captcha.js
@@ -12,7 +12,6 @@ var ARKOSE_PROVIDER = 'arkose';
 var AUTH0_PROVIDER = 'auth0';
 var TIMEOUT_MS = 500;
 var MAX_RETRY = 3;
-var retryCount = 0;
 
 var defaults = {
   lang: 'en',
@@ -162,6 +161,7 @@ function injectCaptchaScript(element, opts, callback, setValue) {
     opts.siteKey
   );
   if (opts.provider === ARKOSE_PROVIDER) {
+    var retryCount = 0;
     attributes['data-callback'] = callbackName;
     attributes['onerror'] = function () {
       if (retryCount < MAX_RETRY) {
@@ -251,6 +251,7 @@ function handleCaptchaProvider(element, options, challenge, arkoseConfig) {
     function (arkose) {
       var global = globalForCaptchaProvider(challenge.provider);
       if (challenge.provider === ARKOSE_PROVIDER) {
+        var retryCount = 0;
         arkose.setConfig({
           onReady: function () {
             if (arkoseConfig && arkoseConfig.onReady) {

diff --git a/test/web-auth/captcha.test.js b/test/web-auth/captcha.test.js
@@ -939,6 +939,7 @@ describe('passwordless captcha rendering', function () {
     let c,
       captchaScript,
       arkoseCallback,
+      scriptErrorCallback,
       arkose,
       arkoseConfig,
       element,
@@ -973,6 +974,7 @@ describe('passwordless captcha rendering', function () {
         s.src.match('arkoselabs.com')
       );
       arkoseCallback = window[captchaScript.getAttribute('data-callback')];
+      scriptErrorCallback = captchaScript.onerror;
     });
 
     afterEach(function () {
@@ -991,6 +993,21 @@ describe('passwordless captcha rendering', function () {
       expect(arkoseCallback).to.be.a('function');
     });
 
+    it('should reinject the captcha script on error', function () {
+      expect(scriptErrorCallback).to.be.a('function');
+      scriptErrorCallback();
+      captchaScript = [...window.document.querySelectorAll('script')].find(s =>
+        s.src.match('arkoselabs.com')
+      );
+      const scriptUrl = url.parse(captchaScript.src, true);
+      expect(scriptUrl.hostname).to.equal(
+        `${challenge.clientSubdomain}.${'arkoselabs.com'}`
+      );
+      expect(scriptUrl.pathname).to.equal(
+        `/${'v2'}/${challenge.siteKey}/${'api.js'}`
+      );
+    });
+
     describe('after captcha is loaded', function () {
       let configOptions, setConfigSpy, runSpy, resetSpy;
       this.beforeEach(function () {