Use testtools to run the examples tests.
Metadata
Current evaluation
Merged PR #122 as commit 47e9b6c06, switching examples tests to testtools for clearer failure output; merged prerequisite PR #107 established the suite.
Resolution: closed resolved
Reason: PR #122 was merged in commit 47e9b6c06 and updated the examples test runner to use testtools. Review feedback also called for adding python3-testtools to debian/control.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Just by using testtools we improve a little the output of the tests in case of error:
ERROR: examples_tests.tests.TestSnapcraftExamples.test_example(webchat)
Requires #107.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #122 as commit 47e9b6c06, switching examples tests to testtools for clearer failure output; merged prerequisite PR #107 established the suite. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #122 (commit 47e9b6c06) switched the examples tests to testtools, improving failure output; it built on merged PR #107. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Switches examples tests to testtools for improved error output. A reviewer noted the need to add python3-testtools to debian/control build dependencies. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after adding a python3-testtools build dependency to debian/control. Switches examples tests to testtools for improved error output, requiring PR #107. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#107
(confidence 100%)
Explicit prerequisite that introduced the examples test suite.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #272 Fix the examples tests using testtools as the base. | snapcraft | merged | Merged PR #272 updated the examples tests to use testtools, fixing LP #1539196 in commit 5f39d00e9 (merge 7e5a479ae). | |
| #82 Making the examples individual tests | snapcraft | merged | Merged PR #82 split the examples suite into individual tests, ensuring every example runs and making failures easier to pinpoint (commit 48064f6e1). |