Project

General

Profile

Actions

Bug #39248

open

smart-proxy test failures due to rake 13.4.2 breaking ci_reporter_test_unit

Added by Ondřej Gajdušek about 2 months ago. Updated about 1 month ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

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

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


Related issues 1 (0 open1 closed)

Related to Smart Proxy - Bug #39315: smart-proxy test failures due to rake 13.4.2 breaking ci_reporter_test_unitClosedNadja HeitmannActions
Actions #1

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
Actions #2

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
Actions #3

Updated by Nadja Heitmann about 1 month ago

  • Pull request deleted (https://github.com/theforeman/smart-proxy/pull/938)
Actions #4

Updated by Oleh Fedorenko about 1 month ago

  • Status changed from Ready For Testing to New
Actions

Also available in: Atom PDF