Merge pull request #63 from cltweedie/default-reviewers

Default Reviewers

Author: dmgarland

lib/bitbucket_rest_api/api.rb

@@ -79,6 +79,11 @@ def method_missing(method, *args, &block) # :nodoc:
       end
     end
 
+    def update_and_validate_user_repo_params(user_name, repo_name=nil)
+      _update_user_repo_params(user_name, repo_name)
+      _validate_user_repo_params(user, repo) unless user? && repo?
+    end
+
     def _update_user_repo_params(user_name, repo_name=nil) # :nodoc:
       self.user = user_name || self.user
       self.repo = repo_name || self.repo

lib/bitbucket_rest_api/repos.rb

@@ -15,7 +15,8 @@ class Repos < API
                  :Commits     => 'commits',
                  :Download    => 'download',
                  :Webhooks    => 'webhooks',
-                 :PullRequest => 'pull_request'
+                 :PullRequest => 'pull_request',
+                 :DefaultReviewers => 'default_reviewers'
 
     DEFAULT_REPO_OPTIONS = {
         "website"         => "",
@@ -83,6 +84,10 @@ def pull_request
       @pull_request ||= ApiFactory.new 'Repos::PullRequest'
     end
 
+    def default_reviewers
+      @default_reviewers ||= ApiFactory.new 'Repos::DefaultReviewers'
+    end
+
     # List branches
     #
     # = Examples

lib/bitbucket_rest_api/repos/default_reviewers.rb

@@ -0,0 +1,59 @@
+module BitBucket
+  class Repos::DefaultReviewers < API
+
+    # List default reviewers
+    #
+    # = Examples
+    #  bitbucket = BitBucket.new
+    #  bitbucket.repos.default_reviewers.list 'user-name', 'repo-name'
+    #  bitbucket.repos.default_reviewers.list 'user-name', 'repo-name' { |reviewer| ... }
+    #
+    def list(user_name, repo_name, params={})
+      update_and_validate_user_repo_params(user_name, repo_name)
+      normalize! params
+
+      response = get_request("/2.0/repositories/#{user_name}/#{repo_name}/default-reviewers", params)
+      return response unless block_given?
+      response.each { |el| yield el }
+    end
+    alias :all :list
+
+    # Get a default reviewer's info
+    #
+    # = Examples
+    #  bitbucket = BitBucket.new
+    #  bitbucket.repos.default_reviewers.get 'user-name', 'repo-name', 'reviewer-username'
+    #
+    def get(user_name, repo_name, reviewer_username, params={})
+      update_and_validate_user_repo_params(user_name, repo_name)
+      normalize! params
+
+      get_request("/2.0/repositories/#{user_name}/#{repo_name}/default-reviewers/#{reviewer_username}", params)
+    end
+
+    # Add a user to the default-reviewers list for the repo
+    #
+    # = Examples
+    #  bitbucket = BitBucket.new
+    #  bitbucket.repos.default_reviewers.add 'user-name', 'repo-name', 'reviewer-username'
+    #
+    def add(user_name, repo_name, reviewer_username, params={})
+      update_and_validate_user_repo_params(user_name, repo_name)
+      normalize! params
+
+      put_request("/2.0/repositories/#{user_name}/#{repo_name}/default-reviewers/#{reviewer_username}", params)
+    end
+
+    # Remove a user from the default-reviewers list for the repo
+    #
+    # = Examples
+    #  bitbucket = BitBucket.new
+    #  bitbucket.repos.default_reviewers.remove 'user-name', 'repo-name', 'reviewer-username'
+    #
+    def remove(user_name, repo_name, reviewer_username, params={})
+      update_and_validate_user_repo_params(user_name, repo_name)
+      normalize! params
+      delete_request("/2.0/repositories/#{user_name}/#{repo_name}/default-reviewers/#{reviewer_username}", params)
+    end
+  end
+end

spec/bitbucket_rest_api/repos/default_reviewers_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::DefaultReviewers do
+  subject { described_class.new }
+  describe '#list' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/default-reviewers',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a GET request for all default reviewers belonging to the repo' do
+      subject.list('mock_user', 'mock_repo')
+    end
+  end
+
+  describe '#get' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/default-reviewers/mock_reviewer',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a GET request for a default reviewer by username' do
+      subject.get('mock_user', 'mock_repo', 'mock_reviewer')
+    end
+  end
+
+  describe '#add' do
+    before do
+      expect(subject).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_user/mock_repo/default-reviewers/mock_reviewer',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a PUT request to add the new reviewer to the default reviewers list' do
+      subject.add('mock_user', 'mock_repo', 'mock_reviewer')
+    end
+  end
+
+  describe '#remove' do
+    before do
+      expect(subject).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_user/mock_repo/default-reviewers/mock_reviewer',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a DELETE request to remove a reviewer from the list' do
+      subject.remove('mock_user', 'mock_repo', 'mock_reviewer')
+    end
+  end
+end