Skip to content

Add debugging text to integration_test README#9316

Merged
srawlins merged 2 commits intoflutter:masterfrom
srawlins:docs-1
Jul 15, 2025
Merged

Add debugging text to integration_test README#9316
srawlins merged 2 commits intoflutter:masterfrom
srawlins:docs-1

Conversation

@srawlins
Copy link
Copy Markdown
Contributor

No description provided.

@srawlins srawlins requested a review from a team as a code owner July 15, 2025 18:56
@srawlins srawlins requested review from elliette and removed request for a team July 15, 2025 18:56
@srawlins
Copy link
Copy Markdown
Contributor Author

Please let me know of any other text or tricks or corrections!

CC @DanTup

* `--update-goldens`: behaves like the `--update-goldens` flag for Flutter unit
tests, updating the golden images to the results produced by the test run.

The following flags are available, but should not be used manually. To run a
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This text seems to be just dead? There are no "following flags"?

Comment thread packages/devtools_app/integration_test/README.md
Copy link
Copy Markdown
Member

@kenzieschmoll kenzieschmoll left a comment

Choose a reason for hiding this comment

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

LGTM with one request

@srawlins srawlins merged commit 7ad4173 into flutter:master Jul 15, 2025
40 checks passed
@srawlins srawlins deleted the docs-1 branch July 15, 2025 22:36
@DanTup
Copy link
Copy Markdown
Contributor

DanTup commented Jul 16, 2025

I do think we can get the debugger attached when running the tests on non-web with a few tweaks, but I need to figure out a way we can have them paused until the debugger attaches. However for web, things are a bit more broken. I've opened these issues and if we can fix them, we could update this to have something better.

hrajwade96 pushed a commit to hrajwade96/devtools that referenced this pull request Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants