Skip to content

Fix flaky ANR detection test#1123

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/fix-flaky-anr-test
Nov 4, 2022
Merged

Fix flaky ANR detection test#1123
0xnm merged 1 commit into
developfrom
nogorodnikov/fix-flaky-anr-test

Conversation

@0xnm

@0xnm 0xnm commented Nov 3, 2022

Copy link
Copy Markdown
Member

What does this PR do?

This change fixes flaky test by adding a bit more delay for the main thread, because otherwise it seems it can continue the execution before the ANR detection thread (which is sleeping for the same amount of time here).

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm requested a review from a team as a code owner November 3, 2022 08:20
@codecov-commenter

codecov-commenter commented Nov 3, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1123 (cf9c7bb) into develop (8ba3c1f) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1123      +/-   ##
===========================================
+ Coverage    83.24%   83.26%   +0.02%     
===========================================
  Files          273      273              
  Lines         9372     9372              
  Branches      1504     1504              
===========================================
+ Hits          7801     7803       +2     
+ Misses        1153     1150       -3     
- Partials       418      419       +1     
Impacted Files Coverage Δ
...g/internal/domain/event/DdSpanToSpanEventMapper.kt 96.15% <0.00%> (-3.85%) ⬇️
...android/rum/internal/ndk/DatadogNdkCrashHandler.kt 87.57% <0.00%> (-1.62%) ⬇️
.../android/rum/internal/monitor/DatadogRumMonitor.kt 92.05% <0.00%> (-0.57%) ⬇️
...ndroid/core/internal/persistence/file/EventMeta.kt 80.00% <0.00%> (ø)
...rc/main/java/com/datadog/opentracing/DDTracer.java 56.07% <0.00%> (+0.84%) ⬆️
...ain/java/com/datadog/opentracing/PendingTrace.java 58.62% <0.00%> (+2.59%) ⬆️
...droid/rum/tracking/FragmentViewTrackingStrategy.kt 86.96% <0.00%> (+4.35%) ⬆️
...android/log/internal/logger/TelemetryLogHandler.kt 100.00% <0.00%> (+25.00%) ⬆️

@0xnm
0xnm merged commit a669a1f into develop Nov 4, 2022
@0xnm
0xnm deleted the nogorodnikov/fix-flaky-anr-test branch November 4, 2022 09:05
@xgouchet xgouchet added this to the 1.15.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants