Skip to content

Conversation

@eudamniac
Copy link

@eudamniac eudamniac commented Nov 16, 2025

Attempt to add cordova integration tests - following #27 (comment).

Mocks InAppBrowser plugin by providing a handle for registering event listeners. The urls for the InAppBrowser will be opened in a separate window, leaving the initial page active as it would be with a cordova app.

Next steps could be:

  • Add more tests for cordova adapter
  • Add mock and tests for native cordova adapter
  • Refactor

@jonkoops jonkoops force-pushed the cordova-integration-tests branch from db748fb to da1630f Compare November 18, 2025 16:18
@jonkoops
Copy link
Contributor

Did a quick rebase to include #218 so we can see if the tests will pass.

@eudamniac
Copy link
Author

Did a quick rebase to include #218 so we can see if the tests will pass.

Thanks! Looks like Cordova adapter › allow loaderror on InAppBrowser before finishing login is failing as expected, since there is no fix for #208 included at the moment. I could skip/remove this test for now.

Additionally I see the Firefox - Cross origin tests failing and will have a look at them, I think I missed those those locally some tweaks to the playwright setup.

@eudamniac
Copy link
Author

Fix the tests in 7fbc7fb

@eudamniac
Copy link
Author

@jonkoops just wanted to quickly follow up if I should continue in this direction?

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.

2 participants