Skip to content

Conversation

@Kendru98
Copy link
Collaborator

Fix for #2853

Copy link
Collaborator

@pdenert pdenert left a comment

Choose a reason for hiding this comment

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

LGMT

@@ -33,6 +33,7 @@ class WebTestBackend {

final result = await _processManager.run([
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should extract content of this run method call to separate function toFlutterBuildInvocation like it is already donect for ios and macos. This way we'll be able to test this

Copy link
Collaborator

@pdenert pdenert left a comment

Choose a reason for hiding this comment

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

LGTM

@Kendru98 Kendru98 force-pushed the fix/2853-passing-flutter-command-not-work-on-the-web branch from ea81730 to 14386f0 Compare January 19, 2026 12:28
@Kendru98 Kendru98 merged commit 09e3e99 into master Jan 19, 2026
9 of 15 checks passed
@Kendru98 Kendru98 deleted the fix/2853-passing-flutter-command-not-work-on-the-web branch January 19, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: patrol_cli Related to the patrol_cli package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants