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.

Sign up for GitHub

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

Jump to bottom

Fix RUM Crash Receiver Flakiness #1138

Merged
merged 1 commit into from
Jan 20, 2023
Merged

Fix RUM Crash Receiver Flakiness #1138

merged 1 commit into from
Jan 20, 2023

Conversation

maxep
Copy link
Member

@maxep maxep commented Jan 20, 2023

What and why?

Use no crash count in view event mock.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes

Custom CI job configuration (optional)

  • Run unit tests
  • Run integration tests
  • Run smoke tests

@maxep maxep requested a review from a team as a code owner January 20, 2023 08:57
@maxep maxep self-assigned this Jan 20, 2023
@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Jan 20, 2023 •
edited
Loading

Datadog Report

Branch report: maxep/fix-rum-cr-receiver-test
Commit report: 45f9c0e

✅ dd-sdk-ios: 0 Failed, 0 New Flaky, 116 Passed, 0 Skipped, 4m 7.13s Wall Time

@maxep maxep changed the title Fix RUM crash Receiver Flakiness Fix RUM Crash Receiver Flakiness Jan 20, 2023
@maxep maxep merged commit dcf1d5f into develop Jan 20, 2023
@maxep maxep deleted the maxep/fix-rum-cr-receiver-test branch January 20, 2023 09:59
@fuzzybinary fuzzybinary mentioned this pull request Feb 23, 2023
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maciejburda maciejburda maciejburda approved these changes

@plousada plousada plousada approved these changes

Assignees

@maxep maxep

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maxep @plousada @maciejburda