Skip to content

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Mar 6, 2023

This test is flaky, and it relies on engine implementation details, which is brittle (maybe that's why it's flaky). It's also not useful, because image loading is already tested on the engine side. If that test is missing any interesting scenarios, we should add more tests.

Fixes #121707

@flutter-dashboard flutter-dashboard bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Mar 6, 2023
Copy link
Member

@ditman ditman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good riddance!

@yjbanov yjbanov added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 7, 2023
@auto-submit auto-submit bot merged commit 91dea4d into flutter:master Mar 7, 2023
@XilaiZhang
Copy link
Contributor

learnt a new word from ditman

@ditman
Copy link
Member

ditman commented Mar 7, 2023

@XilaiZhang it's also a very good Green Day song, check it out! :P

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Linux web_long_running_tests_2_5 is 10.10% flaky

3 participants