Bug #39248
opensmart-proxy test failures due to rake 13.4.2 breaking ci_reporter_test_unit
Description
The recent release of rake 13.4.2 has broken `ci_reporter_test_unit`, which is causing test failures in `smart-proxy` across CI environments.
Observed Failures¶
- GitHub Actions: Job 71853766793 - https://github.com/theforeman/smart-proxy/actions/runs/24364942365/job/71853766793?pr=936
- Jenkins Nightly: https://ci.theforeman.org/job/smart-proxy-develop-source-release/824/
Background¶
The `ci_reporter_test_unit` gem has been unmaintained for 10 years. While a fix has been proposed upstream https://github.com/ci-reporter/ci_reporter_test_unit/pull/13, it is unlikely to be merged or released.
Furthermore, there is a desire to modernize the `smart-proxy` test suite. Currently, Jenkins relies on `ci_reporter` for XML reporting https://github.com/theforeman/jenkins-jobs/blob/eed6bf857eeed6b3d78d5b9e12006129d247c556/theforeman.org/pipelines/test/smart-proxy.groovy#L72-L73
Proposed Plan¶
1. Migrate from `test-unit` to `minitest`.
2. Switch to `minitest-reporters` for CI reporting to replace the broken `ci_reporter`.
3. (Optional/Short-term) Only load CI reporting gems when `JENKINS_URL` is detected to avoid unnecessary breakages in other environments.
Additional Context¶
- Discussion on PR 936: https://github.com/theforeman/smart-proxy/pull/936#issuecomment-4271823297
- Discussion in the Matrix channel: https://matrix.to/#/!dxfWqeLNsImROheGXo:matrix.org/$177668282110872ycdWm:matrix.org?via=matrix.org&via=ctrl-c.liu.se&via=ansible.im
Updated by The Foreman Bot about 2 months ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/smart-proxy/pull/938 added
Updated by Nadja Heitmann about 1 month ago
- Related to Bug #39315: smart-proxy test failures due to rake 13.4.2 breaking ci_reporter_test_unit added
Updated by Nadja Heitmann about 1 month ago
- Pull request deleted (
https://github.com/theforeman/smart-proxy/pull/938)
Updated by Oleh Fedorenko about 1 month ago
- Status changed from Ready For Testing to New