Skip to content

Get reasonable output when a stream test fails. - #154377

Merged
auto-submit[bot] merged 4 commits into
flutter:masterfrom
matanlurey:make-test-expectation-better
Sep 3, 2024
Merged

auto-submit[bot] merged 4 commits into
flutter:masterfrom
matanlurey:make-test-expectation-better

Conversation

@matanlurey

Copy link
Copy Markdown
Contributor

Debugging #154268 was hard; when I removed --disable-dart-dev I would get the following error from this test:

Screenshot 2024-08-29 at 4 25 43 PM

After my change, this is what the error looks like:

Screenshot 2024-08-29 at 4 26 50 PM

In general we should embrace more of the test package and matchers when appropriate.

@matanlurey
matanlurey requested a review from jtmcdole August 29, 2024 23:29
@matanlurey
matanlurey force-pushed the make-test-expectation-better branch from 4f728ae to 32ee400 Compare September 3, 2024 19:22
@matanlurey

Copy link
Copy Markdown
Contributor Author

@natebosch helped me offline - the reason: ... string was causing a timeout.

The printOnFailure method looks like it's a better way to say "show me the stderr if this tests fails".

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 3, 2024
@auto-submit

auto-submit Bot commented Sep 3, 2024

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/flutter/154377, due to - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Sep 3, 2024
@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 3, 2024
@auto-submit
auto-submit Bot merged commit f070ffc into flutter:master Sep 3, 2024
@matanlurey
matanlurey deleted the make-test-expectation-better branch September 3, 2024 21:37
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 12, 2024
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants