Call done callback once Arkose is ready

[srijonsaha]

Changes

Call done callback inside Arkose's onReady callback to ensure Arkose is properly loaded. This'll help address the issue of Arkose taking a while to load and callers can rely on it to be sure when Arkose is fully loaded. Other captcha providers don't have a similar onReady callback so done call was moved to right after initialization.

https://developer.arkoselabs.com/docs/callbacks

References

https://auth0team.atlassian.net/browse/IAMRISK-3458

Testing

I tested this by setting a timeout to simulate the delay in Arkose loading. Here's a before and after comparison.

before.mov

after.mov

• This change adds unit test coverage
• This change adds integration test coverage

Checklist

• I have read the Auth0 general contribution guidelines
• I have read the Auth0 Code of Conduct
• All tests and linters described in the Develop section run without errors

[josh-cain]

Overall seems to be headed in the right direction 👍 , just a few clarifying questions (I'm not the most familiar with this .js).

Also, could we please include at least one test that validates the passed done() function is called after Arkose is ready?

[nandan-bhat]

Changes look good. Accepting the PR.