Migrate ".ci-status-link" class to utility classes or test selectors
Summary
Currently the ".ci-status-link" class is used in different files in devopsverify areas as both style class and test selector.
Improvements
This is risky because this class is overloaded with responsibilities without a clear path of ownership. We should migrate away from this class and replace it with utility classes (for the styles) and other selectors for tests (like data-testid
).
Risks
We could face styling issues when changing this class, so pages that use this component should be verified manually with screenshots.
Involved components
These is the production code that renders this class:
app/assets/javascripts/pages/projects/blob/show/index.js
app/assets/javascripts/repository/components/last_commit.vue
app/assets/javascripts/projects/tree/components/commit_pipeline_status.vue
app/helpers/ci/status_helper.rb
Additional tests and css depend on this class.
Edited by Miguel Rincon