1. 20 Mar, 2019 1 commit
    • Luke Duncalfe's avatar
      Handle blank projects in CommitCollection#enrich! · 5d4e02f1
      Luke Duncalfe authored
      A project is needed in order to fetch data from gitaly. Projects can be
      absent from commits in certain rare situations (like when viewing a MR
      of a deleted fork). In these cases, assume that the enriched data is not
      needed.
      
      See this comment:
      
      https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/26144#note_152191106
      
      "It's led to a dilemma about where to "fix" this in code. I'm going
      to fix it by allowing CommitCollection#enrich! to just return unenriched
      commits when a project is missing, essentially "silently failing". I
      hope this is the right decision. It's going with the assumption that
      calls in these situations in the future are not needing the full data.
      The alternative would be to allow CommitCollection#enrich! to still
      error, but handle it in the methods that call #enrich!, however that
      might lead to brittleness in future when working with project-less MRs."
      5d4e02f1
  2. 18 Mar, 2019 2 commits
    • Luke Duncalfe's avatar
      Scope out merge commits in MergeRequest spec · 0420c239
      Luke Duncalfe authored
      Previously the code for excluding merge commits from the commit
      collection (CommitCollection#without_merge_commits) was not working
      when the commits had come from a merge request. Now that this has been
      fixed, these tests fails. They should always have been written to
      exclude merge commits when comparing.
      0420c239
    • Luke Duncalfe's avatar
      Enrich commits with full data in CommitCollection · 8ef0a9d5
      Luke Duncalfe authored
      Allow incomplete commit records to load their full data from gitaly.
      
      Commits can be based on a Hash of data retrieved from PostgreSQL, and
      this data can be intentionally incomplete in order to save space.
      
      A new method #gitaly? has been added to Gitlab::Git::Commit, which
      returns true if the underlying data source of the Commit is a
      Gitaly::GitCommit.
      
      CommitCollection now has a method #enrich which replaces non-gitaly
      commits in place with commits from gitaly.
      
      CommitCollection#without_merge_commits has been updated to call this
      method, as in order to determine a merge commit we need to have parent
      data.
      
      Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/58805
      8ef0a9d5
  3. 13 Mar, 2019 37 commits