• Timothy Andrew's avatar
    Refactor `Issue#related_branches` · 5d88de09
    Timothy Andrew authored
    - Previously, the controller held the logic to calculate
      related branches, which was:
    
      `<branches ending with `issue.iid`> - <branches with a merge request referenced in the current issue>`
    
    - This logic belongs in the `related_branches` method, not in the
      controller. This commit makes this change.
    
    - This means that `Issue#related_branches` now needs to take a `User`.
      When we find the branches that have a merge request referenced in the
      current issue, this is limited to merge requests that the current user
      has access to.
    
    - This is not directly related to #14566, but is a related refactoring.
    5d88de09
issue_spec.rb 6.17 KB