-
Notifications
You must be signed in to change notification settings - Fork 9
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
test: in-app message displayed on wrong page #357
base: main
Are you sure you want to change the base?
Conversation
Sample app builds 📱Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #357 +/- ##
============================================
- Coverage 54.32% 52.78% -1.54%
Complexity 282 282
============================================
Files 109 109
Lines 2511 2582 +71
Branches 355 366 +11
============================================
- Hits 1364 1363 -1
- Misses 1025 1095 +70
- Partials 122 124 +2 ☔ View full report in Codecov by Sentry. |
Build available to test |
📏 SDK Binary Size Comparison ReportNo changes detected in SDK binary size ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be alot lighter but given the way webclient is setup right now, this does the job ! thank you for adding this.
parameters = buildMap { | ||
put("route", route) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
parameters = buildMap { | |
put("route", route) | |
} | |
parameters = mapOf("route" to route) |
buildMap
seems overkill considering the use-case? 😄
part of MBL-353
Changes
BaseIntegrationTest
to allow adding modules when initializing the SDK