Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add workaround for too long test names on Android #1732

Closed
wants to merge 3 commits into from

Conversation

bartekpacia
Copy link
Contributor

@bartekpacia bartekpacia commented Sep 20, 2023

This PR is aimed at #1725

@github-actions github-actions bot added the package: patrol Related to the patrol package (native automation, test bundling) label Sep 20, 2023
@bartekpacia bartekpacia marked this pull request as draft September 20, 2023 12:42
@bartekpacia
Copy link
Contributor Author

DO NOT MERGE right now

This has to be done on the Dart side (because Dart-side tests and native tests must have the same names).

This means the max character limit will also have to apply to iOS.

@bartekpacia
Copy link
Contributor Author

I'd prefer to wait until it's fixed upstream (link).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: patrol Related to the patrol package (native automation, test bundling)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant