Skip to content

Enable running site-specific tests#13387

Merged
parlough merged 1 commit into
mainfrom
misc/test-sites
May 16, 2026
Merged

Enable running site-specific tests#13387
parlough merged 1 commit into
mainfrom
misc/test-sites

Conversation

@parlough
Copy link
Copy Markdown
Member

Contributes to #13148

@parlough parlough requested review from a team and sfshaza2 as code owners May 16, 2026 01:27
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the test-dart command to support testing site directories using the dart executable while continuing to use flutter for example projects. It introduces a tuple-based structure to track directories and their corresponding test executables and adds logic to verify the existence of test directories before execution. I have no feedback to provide.

Copy link
Copy Markdown
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm, once the build is happy

Copy link
Copy Markdown
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

oops, forgot to check approve...

@flutter-website-bot
Copy link
Copy Markdown
Collaborator

Visit the preview URL for this PR (updated for commit 2013e6e):

https://flutter-docs-prod--pr13387-misc-test-sites-l6sp6sqt.web.app

@parlough parlough merged commit e94d94e into main May 16, 2026
10 checks passed
@parlough parlough deleted the misc/test-sites branch May 16, 2026 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants