danger-gitlab_reviewbot 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c9af6b55a55d3c0efaa7b242896f13a64bbfdac0a0d28883db6198aebb27f3f0
-  data.tar.gz: 7cdb84acf577618e1125fdc56a1bb955b0e7b9d2bbc03246d026f7bb24c559af
+  metadata.gz: 0b07048b00ab79e2a4fe6497f1559c00925da2b216240b5180eb5f9e9ac92189
+  data.tar.gz: c0a83a3f273f8b98796a41039f4502307ae9fdd698630745233f7e08a459124a
 SHA512:
-  metadata.gz: 1e7b6a6b032be64c6b4dad9c12ff7ccadbd979f7cbc9f7c7d2b5680296d93a8a7964aa93b37d1e558014f3c590175fbf084f878231033984666651432c6ef07d
-  data.tar.gz: a398091bfeac470d204d41175b1c52b717fa8d871e0895aeebc7211145ca63f38ee837432bd4343358ab428aa2cb1866e7cd4a75f3926719027fc553b6e1c2dc
+  metadata.gz: 1576186ee3d22820f48734093708df1f10ef1b2fcd28899ee82f7f648cc08142aedff7acad11c893847d69098ac014d7367c91aaee6b84e801a96647cc89acea
+  data.tar.gz: 9222d55514970d4dc4d5595d8523f1a8c076ebddb273d2e969f37b254d040bee2ca0350669a9a52272be20d8210d43633794f00f51b7a5c5eaac2af718d36c7c

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-gitlab_reviewbot (1.1.7)
+    danger-gitlab_reviewbot (1.2.0)
       danger-gitlab
       danger-plugin-api (~> 1.0)
 
@@ -100,7 +100,7 @@ GEM
       method_source (~> 0.9.0)
     pry-nav (0.3.0)
       pry (>= 0.9.10, < 0.13.0)
-    public_suffix (4.0.5)
+    public_suffix (4.0.6)
     rainbow (3.0.0)
     rake (13.0.1)
     rb-fsevent (0.10.4)

data/lib/gitlab_reviewbot/gem_version.rb

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

data/spec/gitlab_ext_spec.rb

@@ -16,22 +16,22 @@ module Danger
       @sam = Gitlab::User.new(1, "Sam")
       @tom = Gitlab::User.new(2, "Tom")
       @nic = Gitlab::User.new(3, "Nic")
-      @assignees = [{'id' => 1, 'username' => "Sam"},
+      @reviewers = [{'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]]
+      already_reviewed = [@reviewers[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_requests).with(10, {:per_page=>"100", :state=>"opened"}).and_return([Gitlab::ObjectifiedHash.new({'iid' => 110, 'reviewers' => @reviewers})])
       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_requests).with(10, {:per_page=>"100", :state=>"opened"}).and_return([Gitlab::ObjectifiedHash.new({'iid' => 110, 'reviewers' => @reviewers})])
       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]
@@ -43,7 +43,7 @@ module Danger
       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)
+      allow(@mock_client).to receive(:group_members).with(10, {:per_page=>100}).and_return(active_users + inactive_users + non_developer_users)
       expect(@client_helper.fetch_users_for_group("mygroup")).to be == [@sam, @tom]
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger-gitlab_reviewbot
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Fabio Gallonetto