Skip to content

Write tests for CLOSED / MOVED badge in issue list

The following discussion from gitlab-foss!32959 (closed) should be addressed:

  • @winh started a discussion:

    You’ve made some app changes, but didn’t add any tests.

    • It looks like we don't have tests for the CLOSED situation yet. This means writing a new test suite from scratch which requires significant effort.
    • The change is straight-forward and small.
    • The change works around a problem that we have right now (see thread in Slack).
    • The change was tested locally (see screenshots in the description).

    Therefore I suggest merging this without automated tests and creating a follow-up issue to add them in a separate merge request.