Collect contributors info via API

lib/api/repositories.rb

@@ -160,7 +160,29 @@ module API
       get ':id/repository/contributors' do
         authorize! :download_code, user_project
+        contributors = {}
+        contributors
+
         log = user_project.repository.graph_log
+        log.each do |entry|
+          email = entry[:author_email].to_s
+
+          if contributors.has_key?(email)
+            contributors[email][:commits] += 1
+            contributors[email][:additions] += entry[:additions] || 0
+            contributors[email][:deletions] += entry[:deletions] || 0
+          else
+            contributors[email] = {
+              email: email,
+              name: entry[:author_name],
+              commits: 1,
+              additions: entry[:additions] || 0,
+              deletions: entry[:deletions] || 0,
+            }
+          end
+        end
+
+        contributors
       end
     end
   end

spec/requests/api/repositories_spec.rb

@@ -128,7 +128,7 @@ describe API::API, api: true  do
     end
   end
-  describe 'GET /GET /projects/:id/repository/compare' do
+  describe 'GET /projects/:id/repository/compare' do
     it "should compare branches" do
       get api("/projects/#{project.id}/repository/compare", user), from: 'master', to: 'simple_merge_request'
       response.status.should == 200
@@ -166,4 +166,17 @@ describe API::API, api: true  do
       json_response['compare_same_ref'].should be_true
     end
   end
+
+  describe 'GET /projects/:id/repository/contributors' do
+    it 'should return valid data' do
+      get api("/projects/#{project.id}/repository/contributors", user)
+      response.status.should == 200
+      contributor = json_response.values.first
+      contributor['email'].should == 'dmitriy.zaporozhets@gmail.com'
+      contributor['name'].should == 'Dmitriy Zaporozhets'
+      contributor['commits'].should == 185
+      contributor['additions'].should == 66072
+      contributor['deletions'].should == 63013
+    end
+  end
 end