Follow-up from "Add dismissal descriptions to vulnerability report pages"

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

The following discussion from !137744 (merged) should be addressed:

  • @bmarjanovic started a discussion: (+2 comments)

    suggestion: what do you think of moving this json into a fixture, and reuse it in the rest of specs?

Description

The implementation of the dismissal_reasons descriptions does not fully test the pipeline and vulnerability helpers as those depend on the presence of a Chinese localisation which was not yet present. These localisations need to be added to enable these tests fully function.

Implementation Plan

  • backend Add Chinese fixtures for the dismissal_reason description localisations, add tests to ee/spec/helpers/ee/projects/pipeline_helper_spec.rb and ee/spec/helpers/vulnerabilities_helper_spec.rb to use them.
Edited by 🤖 GitLab Bot 🤖