Skip to content

[CP-stable]fix reorderable_list_test.dart - #164050

Merged
auto-submit[bot] merged 1 commit into
flutter:flutter-3.29-candidate.0from
flutteractionsbot:cp-stable-78c62b00cc58b785ae1d6fca44291634b0c8b3c0
Feb 25, 2025
Merged

auto-submit[bot] merged 1 commit into
flutter:flutter-3.29-candidate.0from
flutteractionsbot:cp-stable-78c62b00cc58b785ae1d6fca44291634b0c8b3c0

Conversation

@flutteractionsbot

@flutteractionsbot flutteractionsbot commented Feb 25, 2025 •

Copy link
Copy Markdown
Contributor

This pull request is created by automatic cherry pick workflow
Please fill in the form below, and a flutter domain expert will evaluate this cherry pick request.

Issue Link:

What is the link to the issue this cherry-pick is addressing?

#164128

Changelog Description:

Explain this cherry pick in one line that is accessible to most Flutter developers. See best practices for examples

This cherry pick lands a fix to a test that is currently causing all presubmits to fail on the 3.29-candidate.0 stable release branch. The failure fixed in this test is dependent on the random ordering seed, which is why it didn't fail before. The current seed in which it fails is --test-randomize-ordering-seed=20250221.

Impact Description:

What is the impact (ex. visual jank on Samsung phones, app crash, cannot ship an iOS app)? Does it impact development (ex. flutter doctor crashes when Android Studio is installed), or the shipping production app (the app crashes on launch)

The only change is to reduce the flakiness of reorderable_list.test.dart.

Workaround:

Is there a workaround for this issue?

Change the seed?

Risk:

What is the risk level of this cherry-pick?

  • Low
  • Medium
  • High

Test Coverage:

Are you confident that your fix is well-tested by automated tests?

  • Yes
  • No

Validation Steps:

What are the steps to validate that this fix works?

 flutter test /Users/jmccandless/Projects/flutter/packages/flutter/test/material/reorderable_list_test.dart --test-randomize-ordering-seed=20250221

Fix `reorderable_list_test.dart`
@flutteractionsbot flutteractionsbot added the cp: review Cherry-picks in the review queue label Feb 25, 2025
@flutteractionsbot

Copy link
Copy Markdown
Contributor Author

@justinmc please fill out the PR description above, afterwards the release team will review this request.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages labels Feb 25, 2025

@eyebrowsoffire eyebrowsoffire left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@justinmc justinmc added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 25, 2025
@justinmc justinmc self-assigned this Feb 25, 2025
@auto-submit
auto-submit Bot merged commit f662dd5 into flutter:flutter-3.29-candidate.0 Feb 25, 2025
auto-submit Bot pushed a commit that referenced this pull request Feb 28, 2025
This PR updates the changelog for this cherry pick: #164050.

If that cherry pick was successful, the presubmit test should not fail on this PR.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
@jasonachkar jasonachkar mentioned this pull request Mar 11, 2025
9 tasks
Heng-code11 added a commit to Heng-code11/untitled that referenced this pull request Mar 12, 2025
This PR updates the changelog for this cherry pick: flutter/flutter#164050.

If that cherry pick was successful, the presubmit test should not fail on this PR.
@reidbaker reidbaker mentioned this pull request Apr 4, 2025
7 of 9 tasks
Fintasys pushed a commit to Fintasys/flutter that referenced this pull request May 14, 2025
This PR updates the changelog for this cherry pick: flutter#164050.

If that cherry pick was successful, the presubmit test should not fail on this PR.
@flutteractionsbot
flutteractionsbot deleted the cp-stable-78c62b00cc58b785ae1d6fca44291634b0c8b3c0 branch July 20, 2026 17:04
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 cp: review Cherry-picks in the review queue framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants