-
Notifications
You must be signed in to change notification settings - Fork 60
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
RUM-5873: Update Session Replay integration test payloads #2318
Merged
ambushwork
merged 1 commit into
feature/sr-integration-test
from
yl/it/generate-json-payload
Oct 23, 2024
Merged
RUM-5873: Update Session Replay integration test payloads #2318
ambushwork
merged 1 commit into
feature/sr-integration-test
from
yl/it/generate-json-payload
Oct 23, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feature/sr-integration-test #2318 +/- ##
===============================================================
+ Coverage 70.35% 70.38% +0.03%
===============================================================
Files 739 739
Lines 27562 27562
Branches 4617 4617
===============================================================
+ Hits 19391 19399 +8
+ Misses 6878 6876 -2
+ Partials 1293 1287 -6 |
ambushwork
force-pushed
the
yl/it/generate-json-payload
branch
3 times, most recently
from
October 21, 2024 15:50
06fdf24
to
259d644
Compare
ambushwork
force-pushed
the
yl/it/generate-json-payload
branch
2 times, most recently
from
October 23, 2024 07:47
0c90c78
to
4cd5f76
Compare
0xnm
previously approved these changes
Oct 23, 2024
...ndroidTest/kotlin/com/datadog/android/sdk/integration/sessionreplay/BaseSessionReplayTest.kt
Outdated
Show resolved
Hide resolved
jonathanmos
previously approved these changes
Oct 23, 2024
ambushwork
force-pushed
the
yl/it/generate-json-payload
branch
from
October 23, 2024 09:17
4cd5f76
to
63fb98d
Compare
0xnm
reviewed
Oct 23, 2024
...ndroidTest/kotlin/com/datadog/android/sdk/integration/sessionreplay/BaseSessionReplayTest.kt
Outdated
Show resolved
Hide resolved
ambushwork
force-pushed
the
yl/it/generate-json-payload
branch
from
October 23, 2024 11:46
63fb98d
to
7acd741
Compare
0xnm
approved these changes
Oct 23, 2024
xgouchet
approved these changes
Oct 23, 2024
jonathanmos
approved these changes
Oct 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Updates Session replay integration test payloads by using the same config of emulator as gitlab ci,
and remove all the
@ignore
for SR tests.Some other changes:
@Ignore
is removed but@SdkSuppress
is added to ignore the SR instrumented tests on legacy-min-api task, because there is an underlying issue make test block forever.edgeToEdge
is enabled by default, which makes top content covered by the toolbar, it can lead different wireframes.Motivation
RUM-5873
Review checklist (to be filled by reviewers)