Restructure assertions to remove magic numbers

[bidetofevil]

Goal

Tweaked tests so the timing assertions are more readible.

[bidetofevil]

• Make FirstDrawDetector a singleton #1796
• Add unregistering of draw callback #1795
• Add render detection to UI Load traces #1794
• Extract frame draw detection to its own component #1793
• Restructure assertions to remove magic numbers #1791 👈 (View in Graphite)
• API and integration tests for manually ending UI load traces #1790
• Allow all activities to ge traced by default #1789
• Remove the reset event #1788
• Change start time of UI Load traces #1787
• Align event emitter names with lifecycle methods #1786
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.47%. Comparing base (c1c937d) to head (85be090).
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1791   +/-   ##
=======================================
  Coverage   85.47%   85.47%           
=======================================
  Files         467      467           
  Lines       10964    10964           
  Branches     1629     1629           
=======================================
  Hits         9372     9372           
  Misses        861      861           
  Partials      731      731           

see 2 files with indirect coverage changes

[fractalwrench]

LGTM

[bidetofevil]

Merge activity

• Jan 8, 11:04 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Jan 8, 11:06 AM EST: Graphite rebased this pull request as part of a merge.
• Jan 8, 11:21 AM EST: A user merged this pull request with Graphite.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Files