Use Gitlab::Git:: for git features across application

Dmitriy Zaporozhets
1 parent bb06e905
Showing 11 changed files with 32 additions and 64 deletions Show diff stats
app/controllers/blame_controller.rb
app/controllers/compare_controller.rb
app/models/commit.rb
app/models/gollum_wiki.rb
app/models/repository.rb
app/models/wiki_page.rb
app/views/blame/show.html.haml
app/views/compare/show.html.haml
app/views/wikis/show.html.haml
lib/gitlab/git/commit.rb
spec/support/test_env.rb
@@ -8,7 +8,6 @@ class BlameController &lt; ProjectResourceController
   before_filter :require_non_empty_project
   def show
-    @repo = @project.repo
-    @blame = Grit::Blob.blame(@repo, @commit.id, @path)
+    @blame = Gitlab::Git::Blame.new(project.repository, @commit.id, @path)
   end
 end
@@ -8,12 +8,12 @@ class CompareController &lt; ProjectResourceController
   end
   def show
-    result = Commit.compare(project, params[:from], params[:to])
+    compare = Gitlab::Git::Compare.new(project.repository, params[:from], params[:to])
-    @commits       = result[:commits]
-    @commit        = result[:commit]
-    @diffs         = result[:diffs]
-    @refs_are_same = result[:same]
+    @commits       = compare.commits
+    @commit        = compare.commit
+    @diffs         = compare.diffs
+    @refs_are_same = compare.same
     @line_notes    = []
   end
@@ -8,6 +8,10 @@ class Commit
   #
   DIFF_SAFE_SIZE = 100
+  def self.decorate(commits)
+    commits.map { |c| self.new(c) }
+  end
+
   attr_accessor :raw
   def initialize(raw_commit)
@@ -114,5 +114,4 @@ class GollumWiki
   def path_to_repo
     @path_to_repo ||= File.join(Gitlab.config.gitlab_shell.repos_path, "#{path_with_namespace}.git")
   end
-
 end
@@ -13,13 +13,13 @@ class Repository
   def commits(ref, path = nil, limit = nil, offset = nil)
     commits = raw_repository.commits(ref, path, limit, offset)
-    commits = decorate_commits(commits) if commits.present?
+    commits = Commit.decorate(commits) if commits.present?
     commits
   end
   def commits_between(target, source)
     commits = raw_repository.commits_between(target, source)
-    commits = decorate_commits(commits) if commits.present?
+    commits = Commit.decorate(commits) if commits.present?
     commits
   end
@@ -32,10 +32,4 @@ class Repository
     super
   end
-
-  protected
-
-  def decorate_commits(commits)
-    commits.map { |c| Commit.new(c) }
-  end
 end
@@ -79,14 +79,14 @@ class WikiPage
   def version
     return nil unless persisted?
-    @version ||= Commit.new(@page.version)
+    @version ||= Commit.new(Gitlab::Git::Commit.new(@page.version))
   end
   # Returns an array of Gitlab Commit instances.
   def versions
     return [] unless persisted?
-    @page.versions.map { |v| Commit.new(v) }
+    @page.versions.map { |v| Commit.new(Gitlab::Git::Commit.new(v)) }
   end
   # Returns the Date that this latest version was
@@ -6,7 +6,7 @@
       %i.icon-angle-right
       = link_to project_tree_path(@project, @ref) do
         = @project.name
-    - @tree.breadcrumbs(6) do |link|
+    - tree_breadcrumbs(@tree, 6) do |link|
       \/
       %li= link
   .clear
@@ -16,7 +16,7 @@
   %div.ui-box
     %h5.title
       Commits (#{@commits.count})
-    %ul.well-list= render @commits
+    %ul.well-list= render Commit.decorate(@commits)
   - unless @diffs.empty?
     %h4 Diff
@@ -13,5 +13,4 @@
     = preserve do
       = render_wiki_content(@wiki)
-- commit = Commit.new(@wiki.version)
-%p.time Last edited by #{commit_author_link(commit, avatar: true, size: 16)} #{time_ago_in_words @wiki.created_at} ago
+%p.time Last edited by #{commit_author_link(@wiki.version, avatar: true, size: 16)} #{time_ago_in_words @wiki.created_at} ago
-# Gitlab::Git::Gitlab::Git::Commit is a wrapper around native Grit::Commit object
+# Gitlab::Git::Commit is a wrapper around native Grit::Commit object
 # We dont want to use grit objects inside app/
 # It helps us easily migrate to rugged in future
 module Gitlab
   module Git
-    class Gitlab::Git::Commit
+    class Commit
       attr_accessor :raw_commit, :head, :refs
       delegate  :message, :authored_date, :committed_date, :parents, :sha,
@@ -18,12 +18,12 @@ module Gitlab
                      repo.commits(root_ref).first
                    end
-          Gitlab::Git::Commit.new(commit) if commit
+          Commit.new(commit) if commit
         end
         def fresh_commits(repo, n = 10)
           commits = repo.heads.map do |h|
-            repo.commits(h.name, n).map { |c| Gitlab::Git::Commit.new(c, h) }
+            repo.commits(h.name, n).map { |c| Commit.new(c, h) }
           end.flatten.uniq { |c| c.id }
           commits.sort! do |x, y|
@@ -34,7 +34,7 @@ module Gitlab
         end
         def commits_with_refs(repo, n = 20)
-          commits = repo.branches.map { |ref| Gitlab::Git::Commit.new(ref.commit, ref) }
+          commits = repo.branches.map { |ref| Commit.new(ref.commit, ref) }
           commits.sort! do |x, y|
             y.committed_date <=> x.committed_date
@@ -45,7 +45,7 @@ module Gitlab
         def commits_since(repo, date)
           commits = repo.heads.map do |h|
-            repo.log(h.name, nil, since: date).each { |c| Gitlab::Git::Commit.new(c, h) }
+            repo.log(h.name, nil, since: date).each { |c| Commit.new(c, h) }
           end.flatten.uniq { |c| c.id }
           commits.sort! do |x, y|
@@ -62,41 +62,11 @@ module Gitlab
             repo.commits(ref, limit, offset)
           else
             repo.commits(ref)
-          end.map{ |c| Gitlab::Git::Commit.new(c) }
+          end.map{ |c| Commit.new(c) }
         end
         def commits_between(repo, from, to)
-          repo.commits_between(from, to).map { |c| Gitlab::Git::Commit.new(c) }
-        end
-
-        def compare(project, from, to)
-          result = {
-            commits: [],
-            diffs: [],
-            commit: nil,
-            same: false
-          }
-
-          return result unless from && to
-
-          first = project.repository.commit(to.try(:strip))
-          last = project.repository.commit(from.try(:strip))
-
-          if first && last
-            result[:same] = (first.id == last.id)
-            result[:commits] = project.repo.commits_between(last.id, first.id).map {|c| Gitlab::Git::Commit.new(c)}
-
-            # Dont load diff for 100+ commits
-            result[:diffs] = if result[:commits].size > 100
-                               []
-                             else
-                               project.repo.diff(last.id, first.id) rescue []
-                             end
-
-            result[:commit] = Gitlab::Git::Commit.new(first)
-          end
-
-          result
+          repo.commits_between(from, to).map { |c| Commit.new(c) }
         end
       end
@@ -142,7 +112,7 @@ module Gitlab
       def prev_commit
         @prev_commit ||= if parents.present?
-                           Gitlab::Git::Commit.new(parents.first)
+                           Commit.new(parents.first)
                          else
                            nil
                          end
@@ -156,7 +126,7 @@ module Gitlab
       #
       # Cuts out the header and stats from #to_patch and returns only the diff.
       def to_diff
-        # see Grit::Gitlab::Git::Commit#show
+        # see Grit::Commit#show
         patch = to_patch
         # discard lines before the diff
@@ -17,15 +17,18 @@ module TestEnv
     repos_path = Rails.root.join('tmp', 'test-git-base-path')
     Gitlab.config.gitlab_shell.stub(repos_path: repos_path)
+    Gitlab::Shell.any_instance.stub(:add_repository) do |path|
+      create_temp_repo(File.join(repos_path, "#{path}.git"))
+    end
+
     Gitlab::Shell.any_instance.stub(
-      add_repository: ->(path) { create_temp_repo(File.join(repos_path, "#{path}.git")) },
       mv_repository: true,
       remove_repository: true,
       add_key: true,
       remove_key: true
     )
-    fake_satellite = double(
+    fake_satellite = stub(
       exists?: true,
       destroy: true,
       create: true
	@@ -8,7 +8,6 @@ class BlameController < ProjectResourceController		@@ -8,7 +8,6 @@ class BlameController < ProjectResourceController
8	before_filter :require_non_empty_project	8	before_filter :require_non_empty_project
9		9
10	def show	10	def show
11	- @repo = @project.repo
12	- @blame = Grit::Blob.blame(@repo, @commit.id, @path)	11	+ @blame = Gitlab::Git::Blame.new(project.repository, @commit.id, @path)
13	end	12	end
14	end	13	end
	@@ -114,5 +114,4 @@ class GollumWiki		@@ -114,5 +114,4 @@ class GollumWiki
114	def path_to_repo	114	def path_to_repo
115	@path_to_repo \|\|= File.join(Gitlab.config.gitlab_shell.repos_path, "#{path_with_namespace}.git")	115	@path_to_repo \|\|= File.join(Gitlab.config.gitlab_shell.repos_path, "#{path_with_namespace}.git")
116	end	116	end
117	-
118	end	117	end
	@@ -79,14 +79,14 @@ class WikiPage		@@ -79,14 +79,14 @@ class WikiPage
79	def version	79	def version
80	return nil unless persisted?	80	return nil unless persisted?
81		81
82	- @version \|\|= Commit.new(@page.version)	82	+ @version \|\|= Commit.new(Gitlab::Git::Commit.new(@page.version))
83	end	83	end
84		84
85	# Returns an array of Gitlab Commit instances.	85	# Returns an array of Gitlab Commit instances.
86	def versions	86	def versions
87	return [] unless persisted?	87	return [] unless persisted?
88		88
89	- @page.versions.map { \|v\| Commit.new(v) }	89	+ @page.versions.map { \|v\| Commit.new(Gitlab::Git::Commit.new(v)) }
90	end	90	end
91		91
92	# Returns the Date that this latest version was	92	# Returns the Date that this latest version was
	@@ -6,7 +6,7 @@		@@ -6,7 +6,7 @@
6	%i.icon-angle-right	6	%i.icon-angle-right
7	= link_to project_tree_path(@project, @ref) do	7	= link_to project_tree_path(@project, @ref) do
8	= @project.name	8	= @project.name
9	- - @tree.breadcrumbs(6) do \|link\|	9	+ - tree_breadcrumbs(@tree, 6) do \|link\|
10	\/	10	\/
11	%li= link	11	%li= link
12	.clear	12	.clear
	@@ -16,7 +16,7 @@		@@ -16,7 +16,7 @@
16	%div.ui-box	16	%div.ui-box
17	%h5.title	17	%h5.title
18	Commits (#{@commits.count})	18	Commits (#{@commits.count})
19	- %ul.well-list= render @commits	19	+ %ul.well-list= render Commit.decorate(@commits)
20		20
21	- unless @diffs.empty?	21	- unless @diffs.empty?
22	%h4 Diff	22	%h4 Diff
	@@ -13,5 +13,4 @@		@@ -13,5 +13,4 @@
13	= preserve do	13	= preserve do
14	= render_wiki_content(@wiki)	14	= render_wiki_content(@wiki)
15		15
16	-- commit = Commit.new(@wiki.version)
17	-%p.time Last edited by #{commit_author_link(commit, avatar: true, size: 16)} #{time_ago_in_words @wiki.created_at} ago	16	+%p.time Last edited by #{commit_author_link(@wiki.version, avatar: true, size: 16)} #{time_ago_in_words @wiki.created_at} ago