bitbucket_rest_api2 0.2.0

data/spec/bitbucket_rest_api/normalizer_spec.rb

@@ -0,0 +1,30 @@
+# encoding: utf-8
+
+require 'spec_helper'
+require 'bitbucket_rest_api/core_ext/hash'
+
+describe BitBucket::Normalizer, '#normalize!' do
+  let(:hash) { { 'a' => { :b => { 'c' => 1 }, 'd' => ['a', { :e => 2 }] } } }
+
+  let(:klass) do
+    Class.new do
+      include BitBucket::Normalizer
+    end
+  end
+
+  subject(:instance) { klass.new }
+
+  context '#normalize!' do
+    it 'converts hash keys to string' do
+      ['a', 'b', 'c'].each do |key|
+        expect(subject.normalize!(hash).has_deep_key?(key)).to eq(true)
+      end
+    end
+
+    it 'should stringify all the keys inside nested hash' do
+      actual = subject.normalize! hash
+      expected = { 'a' => { 'b'=> { 'c' => 1 }, 'd' => ['a', { 'e'=> 2 }] } }
+      expect(actual).to eq expected
+    end
+  end
+end # BitBucket::Normalizer

data/spec/bitbucket_rest_api/parameter_filter_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+require 'bitbucket_rest_api/core_ext/hash'
+
+describe BitBucket::ParameterFilter, '#filter!' do
+  let(:hash) {  { :a => { :b => { :c => 1 } } }  }
+  let(:array) { [{ :a => { :b => { :c => 1 } } }, {d: {e: 2}}] }
+
+  let(:klass) {
+    Class.new do
+      include BitBucket::ParameterFilter
+    end
+  }
+
+  subject(:instance) { klass.new }
+
+  it 'removes unwanted keys from hash' do
+    instance.filter!([:a], hash)
+    expect(hash.has_deep_key?(:a)).to eq(true)
+    expect(hash.has_deep_key?(:b)).to eq(false)
+    expect(hash.has_deep_key?(:c)).to eq(false)
+  end
+
+  it 'removes unwanted keys from array of hashes' do
+    instance.filter!([:a, :d], array)
+    expect(array[0].has_deep_key?(:a)).to eq(true)
+    expect(array[0].has_deep_key?(:b)).to eq(false)
+    expect(array[0].has_deep_key?(:c)).to eq(false)
+    expect(array[1].has_deep_key?(:d)).to eq(true)
+    expect(array[1].has_deep_key?(:e)).to eq(false)
+  end
+
+  it 'recursively filters inputs tree' do
+    instance.filter!([:a, :b], hash)
+    expect(hash.has_deep_key?(:c)).to eq(false)
+  end
+
+  it 'filters inputs tree only on top level' do
+    instance.filter!([:a, :b], hash, :recursive => false)
+    expect(hash.has_deep_key?(:c)).to eq(true)
+  end
+end

data/spec/bitbucket_rest_api/repos/changesets_spec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Changesets do
+  let(:changesets) { BitBucket::Repos::Changesets.new }
+
+  describe '.list' do
+    before do
+      expect(changesets).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/changesets',
+        {},
+        {}
+      ).and_return(['changset1', 'changeset2', 'changeset3'])
+    end
+
+    context 'without a block' do
+      it 'should send a GET request for the changesets belonging to the given repo' do
+        changesets.list('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'should send a GET request for the changesets belonging to the given repo' do
+        changesets.list('mock_username', 'mock_repo') { |changeset| changeset }
+      end
+    end
+  end
+
+  describe '.get' do
+    before do
+      expect(changesets).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/changesets/test_sha',
+        {},
+        {}
+      )
+    end
+
+    it 'should send a GET request for a particular changeset belonging to the given repo' do
+      changesets.get('mock_username', 'mock_repo', 'test_sha')
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/commits_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Commits do
+  let(:commits) { BitBucket::Repos::Commits.new }
+
+  describe '.list' do
+    before do
+      expect(commits).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/commits',
+        {},
+        {}
+      )
+    end
+
+    it 'should send a GET request for the commits belonging to the given repo' do
+      commits.list('mock_username', 'mock_repo')
+    end
+  end
+end

data/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

data/spec/bitbucket_rest_api/repos/download_spec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Download do
+  describe '.get' do
+    it 'should send a GET request for the download link for the given repo' do
+      skip 'this is a regression test and the method is not finished being written'
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/following_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Following do
+  let(:following) { BitBucket::Repos::Following.new }
+
+  describe '.followers' do
+    before do
+      expect(following).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/followers/',
+        {},
+        {}
+      ).and_return(['follower1', 'follower2', 'follower3'])
+    end
+
+    context 'without a block' do
+      it 'should send a GET request for the followers belonging to the given repo' do
+        following.followers('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'should send a GET request for the followers belonging to the given repo' do
+        following.followers('mock_username', 'mock_repo') { |follower| follower }
+      end
+    end
+  end
+
+  # TODO: implement this method in the User class where it belongs
+  describe '.followed' do
+    before do
+      expect(following).to receive(:request).with(
+        :get,
+        '/2.0/user/follows',
+        {},
+        {}
+      ).and_return(['followed1', 'followed2', 'followed3'])
+    end
+
+    context 'without a block' do
+      it 'should send a GET request for the followers belonging to a particular user' do
+        following.followed
+      end
+    end
+
+    context 'with a block' do
+      it 'should send a GET request for the followers belonging to a particular user' do
+        following.followed { |followed| followed }
+      end
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/forks_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Forks do
+  let(:forks) { BitBucket::Repos::Forks.new }
+
+  describe '.list' do
+    before do
+      expect(forks).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/forks/',
+        {},
+        {}
+      ).and_return(['fork1', 'fork2', 'fork3'])
+    end
+
+    context 'without a block' do
+      it 'sends a GET request for the forks of the given repo' do
+        forks.list('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'sends a GET request for the forks of the given repo' do
+        forks.list('mock_username', 'mock_repo') { |fork| fork }
+      end
+    end
+  end
+
+  # TODO: make sure this is documented in the README since it is not so
+  # in the official API docs
+  describe '.create' do
+    before do
+      expect(forks).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_username/mock_repo/fork',
+        { 'name' => 'mock_fork_name'},
+        {}
+      )
+    end
+
+    it 'should send a POST request to create a fork of the given repo' do
+      forks.create('mock_username', 'mock_repo', { 'name' => 'mock_fork_name' })
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/keys_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Keys do
+  let(:deploy_keys) { described_class.new }
+  describe '.list' do
+    before do
+      expect(deploy_keys).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/deploy-keys/',
+        {},
+        {}
+      ).and_return(['key1', 'key2', 'key3'])
+    end
+
+    context 'without a block' do
+      it 'should make a GET request for the deploy keys belonging to the given repo' do
+        deploy_keys.list('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'should make a GET request for the deploy keys belonging to the given repo' do
+        deploy_keys.list('mock_username', 'mock_repo') { |key| key }
+      end
+    end
+  end
+
+  describe '.create' do
+    before do
+      expect(deploy_keys).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_username/mock_repo/deploy-keys/',
+        { 'key' => 'mock_ssh_key', 'label' => 'mock_label' },
+        {}
+      )
+    end
+
+    it 'should make a POST request for the deploy keys belonging to the given repo' do
+      deploy_keys.create('mock_username', 'mock_repo', { key: 'mock_ssh_key', label: 'mock_label' })
+    end
+  end
+
+  describe '.edit' do
+    before do
+      expect(deploy_keys).to receive(:request).with(
+         :put,
+         '/2.0/repositories/mock_username/mock_repo/deploy-keys/1',
+         { 'key' => 'mock_ssh_key', 'label' => 'mock_label' },
+         {}
+       )
+    end
+
+    it 'should make a PUT request for the deploy keys belonging to the given repo' do
+      deploy_keys.edit('mock_username', 'mock_repo', 1, { key: 'mock_ssh_key', label: 'mock_label' })
+    end
+  end
+
+  describe '.delete' do
+    before do
+      expect(deploy_keys).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_username/mock_repo/deploy-keys/mock_id',
+        { 'key' => 'mock_ssh_key', 'label' => 'mock_label' },
+        {}
+      )
+    end
+
+    it 'should make a DELETE request for the deploy keys belonging to the given repo' do
+      deploy_keys.delete('mock_username', 'mock_repo', 'mock_id', { key: 'mock_ssh_key', label: 'mock_label' })
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/pull_request_spec.rb

@@ -0,0 +1,288 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::PullRequest do
+  subject { described_class.new }
+  describe '#list' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests',
+        {},
+        {}
+      ).and_return(['pr1', 'pr2', 'pr3'])
+    end
+
+    context 'without a block' do
+      it 'makes a GET request for all pull requests belonging to the repo' do
+        subject.list('mock_user', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'makes a GET request for all pull requests belonging to the repo' do
+        subject.list('mock_user', 'mock_repo') { |pr| pr }
+      end
+    end
+  end
+
+  describe '#participants' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        "/2.0/repositories/mock_user/mock_repo/pullrequests/mock_pull_request_id/participants",
+        {},
+        {}
+      ).and_return(['participant1', 'participant2', 'participant3'])
+    end
+
+    context 'without a block' do
+      it 'makes a GET request for all participants belonging to the repo' do
+        subject.participants('mock_user', 'mock_repo', 'mock_pull_request_id')
+      end
+    end
+
+    context 'with a block' do
+      it 'makes a GET request for all participants belonging to the repo' do
+        subject.participants('mock_user', 'mock_repo', 'mock_pull_request_id') { |p| p }
+      end
+    end
+  end
+
+  describe '#get' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        "/2.0/repositories/mock_user/mock_repo/pullrequests/mock_pull_request_id",
+        {}
+      )
+    end
+
+    it 'makes a GET request for the pull request belonging to the repo' do
+      subject.get('mock_user', 'mock_repo', 'mock_pull_request_id')
+    end
+  end
+
+  describe '#create' do
+    before do
+      @params = {
+        title: "mock_pr_title",
+        description: "mock_pull_request_description",
+        source: {
+          branch: {
+            name: "mock_source_branch_name"
+          },
+          repository: {
+            full_name: "mock_owner/mock_repo"
+          }
+        },
+        destination: {
+          branch: {
+            name: "mock_destination_branch_name"
+          },
+          commit: {
+            hash: "mock_uuid"
+          }
+        },
+        close_source_branch: true
+      }
+    end
+
+    it 'makes a POST request to create a new pull request' do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests',
+        @params
+      )
+
+      subject.create('mock_user', 'mock_repo', @params)
+    end
+
+    it 'validates presence of required params' do
+      expect do
+      subject.create(
+        'mock_user',
+        'mock_repo',
+        {
+          title: "",
+          description: "mock_pull_request_description",
+          source: {
+            branch: {
+              name: "mock_source_branch_name"
+            },
+            repository: {
+              full_name: "mock_owner/mock_repo"
+            }
+          },
+          destination: {
+            branch: {
+              name: "mock_destination_branch_name"
+            },
+            commit: {
+              hash: "mock_uuid"
+            }
+          },
+          close_source_branch: true
+        }
+      )
+      end.to raise_error
+    end
+  end
+
+  describe '.put' do
+    before do
+      expect(subject).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id',
+        {}
+      )
+    end
+
+    it 'makes a PUT request for the given pull request' do
+      subject.update('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+  describe '.commits' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/commits',
+        {}
+      )
+    end
+
+    it 'makes a GET request for the commits' do
+      subject.commits('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+  describe '.commits' do
+    before do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/approve',
+        {}
+      )
+    end
+
+    it 'makes a POST request' do
+      subject.approve('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+  describe '.delete_approval' do
+    before do
+      expect(subject).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/approve',
+        {}
+      )
+    end
+
+    it 'makes a DELTE request' do
+      subject.delete_approval('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+
+  describe '.diff' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/diff',
+        {}
+      )
+    end
+
+    it 'makes a GET request for the diff of the pull request' do
+      subject.diff('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+  describe '.all_activity' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/activity',
+        {}
+      )
+    end
+
+    it 'makes a GET request' do
+      subject.all_activity('mock_user', 'mock_repo')
+    end
+  end
+
+
+  describe '.activity' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/activity',
+        {}
+      )
+    end
+
+    it 'makes a GET request' do
+      subject.activity('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+
+  describe '.accept_and_merge' do
+    before do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/merge',
+        {}
+      )
+    end
+
+    it 'makes a POST request' do
+      subject.merge('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+  describe '.decline' do
+    before do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/decline',
+        {}
+      )
+    end
+
+    it 'makes a POST request' do
+      subject.decline('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+
+  describe '.comments' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/comments',
+        {}
+      )
+    end
+
+    it 'makes a GET request' do
+      subject.comments('mock_user', 'mock_repo', 'mock_id')
+    end
+  end
+
+  describe '.comment' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/pullrequests/mock_id/comments/comment_id',
+        {}
+      )
+    end
+
+    it 'makes a GET request' do
+      subject.comment('mock_user', 'mock_repo', 'mock_id', 'comment_id')
+    end
+  end
+end