-
Notifications
You must be signed in to change notification settings - Fork 161
feat: add web integration test setup #3295
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
base: main
Are you sure you want to change the base?
Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) |
Signed-off-by: venilinvasilev <venilin.vasilev@gmail.com>
bed4c6a
to
694e455
Compare
Signed-off-by: venilinvasilev <venilin.vasilev@gmail.com>
694e455
to
981f2c0
Compare
Codecov Report❌ Patch coverage is
... and 4 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Signed-off-by: venilinvasilev <venilin.vasilev@gmail.com>
Signed-off-by: venilinvasilev <venilin.vasilev@gmail.com>
Signed-off-by: venilinvasilev <venilin.vasilev@gmail.com>
Great work so far. I’d like to propose adding two distinct React environments to test whether all transactions run successfully in both setups. This is important because the majority of our web clients use React with the SDK. The two environments should differ in their module bundlers—one using Webpack and the other using Vite. We’ve encountered compatibility issues in the past with different bundlers, so this approach will help us proactively address potential problems. This setup would also allow us to:
I understand this would require considerable effort, might involve adding new development dependencies to the SDK, changes in the tests and some rework on the tests. However, the long-term benefit of reducing recurring compatibility issues makes this a worthwhile investment. The current implementation is solid; however, I’d like to align it as closely as possible with the real-world use cases our users encounter. |
Description:
WIP