Skip to content

Remove duplicated code from MirrorsController#update

Vasilii Iakliushin requested to merge 455515_remove_mirror_controller_duplicates into master

What does this MR do and why?

Contributes to #455515 (closed)

Problem

The code is heavely duplicated between two implementations. Any attempt to change it leads to unavoidable rspec::undercoverage error.

Solution

  • Move common logic into regular MirrorsController#update method
  • Extract differences into methods and redefine them in EE
Edited by Vasilii Iakliushin

Merge request reports