tests: use smoke intergration tests for macOS
Metadata
Current evaluation
Merged PR #458 (commit f2d603e) switched macOS tests to use the smoke integration tests, keeping coverage at 99%. The smoketest marker exists in the test suite and CI.
Resolution: closed resolved
Reason: PR #458 was approved by 2 reviewers and merged into main as commit f2d603e ("tests: use smoke intergration tests for macOS (#458)"), adding a pytest.mark.smoketest basic test. Codecov confirmed 99% coverage. A later commit a3af10b (#982) temporarily disabled the macos smoke tests in CI but did not revert this change.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #458 (commit f2d603e) switched macOS tests to use the smoke integration tests, keeping coverage at 99%. The smoketest marker exists in the test suite and CI. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR #458 to update the test suite with smoke integration tests for macOS, maintaining 99% code coverage. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#982
(confidence 80%)
Commit a3af10b (#982) later temporarily disabled the macos smoke tests in CI, a follow-up to this change.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #457 ci(tests): use simplify intergration tests for macos | craft-providers | closed | Abandoned unmerged after long inactivity; its macOS integration-test workflow was added by merged PR #460 in commit bc946f0. | |
| #982 ci: temporarily disable macos smoke tests | craft-providers | merged | Merged PR #982 temporarily disabled macOS smoke-test CI after external runner failures; the underlying problem remains tracked in #452. | |
| #460 tests(ci): add weekly slow tests for macOS | craft-providers | merged | Merged weekly slow macOS integration tests into CI via PR #460, commit bc946f0. | |
| #1823 ci(macos): add a macos init/pack smoke test | charmcraft | merged | Merged as 10216e4e; PR #1823 added macOS init/pack smoke coverage for #1767, following #1822’s craft-application 4.1 update in 6a2501ee. |