← Back to issue list

tests: disable sentry

View original Github issue

Metadata

Project
snapcraft
Number
#2154
Type
pull request
State
merged
Author
kyrofa
Labels
—
Created
Updated
Closed

Current evaluation

Merged in snapcraft#2154 (commit aa9fdafc9), disabling Sentry during tests so test errors no longer hang waiting for user input.

Resolution: closed resolved

Reason: PR #2154 was merged as commit aa9fdafc9. It disabled Sentry reporting during tests, preventing test errors from blocking on user input.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? ----- Currently, if an error occurs during a test, it just hangs waiting for user input that is never given. Fix this by disabling Sentry reporting when under test.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in snapcraft#2154 (commit aa9fdafc9), disabling Sentry during tests so test errors no longer hang waiting for user input.
stealth/space-bunny-alpha — closed resolved Merged PR #2154 disabled Sentry during tests, preventing test errors from hanging on prompts for error-report input (commit aa9fdafc9).
stealth/space-bunny-alpha — closed resolved PR #2154 merged as commit aa9fdafc9: Sentry error reporting is disabled under test so failing tests no longer hang on the interactive prompt. Follow-ups #2187/#2214 later reworked the feature flag.
stealth/space-bunny-alpha — closed resolved Disabled Sentry during tests via merged PR #2154 (commit aa9fdafc9), preventing test errors from hanging while waiting for user input.
qwen/qwen3.6-35b-a3b — — Merged into master. Disabled Sentry error reporting during test execution to prevent hangs caused by waiting for user input. Approved by a maintainer, passed CI checks, and resulted in no coverage change.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes to disable Sentry reporting during tests, preventing hangs caused by unhandled error prompts. Test coverage remains unchanged.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2214 sentry: support disabling error reporting snapcraft merged Merged PR #2214 (commit 1c93ab107), restoring the ability to disable Sentry after regression #2154 caused tests to hang.
75%