danger-gitlab_reviewbot 1.0.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: c8f3b6c9f3e73ce3bdd2a7715f556dd2d3a3f220e10525a6e1843405a9b465d5
+  data.tar.gz: 9f3dad2be11de00ebde8a301a9b8fdfefcf6b6642b8c32edcb2fdb712177ac4f
+SHA512:
+  metadata.gz: 72083bfe33060a8be99029d18fd12f8954ebb689e0c39d47f482c492c16524ca3c6e91ba6dd77339a08ab86d7dbe1a987fa1e832c4317890fb10e2656f6a1942
+  data.tar.gz: 36b6f7c72796643d677e63291ad328635901fbb154dcc726ed763d8dfb25bec90ac8d34930b7c464d6c4f5a5c98efd30d65e563cc0aabcefce2d6e8906591bae

data/.gitignore

@@ -0,0 +1,4 @@
+.DS_Store
+pkg
+.idea/
+.yardoc

data/.rubocop.yml

@@ -0,0 +1,148 @@
+# Defaults can be found here: https://github.com/bbatsov/rubocop/blob/master/config/default.yml
+
+# If you don't like these settings, just delete this file :)
+
+AllCops:
+  TargetRubyVersion: 2.4
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+  Enabled: true
+
+# kind_of? is a good way to check a type
+Style/ClassCheck:
+  EnforcedStyle: kind_of?
+
+# specs sometimes have useless assignments, which is fine
+Lint/UselessAssignment:
+  Exclude:
+    - '**/spec/**/*'
+
+# We could potentially enable the 2 below:
+Layout/FirstHashElementIndentation:
+  Enabled: false
+
+Layout/HashAlignment:
+  Enabled: false
+
+# HoundCI doesn't like this rule
+Layout/DotPosition:
+  Enabled: false
+
+# We allow !! as it's an easy way to convert ot boolean
+Style/DoubleNegation:
+  Enabled: false
+
+# Cop supports --auto-correct.
+Lint/UnusedBlockArgument:
+  Enabled: false
+
+# We want to allow class Fastlane::Class
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+Metrics/AbcSize:
+  Max: 60
+
+# The %w might be confusing for new users
+Style/WordArray:
+  MinSize: 19
+
+# raise and fail are both okay
+Style/SignalException:
+  Enabled: false
+
+# Better too much 'return' than one missing
+Style/RedundantReturn:
+  Enabled: false
+
+# Having if in the same line might not always be good
+Style/IfUnlessModifier:
+  Enabled: false
+
+# and and or is okay
+Style/AndOr:
+  Enabled: false
+
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 350
+
+Metrics/CyclomaticComplexity:
+  Max: 17
+
+# Configuration parameters: AllowURI, URISchemes.
+Metrics/LineLength:
+  Max: 370
+
+# Configuration parameters: CountKeywordArgs.
+Metrics/ParameterLists:
+  Max: 10
+
+Metrics/PerceivedComplexity:
+  Max: 18
+
+# Sometimes it's easier to read without guards
+Style/GuardClause:
+  Enabled: false
+
+# something = if something_else
+# that's confusing
+Style/ConditionalAssignment:
+  Enabled: false
+
+# Better to have too much self than missing a self
+Style/RedundantSelf:
+  Enabled: false
+
+Metrics/MethodLength:
+  Max: 60
+
+# We're not there yet
+Style/Documentation:
+  Enabled: false
+
+# Adds complexity
+Style/IfInsideElse:
+  Enabled: false
+
+# danger specific
+
+Style/BlockComments:
+  Enabled: false
+
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+
+# FIXME: 25
+Metrics/BlockLength:
+  Max: 345
+  Exclude:
+    - "**/*_spec.rb"
+
+Style/MixinGrouping:
+  Enabled: false
+
+Style/FileName:
+  Enabled: false
+
+Layout/HeredocIndentation:
+  Enabled: false
+
+Style/SpecialGlobalVars:
+  Enabled: false
+
+PercentLiteralDelimiters:
+  PreferredDelimiters:
+    "%":  ()
+    "%i": ()
+    "%q": ()
+    "%Q": ()
+    "%r": "{}"
+    "%s": ()
+    "%w": ()
+    "%W": ()
+    "%x": ()
+
+Security/YAMLLoad:
+  Enabled: false

data/Gemfile

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+
+# Specify your gem's dependencies in danger-gitlab_reviewbot.gemspec
+gemspec

data/Gemfile.lock

@@ -0,0 +1,157 @@
+PATH
+  remote: .
+  specs:
+    danger-gitlab_reviewbot (1.0.0)
+      danger-gitlab
+      danger-plugin-api (~> 1.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    addressable (2.7.0)
+      public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.0)
+    claide (1.0.3)
+    claide-plugins (0.9.2)
+      cork
+      nap
+      open4 (~> 1.3)
+    coderay (1.1.2)
+    colored2 (3.1.2)
+    cork (0.3.0)
+      colored2 (~> 3.1)
+    danger (6.3.2)
+      claide (~> 1.0)
+      claide-plugins (>= 0.9.2)
+      colored2 (~> 3.1)
+      cork (~> 0.1)
+      faraday (~> 0.9)
+      faraday-http-cache (~> 2.0)
+      git (~> 1.6)
+      kramdown (~> 2.0)
+      kramdown-parser-gfm (~> 1.0)
+      no_proxy_fix
+      octokit (~> 4.7)
+      terminal-table (~> 1)
+    danger-gitlab (7.0.0)
+      danger (~> 6.0)
+      gitlab (~> 4.2, >= 4.2.0)
+    danger-plugin-api (1.0.0)
+      danger (> 2.0)
+    diff-lcs (1.3)
+    faraday (0.17.3)
+      multipart-post (>= 1.2, < 3)
+    faraday-http-cache (2.2.0)
+      faraday (>= 0.8)
+    ffi (1.12.2)
+    formatador (0.2.5)
+    git (1.7.0)
+      rchardet (~> 1.8)
+    gitlab (4.14.1)
+      httparty (~> 0.14, >= 0.14.0)
+      terminal-table (~> 1.5, >= 1.5.1)
+    guard (2.16.2)
+      formatador (>= 0.2.4)
+      listen (>= 2.7, < 4.0)
+      lumberjack (>= 1.0.12, < 2.0)
+      nenv (~> 0.1)
+      notiffany (~> 0.0)
+      pry (>= 0.9.12)
+      shellany (~> 0.0)
+      thor (>= 0.18.1)
+    guard-compat (1.2.1)
+    guard-rspec (4.7.3)
+      guard (~> 2.1)
+      guard-compat (~> 1.1)
+      rspec (>= 2.99.0, < 4.0)
+    httparty (0.18.0)
+      mime-types (~> 3.0)
+      multi_xml (>= 0.5.2)
+    jaro_winkler (1.5.4)
+    kramdown (2.2.1)
+      rexml
+    kramdown-parser-gfm (1.1.0)
+      kramdown (~> 2.0)
+    listen (3.0.7)
+      rb-fsevent (>= 0.9.3)
+      rb-inotify (>= 0.9.7)
+    lumberjack (1.2.4)
+    method_source (1.0.0)
+    mime-types (3.3.1)
+      mime-types-data (~> 3.2015)
+    mime-types-data (3.2020.0425)
+    multi_xml (0.6.0)
+    multipart-post (2.1.1)
+    nap (1.1.0)
+    nenv (0.3.0)
+    no_proxy_fix (0.1.2)
+    notiffany (0.1.3)
+      nenv (~> 0.1)
+      shellany (~> 0.0)
+    octokit (4.18.0)
+      faraday (>= 0.9)
+      sawyer (~> 0.8.0, >= 0.5.3)
+    open4 (1.3.4)
+    parallel (1.19.1)
+    parser (2.7.1.2)
+      ast (~> 2.4.0)
+    pry (0.13.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    public_suffix (4.0.4)
+    rainbow (3.0.0)
+    rake (10.5.0)
+    rb-fsevent (0.10.4)
+    rb-inotify (0.10.1)
+      ffi (~> 1.0)
+    rchardet (1.8.0)
+    rexml (3.2.4)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.2)
+      rspec-support (~> 3.9.3)
+    rspec-expectations (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.3)
+    rubocop (0.82.0)
+      jaro_winkler (~> 1.5.1)
+      parallel (~> 1.10)
+      parser (>= 2.7.0.1)
+      rainbow (>= 2.2.2, < 4.0)
+      rexml
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 2.0)
+    ruby-progressbar (1.10.1)
+    sawyer (0.8.2)
+      addressable (>= 2.3.5)
+      faraday (> 0.8, < 2.0)
+    shellany (0.0.1)
+    terminal-table (1.8.0)
+      unicode-display_width (~> 1.1, >= 1.1.1)
+    thor (1.0.1)
+    unicode-display_width (1.7.0)
+    yard (0.9.25)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler (~> 2.1)
+  danger-gitlab_reviewbot!
+  guard (~> 2.14)
+  guard-rspec (~> 4.7)
+  listen (= 3.0.7)
+  pry
+  rake (~> 10.0)
+  rspec (~> 3.4)
+  rubocop
+  yard
+
+BUNDLED WITH
+   2.1.4

data/Guardfile

@@ -0,0 +1,19 @@
+# A guardfile for making Danger Plugins
+# For more info see https://github.com/guard/guard#readme
+
+# To run, use `bundle exec guard`.
+
+guard :rspec, cmd: 'bundle exec rspec' do
+  require 'guard/rspec/dsl'
+  dsl = Guard::RSpec::Dsl.new(self)
+
+  # RSpec files
+  rspec = dsl.rspec
+  watch(rspec.spec_helper) { rspec.spec_dir }
+  watch(rspec.spec_support) { rspec.spec_dir }
+  watch(rspec.spec_files)
+
+  # Ruby files
+  ruby = dsl.ruby
+  dsl.watch_spec_files_for(ruby.lib_files)
+end

data/LICENSE.txt

@@ -0,0 +1,22 @@
+Copyright (c) 2020 Fabio Gallonetto <fabio.gallonetto@imaginecurve.com>
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -0,0 +1,20 @@
+# danger-gitlab_reviewbot
+
+A description of danger-gitlab_reviewbot.
+
+## Installation
+
+    $ gem install danger-gitlab_reviewbot
+
+## Usage
+
+    Methods and attributes from this plugin are available in
+    your `Dangerfile` under the `gitlab_reviewbot` namespace.
+
+## Development
+
+1. Clone this repo
+2. Run `bundle install` to setup dependencies.
+3. Run `bundle exec rake spec` to run the tests.
+4. Use `bundle exec guard` to automatically have tests run as you make changes.
+5. Make your changes.

data/Rakefile

@@ -0,0 +1,23 @@
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
+
+RSpec::Core::RakeTask.new(:specs)
+
+task default: :specs
+
+task :spec do
+  Rake::Task['specs'].invoke
+#  Rake::Task['rubocop'].invoke
+  Rake::Task['spec_docs'].invoke
+end
+
+desc 'Run RuboCop on the lib/specs directory'
+RuboCop::RakeTask.new(:rubocop) do |task|
+  task.patterns = ['lib/**/*.rb', 'spec/**/*.rb']
+end
+
+desc 'Ensure that the plugin passes `danger plugins lint`'
+task :spec_docs do
+  sh 'bundle exec danger plugins lint'
+end

data/danger-gitlab_reviewbot.gemspec

@@ -0,0 +1,50 @@
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'gitlab_reviewbot/gem_version.rb'
+
+Gem::Specification.new do |spec|
+  spec.name          = 'danger-gitlab_reviewbot'
+  spec.version       = GitlabReviewbot::VERSION
+  spec.authors       = ['Fabio Gallonetto']
+  spec.email         = ['fabio.gallonetto@curve.com']
+  spec.description   = %q{A review raffle bot for Gitlab }
+  spec.summary       = %q{A review raffle bot for Gitlab.}
+  spec.homepage      = 'https://github.com/curve-technology/danger-gitlab_reviewbot'
+  spec.license       = 'MIT'
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ['lib']
+
+  spec.add_runtime_dependency 'danger-plugin-api', '~> 1.0'
+  spec.add_runtime_dependency 'danger-gitlab'
+
+  # General ruby development
+  spec.add_development_dependency 'bundler', '~> 2.1'
+  spec.add_development_dependency 'rake', '~> 10.0'
+
+  # Testing support
+  spec.add_development_dependency 'rspec', '~> 3.4'
+
+  # Linting code and docs
+  spec.add_development_dependency "rubocop"
+  spec.add_development_dependency "yard"
+
+  # Makes testing easy via `bundle exec guard`
+  spec.add_development_dependency 'guard', '~> 2.14'
+  spec.add_development_dependency 'guard-rspec', '~> 4.7'
+
+  # If you want to work on older builds of ruby
+  spec.add_development_dependency 'listen', '3.0.7'
+
+  # This gives you the chance to run a REPL inside your tests
+  # via:
+  #
+  #    require 'pry'
+  #    binding.pry
+  #
+  # This will stop test execution and let you inspect the results
+  spec.add_development_dependency 'pry'
+end

data/lib/danger_gitlab_reviewbot.rb

@@ -0,0 +1 @@
+require "gitlab_reviewbot/gem_version"

data/lib/danger_plugin.rb

@@ -0,0 +1 @@
+require "gitlab_reviewbot/plugin"

data/lib/gitlab_reviewbot/gem_version.rb

@@ -0,0 +1,3 @@
+module GitlabReviewbot
+  VERSION = "1.0.0".freeze
+end

data/lib/gitlab_reviewbot/gitlab.rb

@@ -0,0 +1,61 @@
+require 'gitlab'
+
+module Gitlab
+  class User
+    attr_accessor :username
+    attr_accessor :id
+    attr_accessor :review_count
+
+    def initialize(id, username, review_count = 0)
+      @id = id
+      @username = username
+      @review_count = review_count
+    end
+  end
+
+  class Client < API
+    def fetch_users_for_group(group_name)
+      group_id = search_group(group_name)
+      return nil if group_id.nil?
+
+      res = group_members(group_id)
+
+      developer_access_level = 30
+      res.select { |u| u.state == 'active' && u.access_level >= developer_access_level }.map { |u| User.new(u.id, u.username) }
+    end
+
+    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)
+    end
+
+    def fetch_author_for_mr(project_id, mr_iid)
+      res = 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').select { |mr| mr.merge_status != 'can_be_merged' }
+    end
+
+    def users_with_pending_mr_review(project_id)
+      outstanding_mrs = fetch_mrs_requiring_review(project_id)
+      outstanding_mrs.reduce([]) { |acc, mr| acc + mr.assignees}
+                     .map { |a| User.new(a['id'], a['username']) }
+    end
+
+    private
+    def search_group(group_name)
+      short_name = group_name.split('/').last
+      res = group_search(short_name)
+      res = res.find { |i| i.full_path == group_name }
+
+      if res.nil?
+        nil
+      else
+        res.id
+      end
+    end
+  end
+end
+

data/lib/gitlab_reviewbot/plugin.rb

@@ -0,0 +1,91 @@
+require 'gitlab_reviewbot/strategies'
+
+
+module Danger
+  # This is your plugin class. Any attributes or methods you expose here will
+  # be available from within your Dangerfile.
+  #
+  # To be published on the Danger plugins site, you will need to have
+  # the public interface documented. Danger uses [YARD](http://yardoc.org/)
+  # for generating documentation from your plugin source, and you can verify
+  # by running `danger plugins lint` or `bundle exec rake spec`.
+  #
+  # You should replace these comments with a public description of your library.
+  #
+  # @example Ensure people are well warned about merging on Mondays
+  #
+  #          my_plugin.warn_on_mondays
+  #
+  # @see  Fabio Gallonetto/danger-gitlab_reviewbot
+  # @tags monday, weekends, time, rattata
+  #
+  class DangerGitlabReviewbot < Plugin
+
+    # Define the group to take the reviewers from. 
+    # NOTE: This is the group full path as in 'tech/iOS' instead of just the group name
+    #
+    # @return   String
+    attr_accessor :gitlab_group
+
+    # Define the amount of reviewers to add to the merge requests.
+    # Default is 1.
+    # NOTE: The plugin won't remove existing assigned reviewers
+    #
+    # @return   Int
+    attr_accessor :assignees_amount
+    def assignees_amount
+      @assignees_amount || 1
+    end
+
+    # Define the strategy for chosing reviewers.
+    # Valid values are:
+    # * Danger::AssignStrategies::RandomStrategy - assigns N reviewers at random from the group
+    #   (excluding the author).
+    # * Danger::AssignStrategies::LeastBusyStrategy - assign the N users with the least amount of open MRs
+    #   to review
+    #
+    attr_accessor :strategy
+    def strategy
+      @strategy || Danger::AssignStrategies::RandomStrategy
+    end
+
+    # Add verbosity to the logs
+    # @return Bool
+    attr_accessor :verbose
+
+    # Call this method from the Dangerfile to assign reviewers to your merge requests
+    # @return   The usernames list of assigned reviewes [Array<String>]
+    #
+    def assign!
+      project_id = ENV['CI_PROJECT_ID']
+      mr_iid = ENV['CI_MERGE_REQUEST_IID']
+      if mr_iid.nil?
+        raise "Env variable CI_MERGE_REQUEST_IID doesn't point to a valid merge request iid"
+      end
+
+      if project_id.nil?
+        raise "Env variable CI_PROJECT_ID doesn't point to a valid project id"
+      end
+
+      current_assignees = (ENV['CI_MERGE_REQUEST_ASSIGNEES'] || '').split(',')
+      required_assignees_count = [assignees_amount - current_assignees.length, 0].max
+
+      puts "Project ID: #{project_id}" if @verbose
+      puts "MR IID: #{mr_iid}" if @verbose
+      puts "Currently assigned: #{current_assignees}" if @verbose
+      puts "Required: #{required_assignees_count}" if @verbose
+      if required_assignees_count == 0
+        puts "Nothing to do" if @verbose
+        return
+      end
+
+      strategy_class = strategy.new(client: gitlab.api, project: project_id, mr: mr_iid, group: gitlab_group)
+
+      assignees = strategy_class.assign! required_assignees_count
+
+      puts "Assigning: #{assignees}" if @verbose
+      return assignees
+    end
+  end
+end
+

data/lib/gitlab_reviewbot/strategies/least_busy.rb

@@ -0,0 +1,24 @@
+require 'gitlab_reviewbot/gitlab'
+
+module Danger
+  module AssignStrategies
+    class LeastBusyStrategy < Strategy
+      def assignees(amount)
+        review_counter = client.fetch_users_for_group(group_name).reduce({}) do |counter, user|
+          counter[user.id] = user
+          counter
+        end
+
+        users = client.users_with_pending_mr_review(project_id) do |counter, user|
+          next if counter[user.id].nil?
+          counter[user.id].review_count += 1
+          counter
+        end
+        users.filter { |u| u.id != author.id }
+             .sort_by(&:review_count)
+             .last(amount)
+      end
+    end
+  end
+end
+

data/lib/gitlab_reviewbot/strategies/random.rb

@@ -0,0 +1,13 @@
+require 'gitlab_reviewbot/gitlab'
+
+module Danger
+  module AssignStrategies
+    class RandomStrategy < Strategy
+      def assignees(amount)
+        client.fetch_users_for_group(group_name)
+              .filter { |u| u.id != author.id }
+              .sample(amount)
+      end
+    end
+  end
+end

data/lib/gitlab_reviewbot/strategies/strategy.rb

@@ -0,0 +1,36 @@
+module Danger
+  module AssignStrategies
+    class Strategy
+      attr_accessor :mr_iid
+      attr_accessor :project_id
+      attr_accessor :group_name
+      attr_accessor :client
+
+      def initialize(client:, project:, mr:, group:)
+        @client = client
+        @project_id = project
+        @mr_iid = mr
+        @group_name = group
+      end
+
+      def assign!(amount)
+        to_be_assigned = assignees(amount)
+        response = client.assign_mr_to_users(project_id, mr_iid, to_be_assigned)
+        return to_be_assigned.map(&:username)
+      end
+
+      def assignees(amount)
+        raise "To be implemented in the subclasses"
+      end
+
+      def author
+        client.fetch_author_for_mr(@project_id, @mr_iid)
+      end
+
+      def users_in_group
+        client.fetch_users_for_group(group_name)
+      end
+    end
+  end
+end
+

data/lib/gitlab_reviewbot/strategies.rb

@@ -0,0 +1,8 @@
+module Danger
+  module AssignStrategies
+    require 'gitlab_reviewbot/strategies/strategy'
+    require 'gitlab_reviewbot/strategies/random'
+    require 'gitlab_reviewbot/strategies/least_busy'
+  end
+end
+

data/spec/gitlab_reviewbot_spec.rb

@@ -0,0 +1,75 @@
+require File.expand_path("../spec_helper", __FILE__)
+
+module Danger
+  describe Danger::DangerGitlabReviewbot do
+    it "should be a plugin" do
+      expect(Danger::DangerGitlabReviewbot.new(nil)).to be_a Danger::Plugin
+    end
+
+    #
+    # You should test your custom attributes and methods here
+    #
+    describe "with Dangerfile" do
+      before do
+        testing_env.each { |k,v| ENV[k] = "#{v}" }
+
+        @dangerfile = testing_dangerfile
+        @plugin = @dangerfile.gitlab_reviewbot
+        @plugin.strategy = Danger::AssignStrategies::RandomStrategy
+        @strategy_mock = instance_double(Danger::AssignStrategies::Strategy)
+        allow(Danger::AssignStrategies::RandomStrategy).to receive(:new).and_return(@strategy_mock)
+      end
+
+      it "Assign one reviewer" do
+        @plugin.gitlab_group = 'tech/ios'
+
+        expect(@strategy_mock).to receive(:assign!).with(1).and_return(['Sam'])
+
+        @plugin.assign!
+      end
+      it "Assign one reviewer" do
+        @plugin.gitlab_group = 'tech/ios'
+
+        expect(@strategy_mock).to receive(:assign!).with(1).and_return(['Sam'])
+
+        @plugin.assign!
+      end
+
+      it "Assign multiple reviewers" do
+        @plugin.gitlab_group = 'tech/ios'
+        @plugin.assignees_amount = 2
+
+        expect(@strategy_mock).to receive(:assign!).with(2).and_return(['Sam, Nic'])
+
+        @plugin.assign!
+      end
+
+      it "Doesn't assign if already asssigned" do
+        ENV['CI_MERGE_REQUEST_ASSIGNEES'] = 'Sam'
+        @plugin.gitlab_group = 'tech/ios'
+
+        expect(@strategy_mock).not_to receive(:assign!)
+
+        @plugin.assign!
+      end
+
+      it "Only assigns delta" do
+        ENV['CI_MERGE_REQUEST_ASSIGNEES'] = 'Sam,Nic'
+        @plugin.gitlab_group = 'tech/ios'
+        @plugin.assignees_amount = 3
+
+        expect(@strategy_mock).to receive(:assign!).with(1).and_return(['Rob'])
+
+        @plugin.assign!
+      end
+
+      ['CI_PROJECT_ID', 'CI_MERGE_REQUEST_IID'].each do |var|
+        it "Fails when required #{var} variables are not available" do
+          ENV[var] = nil
+          expect{@plugin.assign!}.to raise_error(RuntimeError)
+        end
+      end
+    end
+  end
+end
+

data/spec/least_busy_strategy_spec.rb

@@ -0,0 +1,47 @@
+require File.expand_path("../spec_helper", __FILE__)
+
+module Danger
+  describe Danger::AssignStrategies::RandomStrategy do
+    before do
+      testing_env.each { |k,v| ENV[k] = "#{v}" }
+      @dangerfile = testing_dangerfile
+
+      @mock_client = double(Gitlab::Client)
+      @author = Gitlab::User.new(1, 'Nic', 0)
+      @members = [@author, Gitlab::User.new(2, 'Tom'), Gitlab::User.new(3, 'Sam')]
+      allow(@mock_client).to receive(:fetch_author_for_mr).and_return(@author)
+      allow(@mock_client).to receive(:fetch_users_for_group).with(2200).and_return(@members)
+
+      @strategy = AssignStrategies::LeastBusyStrategy.new(client: @mock_client, project: 10, mr: 110, group: 2200)
+    end
+
+    it "Assign the one least busy" do
+      users_with_pending_mr_review = [@author, Gitlab::User.new(2, 'Tom', 1), Gitlab::User.new(3, 'Sam',2)]
+      expect(@mock_client).to receive(:users_with_pending_mr_review).and_return(users_with_pending_mr_review)
+
+      expect(@mock_client).to receive(:assign_mr_to_users) do |project, mr, users|
+        expect(project).to be == 10
+        expect(mr).to be == 110
+        expect(users).target.count eq 1
+        expect(users).target[0].username == 'Tom'
+      end
+
+      @strategy.assign!(1)
+    end
+
+    it "Assign the one least busy (if two are available)" do
+      users_with_pending_mr_review = [@author, Gitlab::User.new(2, 'Tom', 1), Gitlab::User.new(3, 'Sam',1)]
+      expect(@mock_client).to receive(:users_with_pending_mr_review).and_return(users_with_pending_mr_review)
+
+      expect(@mock_client).to receive(:assign_mr_to_users) do |project, mr, users|
+        expect(project).to be == 10
+        expect(mr).to be == 110
+        expect(users).target.count eq 1
+        expect(users).target[0].username == 'Tom'
+      end
+
+      @strategy.assign!(1)
+    end
+  end
+end
+

data/spec/random_strategy_spec.rb

@@ -0,0 +1,40 @@
+require File.expand_path("../spec_helper", __FILE__)
+
+module Danger
+  describe Danger::AssignStrategies::RandomStrategy do
+    before do
+      testing_env.each { |k,v| ENV[k] = "#{v}" }
+      @dangerfile = testing_dangerfile
+
+      @mock_client = double(Gitlab::Client)
+      @author = Gitlab::User.new(1, 'Nic')
+      @members = [@author, Gitlab::User.new(2, 'Tom'), Gitlab::User.new(3, 'Sam')]
+      allow(@mock_client).to receive(:fetch_author_for_mr).and_return(@author)
+      allow(@mock_client).to receive(:fetch_users_for_group).with(2200).and_return(@members)
+
+      @strategy = AssignStrategies::RandomStrategy.new(client: @mock_client, project: 10, mr: 110, group: 2200)
+    end
+
+    it "Assign the right amount of reviewers" do
+      expect(@mock_client).to receive(:assign_mr_to_users) do |project, mr, users|
+        expect(project).to be == 10
+        expect(mr).to be == 110
+        expect(users).target.count eq 2
+      end
+
+      @strategy.assign!(2)
+    end
+
+    it "Doesn't assign author" do
+      expect(@mock_client).to receive(:assign_mr_to_users) do |project, mr, users|
+        expect(project).to be == 10
+        expect(mr).to be == 110
+        expect(users).target.count eq 2
+      end
+
+      @strategy.assign!(3)
+    end
+
+  end
+end
+

data/spec/spec_helper.rb

@@ -0,0 +1,68 @@
+require "pathname"
+ROOT = Pathname.new(File.expand_path("../../", __FILE__))
+$:.unshift((ROOT + "lib").to_s)
+$:.unshift((ROOT + "spec").to_s)
+
+require "bundler/setup"
+require "pry"
+
+require "rspec"
+require "danger"
+
+if `git remote -v` == ''
+  puts "You cannot run tests without setting a local git remote on this repo"
+  puts "It's a weird side-effect of Danger's internals."
+  exit(0)
+end
+
+# Use coloured output, it's the best.
+RSpec.configure do |config|
+  config.filter_gems_from_backtrace "bundler"
+  config.color = true
+  config.tty = true
+end
+
+require "danger_plugin"
+
+# These functions are a subset of https://github.com/danger/danger/blob/master/spec/spec_helper.rb
+# If you are expanding these files, see if it's already been done ^.
+
+# A silent version of the user interface,
+# it comes with an extra function `.string` which will
+# strip all ANSI colours from the string.
+
+# rubocop:disable Lint/NestedMethodDefinition
+def testing_ui
+  @output = StringIO.new
+  def @output.winsize
+    [20, 9999]
+  end
+
+  cork = Cork::Board.new(out: @output)
+  def cork.string
+    out.string.gsub(/\e\[([;\d]+)?m/, "")
+  end
+  cork
+end
+# rubocop:enable Lint/NestedMethodDefinition
+
+# Example environment (ENV) that would come from
+# running a PR on TravisCI
+def testing_env
+  {
+    'CI_MERGE_REQUEST_IID' => '549',
+    'CI_MERGE_REQUEST_PROJECT_PATH' => '...',
+    'CI_MERGE_REQUEST_PROJECT_URL' => '...',
+    'DANGER_GITLAB_HOST' => 'github.com', # This needs to be the same as where the repo is stored due to Danger internals :facepalm:
+    'CI_API_V4_URL' => "https://gitlab.com/api/v4",
+    'CI_PROJECT_ID' => '346',
+    "GITLAB_CI" => true,
+    "DANGER_GITLAB_API_TOKEN" => "token-token-token"
+  }
+end
+
+# A stubbed out Dangerfile for use in tests
+def testing_dangerfile
+  env = Danger::EnvironmentManager.new(testing_env)
+  Danger::Dangerfile.new(env, testing_ui)
+end

metadata

@@ -0,0 +1,223 @@
+--- !ruby/object:Gem::Specification
+name: danger-gitlab_reviewbot
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+platform: ruby
+authors:
+- Fabio Gallonetto
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2020-05-06 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: danger-plugin-api
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: danger-gitlab
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.14'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+- !ruby/object:Gem::Dependency
+  name: listen
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.7
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.7
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: 'A review raffle bot for Gitlab '
+email:
+- fabio.gallonetto@curve.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".rubocop.yml"
+- Gemfile
+- Gemfile.lock
+- Guardfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- danger-gitlab_reviewbot.gemspec
+- lib/danger_gitlab_reviewbot.rb
+- lib/danger_plugin.rb
+- lib/gitlab_reviewbot/gem_version.rb
+- lib/gitlab_reviewbot/gitlab.rb
+- lib/gitlab_reviewbot/plugin.rb
+- lib/gitlab_reviewbot/strategies.rb
+- lib/gitlab_reviewbot/strategies/least_busy.rb
+- lib/gitlab_reviewbot/strategies/random.rb
+- lib/gitlab_reviewbot/strategies/strategy.rb
+- spec/gitlab_reviewbot_spec.rb
+- spec/least_busy_strategy_spec.rb
+- spec/random_strategy_spec.rb
+- spec/spec_helper.rb
+homepage: https://github.com/curve-technology/danger-gitlab_reviewbot
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubygems_version: 3.0.3
+signing_key: 
+specification_version: 4
+summary: A review raffle bot for Gitlab.
+test_files:
+- spec/gitlab_reviewbot_spec.rb
+- spec/least_busy_strategy_spec.rb
+- spec/random_strategy_spec.rb
+- spec/spec_helper.rb