danger-gitlab_reviewbot 1.1.6 → 1.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 56b35c5cb823cb3e1355a0ab855c4f8745a7fed8d580d3df01a4ee8a05b204ef
-  data.tar.gz: ba5391618aff949ecd6f50e0b33bcfe2db0ba63efa4b3d1394ba2456935c5115
+  metadata.gz: 4a7eaa5cd15e0c7c399b577b76461cf7eb3580b1d1350bcc0175c5126784fc1c
+  data.tar.gz: 17c5294182aad19fb6d68dcf6858ab313a1ed140a53acffa6ef815abdb4b5a48
 SHA512:
-  metadata.gz: 5aa5f29bab94af692e58c6c4f4b67f63006d8be957a39a9b5666c21dbd4e7ce0d0799b213bdd6fca05e477342a21c4346e6821055f43948a263c80f942d99309
-  data.tar.gz: 00b534deaef5d197c01b3743849b0131bed30c555f21c61e9143c74cd8040686bc61e05774992c8416742ab6dce5fc69e372b4e211f320472028a83dc266cf44
+  metadata.gz: b66136604fadac6c2d49a53739b5435a40a3634a269683d68af032a0bd5475a8070f3a6f8a49801bc537a167c4f3b5955fd97fa464ea9349c268056612526708
+  data.tar.gz: 776cc841222ceb2f057aa7c9bc4fda90b3dff2fddfa602c4863e036f71ab6af098a1a22b0606dda00a829d98edcd7da77d02bf68b4c3ab9adc299489d0471f4a

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-gitlab_reviewbot (1.1.5)
+    danger-gitlab_reviewbot (1.1.7)
       danger-gitlab
       danger-plugin-api (~> 1.0)
 

data/lib/gitlab_reviewbot/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GitlabReviewbot
-  VERSION = "1.1.6"
+  VERSION = "1.1.8"
 end

data/lib/gitlab_reviewbot/gitlab.rb

@@ -19,12 +19,33 @@ module Gitlab
     end
   end
 
-  class Client < API
+  class ClientHelper
+    def initialize(client)
+      @client = client
+    end
+
+    def users_with_pending_mr_review(project_id)
+      outstanding_mrs = fetch_mrs_requiring_review(project_id)
+      all_assignees = outstanding_mrs.reduce([]) { |acc, mr| acc + mr.assignees - assignees_with_review(project_id, mr.id, mr.assignees)}
+      assignees_id_map = all_assignees.each_with_object({}) do |a, acc|
+        aid = a["id"]
+        ausername = a["username"]
+        assignee = acc[aid] || User.new(aid, ausername)
+        assignee.review_count += 1
+        acc[aid] = assignee
+      end
+      assignees_id_map.values
+    end
+
+    def fetch_mrs_requiring_review(project_id)
+      @client.merge_requests(project_id, state: "opened", per_page: "100")
+    end
+
     def fetch_users_for_group(group_name)
-      group_id = search_group(group_name)
+      group_id = search_group_with_path(group_name)
       return nil if group_id.nil?
 
-      res = group_members(group_id)
+      res = @client.group_members(group_id, :per_page => 100)
 
       developer_access_level = 30
       res.select { |u| u.state == "active" && u.access_level >= developer_access_level }.map { |u| User.new(u.id, u.username) }
@@ -32,44 +53,28 @@ module Gitlab
 
     def assign_mr_to_users(project_id, mr_iid, users)
       user_ids = users.map(&:id)
-      update_merge_request(project_id, mr_iid, "assignee_ids" => user_ids)
+      @client.update_merge_request(project_id, mr_iid, "assignee_ids" => user_ids)
+
     end
 
     def fetch_author_for_mr(project_id, mr_iid)
-      res = merge_request(project_id, mr_iid)
+      res = @client.merge_request(project_id, mr_iid)
       User.new(res.author.id, res.author.name)
     end
 
-    def fetch_mrs_requiring_review(project_id)
-      merge_requests(project_id, state: "opened", per_page: "100")
-    end
-
     def find_user_with_username(username)
-      users({ username: username }).map { |u| User.new(u.id, u.username) }
-    end
-
-    def users_with_pending_mr_review(project_id)
-      outstanding_mrs = fetch_mrs_requiring_review(project_id)
-      all_assignees = outstanding_mrs.reduce([]) { |acc, mr| acc + mr.assignees }
-      assignees_id_map = all_assignees.each_with_object({}) do |a, acc|
-        aid = a["id"]
-        ausername = a["username"]
-        assignee = acc[aid] || User.new(aid, ausername)
-        assignee.review_count += 1
-        acc[aid] = assignee
-      end
-      assignees_id_map.values
+      @client.users({ username: username }).map { |u| User.new(u.id, u.username) }
     end
 
     def fetch_mr_reviewers(project_id, mr_iid)
-      merge_request(project_id, mr_iid).assignees.map { |u| User.new(u["id"], u["username"]) }
+      @client.merge_request(project_id, mr_iid).assignees.map { |u| User.new(u["id"], u["username"]) }
     end
 
     private
 
-    def search_group(group_name)
+    def search_group_with_path(group_name)
       short_name = group_name.split("/").last
-      res = group_search(short_name)
+      res = @client.group_search(short_name)
       res = res.find { |i| i.full_path == group_name }
 
       if res.nil?
@@ -78,5 +83,28 @@ module Gitlab
         res.id
       end
     end
+
+    def assignees_with_review(project_id, mr_iid, gusers)
+      approvals =  @client.merge_request_approval_state(project_id, mr_iid)
+      approved_by = approvals.rules.reduce([]) { |acc, r| acc + r['approved_by'] }
+      gusers.filter { |u| approved_by.index { |au| au['id'] == u['id'] } != nil }
+    end
+
+  end
+
+  class Client < API
+
+    def client_helper
+      @client_helper ||= ClientHelper.new(self)
+    end
+
+    def method_missing(method, *args)
+      if @client_helper.respond_to?(method)
+        @client_helper.send(method, *args)
+      else
+        super
+      end
+    end
+
   end
 end

data/spec/gitlab_ext_spec.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require File.expand_path("spec_helper", __dir__)
+
+
+module Danger
+  describe Gitlab::Client do
+    before do
+      #
+      # @sam = Gitlab::User.new(1, "Sam")
+      # @tom = Gitlab::User.new(2, "Tom")
+      # @nic = Gitlab::User.new(3, "Nic")
+      # @luke = Gitlab::User.new(4, "Luke")
+      @mock_client = double(Gitlab::Client)
+      @client_helper = Gitlab::ClientHelper.new(@mock_client)
+      @sam = Gitlab::User.new(1, "Sam")
+      @tom = Gitlab::User.new(2, "Tom")
+      @nic = Gitlab::User.new(3, "Nic")
+      @assignees = [{'id' => 1, 'username' => "Sam"},
+                    {'id' => 2, 'username' => "Tom"},
+                    {'id' => 3, 'username' => "Nic"}]
+    end
+
+    it "Correctly calculates users with pending reviews" do
+      already_reviewed = [@assignees[0]]
+
+      allow(@mock_client).to receive(:merge_requests).with(10, {:per_page=>"100", :state=>"opened"}).and_return([Gitlab::ObjectifiedHash.new({'id' => 110, 'assignees' => @assignees})])
+      allow(@mock_client).to receive(:merge_request_approval_state).with(10, 110).and_return(Gitlab::ObjectifiedHash.new({'rules' => [{'approved_by' => already_reviewed}]}))
+
+      expect(@client_helper.users_with_pending_mr_review(10)).to be == [@tom, @nic]
+    end
+
+    it "Correctly calculates users with pending reviews (multi rule)" do
+      allow(@mock_client).to receive(:merge_requests).with(10, {:per_page=>"100", :state=>"opened"}).and_return([Gitlab::ObjectifiedHash.new({'id' => 110, 'assignees' => @assignees})])
+      allow(@mock_client).to receive(:merge_request_approval_state).with(10, 110).and_return(Gitlab::ObjectifiedHash.new({'rules' => [{'approved_by' => [{'id' => 1, 'username' => "Sam"}]}, {'approved_by' => [{'id' => 2, 'username' => "Tom"}]}]}))
+
+      expect(@client_helper.users_with_pending_mr_review(10)).to be == [@nic]
+    end
+
+    it "Fetches only active developer users for group" do
+      active_users = [Gitlab::ObjectifiedHash.new({'id' => 1, 'username' => 'Sam', 'state' => 'active', 'access_level' => 30}),
+                      Gitlab::ObjectifiedHash.new({'id' => 2, 'username' => 'Tom', 'state' => 'active', 'access_level' => 50})]
+      inactive_users = [Gitlab::ObjectifiedHash.new({'id' => 3, 'username' => 'OldSam', 'state' => 'inactive', 'access_level' => 30})]
+      non_developer_users = [Gitlab::ObjectifiedHash.new({'id' => 4, 'username' => 'Reporter', 'state' => 'active', 'access_level' => 20})]
+      allow(@mock_client).to receive(:group_search).with("mygroup").and_return([Gitlab::ObjectifiedHash.new({'id' => 10, 'full_path' => "mygroup"})])
+      allow(@mock_client).to receive(:group_members).with(10).and_return(active_users + inactive_users + non_developer_users)
+      expect(@client_helper.fetch_users_for_group("mygroup")).to be == [@sam, @tom]
+    end
+  end
+end
+

data/spec/spec_helper.rb

@@ -10,6 +10,7 @@ require "pry"
 
 require "rspec"
 require "danger"
+require File.expand_path("../lib/gitlab_reviewbot/gitlab", __dir__)
 
 if `git remote -v` == ""
   puts "You cannot run tests without setting a local git remote on this repo"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-gitlab_reviewbot
 version: !ruby/object:Gem::Version
-  version: 1.1.6
+  version: 1.1.8
 platform: ruby
 authors:
 - Fabio Gallonetto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-16 00:00:00.000000000 Z
+date: 2021-08-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -218,6 +218,7 @@ files:
 - lib/gitlab_reviewbot/strategies/least_busy.rb
 - lib/gitlab_reviewbot/strategies/random.rb
 - lib/gitlab_reviewbot/strategies/strategy.rb
+- spec/gitlab_ext_spec.rb
 - spec/gitlab_reviewbot_spec.rb
 - spec/least_busy_strategy_spec.rb
 - spec/random_strategy_spec.rb
@@ -246,6 +247,7 @@ signing_key:
 specification_version: 4
 summary: A review raffle bot for Gitlab.
 test_files:
+- spec/gitlab_ext_spec.rb
 - spec/gitlab_reviewbot_spec.rb
 - spec/least_busy_strategy_spec.rb
 - spec/random_strategy_spec.rb