Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to skip recaptcha services in e2e env #53

Closed
wants to merge 5 commits into from
Closed

Add option to skip recaptcha services in e2e env #53

wants to merge 5 commits into from

Conversation

brunoocasali
Copy link
Contributor

  • Added sinonjs for mocking
  • Added basic tests covering @skip option usage + normal usage
  • Improve testing setup by upgrading base libraries (qunit/qunit-dom etc)

Closes #52

@brunoocasali
Copy link
Contributor Author

I'll be glad to improve this setup + testing more use cases, this was a #1 try only to take the ideas out of my head!

@brunoocasali
Copy link
Contributor Author

brunoocasali commented Feb 27, 2021

This PR makes a HUGE difference in my use cases:

before:
image

after:
image

@sinankeskin sinankeskin self-requested a review February 26, 2022 09:56
@sinankeskin
Copy link
Member

Hey @brunoocasali Thanks for the PR. I'm planning to modernize the addon with latest JS syntax and to the latest LTS. I'll check your PR after and if everything is ok I'll merge.

@sinankeskin
Copy link
Member

Hey @brunoocasali could you rebase your branch to master and fix the conflict and of course you need to use the skip option from args like @Skip

@brunoocasali
Copy link
Contributor Author

Yeah I can do that! 👯

@brunoocasali brunoocasali mentioned this pull request Mar 11, 2022
@brunoocasali
Copy link
Contributor Author

Closed, see #76

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable GRecaptcha in e2e tests
2 participants