← Back to issue list

ci: remove broken macos integration tests

View original Github issue

Metadata

Project
craft-application
Number
#77
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Removed broken macOS integration tests from CI. Merged after approval by two reviewers. Author noted tests can be re-enabled once the image is fixed.

Suggested action:

No scores available.

Issue body

We can add back once the image is fixed. - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Removed broken macOS integration tests from CI. Merged after approval by two reviewers. Author noted tests can be re-enabled once the image is fixed.
qwen3.6-35b-a3b-mtp-q6 Merged into main, removing broken macOS integration tests from CI. Tests will be re-added once the underlying image is fixed. Code coverage remains unchanged.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2606 ci: disable macos tests charmcraft merged Merged changes to disable macOS CI tests, resolving issue #2605. Approved by two reviewers and passed all required CI checks before integration.
86%
#279 ci: remove test runs on macos-latest rockcraft merged Merged changes to remove macOS test runs from the CI workflow. Approved by reviewers and passed checks, as macOS workers were frequently busy and most tests are skipped on non-Linux systems anyway.
77%
#307 tests: disable broken integration test craft-application merged Merged a temporary workaround disabling a broken integration test to restore CI functionality, pending a permanent fix in issue #306. Approved by two reviewers with passing checks.
77%
#1823 ci(macos): add a macos init/pack smoke test charmcraft merged Merged a macOS init/pack smoke test into CI. Fixes #1767, depends on #1822. Approved by two reviewers, temporarily paused for comments, then merged with 48 lines added.
74%
#982 ci: temporarily disable macos smoke tests craft-providers merged Merged after approval, temporarily disabling macOS smoke tests in CI due to external failures. Awaits fix for issue #452.
73%
#2635 ci: disable macos tests for Charmcraft 3.5 charmcraft merged Merged PR temporarily disabling macOS tests for Charmcraft 3.5 CI. macOS testing will be re-enabled later to resolve issue #2605. Approved by two reviewers and merged after passing required checks.
73%
#1509 test: ignore failing init test on macos 11 charmcraft merged Merged a temporary workaround to skip a failing init test on macOS 11, unblocking CI while issue #1508 is resolved. Approved by two reviewers with a five-line change.
72%
#2520 fix: make integration tests work on ARM Mac runners charmcraft closed Closed without merge. Changes to fix ARM Mac integration tests were reverted after reviewer feedback indicated they worsened the issue. The contributor identified missing expected manifest entries as the root cause but did not resubmit a working fix.
72%
#1910 ci: fix tics tests charmcraft merged Merged a single-file update to fix TICS CI tests. Approved by two reviewers with no unresolved comments. The change resolves failing continuous integration checks.
71%
#1197 ci: temporarily disable externally broken spread tests charmcraft merged Merged to temporarily disable broken spread tests, restoring CI. Breakage stems from third-party dependency issues. Tests will be re-enabled once upstream fixes land. Approved by one reviewer, modified 3 files.
70%