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

Use Appium to get iOS E2E logs #1480

Merged
merged 1 commit into from
Sep 13, 2022
Merged

Use Appium to get iOS E2E logs #1480

merged 1 commit into from
Sep 13, 2022

Conversation

nickdowell
Copy link
Contributor

Goal

Make it easier to investigate test failures by including device syslog in maze_output.

Changeset

Uses Appium's get_log method to fetch the device log for each scenario.

@twometresteve once proven, this probably should find its way into maze-runner.
On Android, I think get_log('logcat') should work.

Testing

Verified logs are populated when running locally.

@github-actions
Copy link

Bugsnag.framework binary size did not change - 757,680 bytes

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  [ = ]       0  [ = ]       0    TOTAL

Generated by 🚫 Danger

Copy link
Contributor

@twometresteve twometresteve left a comment

Choose a reason for hiding this comment

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

Were you expecting at least one of the pipeline steps to upload files as buildkite artifacts?

@nickdowell
Copy link
Contributor Author

Were you expecting at least one of the pipeline steps to upload files as buildkite artifacts?

Only failed scenarios have their logs uploaded, so I wasn't expecting to see any in a successful build.

Copy link
Contributor

@twometresteve twometresteve left a comment

Choose a reason for hiding this comment

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

Nice

@nickdowell nickdowell merged commit 5422f32 into next Sep 13, 2022
@nickdowell nickdowell deleted the nickdowell/e2e-ios-logs branch September 13, 2022 08:45
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.

2 participants