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

Reporting of Appium session success statuses to Bugsnag #728

Merged
merged 12 commits into from
Mar 6, 2025

Conversation

twometresteve
Copy link
Contributor

@twometresteve twometresteve commented Mar 4, 2025

Goal

Send reports for Appium session outcomes to Bugsnag, overloading the product as a database and UI with search functionality. This will allow us to monitoring the overall stability of Appium sessions created and the frequency with which certain errors occur.

Design

A new environment variable, MAZE_APPIUM_BUGSNAG_API_KEY is used to specific the Bugsnag project to send reports to. When an Appium session is created, a new SessionMetadata object is created to start capturing details about the session - some of which do not become available until the end of the run.

Successful Appium sessions are modelled by errors with a message of simply "Success" and severity info. When a session fails, the exception message is captured and used for the report. In future we may want to perform some processing on these messages to group them into more abstract categories before sending the report to Bugsnag.

Tests

The Maze Runner pipeline is set up to make reports and results can be found here. At the time of raising this PR, some failure cases were synthesised by artificially raising an error in the code locally.

@twometresteve twometresteve changed the title Reporting of Appium Session statuses to Bugsnag Reporting of Appium session success statuses to Bugsnag Mar 4, 2025
@twometresteve twometresteve changed the base branch from main to tms/deprecated-devices March 5, 2025 23:43
@twometresteve twometresteve requested a review from Cawllec March 6, 2025 11:27
Base automatically changed from tms/deprecated-devices to main March 6, 2025 12:10
@twometresteve twometresteve merged commit 392104e into main Mar 6, 2025
24 checks passed
@twometresteve twometresteve deleted the tms/appium-reporter branch March 6, 2025 13:22
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