patrol icon indicating copy to clipboard operation
patrol copied to clipboard

fix: Dynamic port selection flow on Android

Open piotruela opened this issue 7 months ago • 3 comments

Fixes reported bugs on dynamic port selection solution.

  • NativeAutomator gets the test server port from within the constructor. Because of that, NativeAutomatorConfig.port was deprecated. It should not be possible to manually set the port value.
  • Added --debug flag to flutter build --config-only invocation to prevent PatrolPlugin being removed from the app code.

Also fixes #2614.

piotruela avatar May 21 '25 20:05 piotruela

To view this pull requests documentation preview, visit the following URL:

docs.page/leancodepl/patrol~2636

Documentation is deployed and generated using docs.page.

docs-page[bot] avatar May 21 '25 20:05 docs-page[bot]

Hi, would be awesome if this PR can be completed :) Atm we cannot run the tests with latest flutter in our pipeline

ahlstro avatar May 26 '25 12:05 ahlstro

Hi, would be awesome if this PR can be completed :) Atm we cannot run the tests with latest flutter in our pipeline

This was already published in patrol_cli: 3.6.0: https://pub.dev/packages/patrol_cli/changelog#360. It was announced in the linked issue: https://github.com/leancodepl/patrol/issues/2614#issuecomment-2900015560

pdenert avatar May 27 '25 12:05 pdenert