Decrease the execution time of the longest E2E tests #64829
+3
−3
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.
Fixes #64075
Root cause:
The test component used 5000 items × 30px = 150,000px of scroll height, requiring ~250
PageDowniterations to scroll through.Justification:
There is no justification for 5k items, the same check, reflecting the conditions from the original issue #63651 can be done with 500 elements.
Fix:
Reduced to 500 items × 20px = 10,000px (~17 iterations) while preserving the test's purpose: validating that
OverscanCount > MaxItemCounttriggers elevation of effective max.Safety measures:
We could go down even more. The fixing PR mentioned that the core of the problem was with
OverscanCount > MaxItemCount. We could use e.g. 100 items withOverscanCount=40 andMaxItemCount= 20 that would reduce the test time ~8x. But only if @javiercn does not have any arguments against that breaking the goal of the test.What's preserved:
OverscanCount="200"andMaxItemCount="100"(original relationship)ItemSize="20"(matches the Aspire issue that triggered the original fix)>= 200items rendered initially