Skip to content

RUM-1492 Have more consistent results when using the load picture sample screen#1692

Merged
xgouchet merged 1 commit into
developfrom
xgouchet/RUM-1492/have_more_consistent_successful_image_loading
Nov 9, 2023
Merged

RUM-1492 Have more consistent results when using the load picture sample screen#1692
xgouchet merged 1 commit into
developfrom
xgouchet/RUM-1492/have_more_consistent_successful_image_loading

Conversation

@xgouchet

@xgouchet xgouchet commented Nov 9, 2023

Copy link
Copy Markdown
Contributor

What does this PR do?

To ensure our E2E tests have a consistent number of event, while having some picture load failure, we replace the "pure" random logic with alternating loading failure and success

@xgouchet
xgouchet requested a review from a team as a code owner November 9, 2023 10:44
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #1692 (897dbdd) into develop (4db01f1) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1692      +/-   ##
===========================================
- Coverage    83.76%   83.63%   -0.13%     
===========================================
  Files          462      462              
  Lines        15858    15855       -3     
  Branches      2366     2365       -1     
===========================================
- Hits         13282    13259      -23     
- Misses        1939     1963      +24     
+ Partials       637      633       -4     

see 21 files with indirect coverage changes

@xgouchet
xgouchet merged commit e8500ce into develop Nov 9, 2023
@xgouchet
xgouchet deleted the xgouchet/RUM-1492/have_more_consistent_successful_image_loading branch November 9, 2023 11:48
@xgouchet xgouchet added this to the 2.3.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants