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

Make short error messages descriptive in iOS reports #1886

Closed
fylyppo opened this issue Nov 9, 2023 · 3 comments · Fixed by #2114
Closed

Make short error messages descriptive in iOS reports #1886

fylyppo opened this issue Nov 9, 2023 · 3 comments · Fixed by #2114
Labels
P2 Issues not at the top of the work list package: patrol Related to the patrol package (native automation, test bundling) platform: ios iOS is affected

Comments

@fylyppo
Copy link
Collaborator

fylyppo commented Nov 9, 2023

This issue has been edited by me (@bartekpacia)

Expected behavior

Reproduce:

  1. Make one test failing
  2. Run the failing test locally on iOS Simulator (patrol test -t integration_test/failing_test.dart)
  3. After run finishes, open xcresult report.

Result

Zrzut ekranu 2023-11-9 o 23 07 18

Actual behavior

Reproduce

  1. Make one test failing
  2. Run the failing test on BrowserStack
  3. After run, check logs or open xcresult report.
  4. Notice that cause of test failure is not present
Zrzut ekranu 2023-11-9 o 22 25 15

More context

I run my tests on Browserstack and when I look into xcresult file there's only a short message per test case. This is very annoying.

More random ideas:

@bartekpacia bartekpacia added package: patrol Related to the patrol package (native automation, test bundling) platform: ios iOS is affected labels Nov 9, 2023
@mateuszwojtczak mateuszwojtczak added the P1 High-priority issues at the top of the work list label Nov 15, 2023
@bartekpacia bartekpacia changed the title Make short error messages descriptive in iOS reports Custom error message is missing from iOS reports on BrowserStack Nov 22, 2023
@bartekpacia bartekpacia changed the title Custom error message is missing from iOS reports on BrowserStack Make short error messages descriptive in iOS reports Nov 22, 2023
@jBorkowska jBorkowska added P2 Issues not at the top of the work list and removed P1 High-priority issues at the top of the work list labels Nov 27, 2023
@fylyppo
Copy link
Collaborator Author

fylyppo commented Dec 4, 2023

@bartekpacia
Copy link
Contributor

bartekpacia commented Dec 6, 2023

To reproduce build of the native iOS app run any native iOS app with a one failing test, for example browserstack/xcuitest-sample-browserstack (modify code to make test failing). Make sure to run on XCUITest v2 API (the one that uses XCTestPlans).

To reproduce build of the Flutter app:

  1. Install Flutter SDK
  2. Install Patrol CLI: dart pub global activate patrol_cli
  3. Clone this repository: git clone git clone [email protected]:leancodepl/patrol.git, cd packages/patrol/example.
  4. Make sure the app builds fine for iOS device: flutter build ios --release. Fix any code signing issues that might arise.
  5. Upload the Flutter app and instrumentation build to BrowserStack using our bs_ios. It can be installed with brew tap leancodepl/tools && brew install mobile-tools.

Please ask in case of any questions, we're be happy to help.

For BrowserStack people: see also #1923

Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar problem, please file a new issue. Make sure to follow the template and provide all the information necessary to reproduce the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P2 Issues not at the top of the work list package: patrol Related to the patrol package (native automation, test bundling) platform: ios iOS is affected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants