bitbuckets 0.2.0

data/spec/bitbucket_rest_api/issues_spec.rb

@@ -0,0 +1,91 @@
+# frozen_string_literal: true
+require 'spec_helper'
+require 'ostruct'
+
+describe BitBucket::Issues do
+  let(:issue) { BitBucket::Issues.new }
+
+  describe '.create' do
+    before do
+      expect(issue).to receive(:request).with(
+        :post,
+        '/1.0/repositories/mock_username/mock_repo/issues/',
+        { 'title' => 'mock_issue' },
+        {}
+      )
+    end
+
+    it 'should send a POST request to create the issue' do
+      issue.create('mock_username', 'mock_repo', 'title' => 'mock_issue')
+    end
+  end
+
+  describe '.edit' do
+    before do
+      expect(issue).to receive(:request).with(
+        :put,
+        '/1.0/repositories/mock_username/mock_repo/issues/1/',
+        { 'title' => 'new_title' },
+        {}
+      )
+    end
+
+    it 'should send a PUT request for the given issue' do
+      issue.edit('mock_username', 'mock_repo', 1, 'title' => 'new_title')
+    end
+  end
+
+  describe '.get' do
+    before do
+      expect(issue).to receive(:request).with(
+        :get,
+        '/1.0/repositories/mock_username/mock_repo/issues/1',
+        {},
+        {}
+      )
+    end
+
+    it 'should send a GET request for the given issue' do
+      issue.get('mock_username', 'mock_repo', 1, {})
+    end
+  end
+
+  describe '.delete' do
+    before do
+      expect(issue).to receive(:request).with(
+        :delete,
+        '/1.0/repositories/mock_username/mock_repo/issues/1',
+        {},
+        {}
+      )
+    end
+
+    it 'should send a DELETE request for the given issue' do
+      issue.delete('mock_username', 'mock_repo', 1)
+    end
+  end
+
+  describe '.list_repo' do
+    before do
+      expect(issue).to receive(:request).with(
+        :get,
+        '/1.0/repositories/mock_username/mock_repo/issues',
+        {},
+        {}
+      ).and_return(OpenStruct.new(issues: [])).twice
+    end
+
+    it 'should send a GET request for the issues for that repo' do
+      issue.list_repo('mock_username', 'mock_repo')
+      issue.list_repo('mock_username', 'mock_repo') { |issue| issue }
+    end
+  end
+
+  describe 'getter methods' do
+    it 'returns an object of the correct class' do
+      expect(issue.comments).to be_a BitBucket::Issues::Comments
+      expect(issue.components).to be_a BitBucket::Issues::Components
+      expect(issue.milestones).to be_a BitBucket::Issues::Milestones
+    end
+  end
+end

data/spec/bitbucket_rest_api/normalizer_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+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
+      %w[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,42 @@
+# frozen_string_literal: true
+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) do
+    Class.new do
+      include BitBucket::ParameterFilter
+    end
+  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!(%i[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!(%i[a b], hash)
+    expect(hash.has_deep_key?(:c)).to eq(false)
+  end
+
+  it 'filters inputs tree only on top level' do
+    instance.filter!(%i[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,44 @@
+# frozen_string_literal: true
+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,
+        '/1.0/repositories/mock_username/mock_repo/changesets',
+        {},
+        {}
+      ).and_return(%w[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,
+        '/1.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,21 @@
+# frozen_string_literal: true
+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/components_spec.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+describe BitBucket::Repos::Components do
+  subject { described_class.new }
+  describe '#list' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/components',
+        {},
+        {}
+      ).and_return('values' => %w[component1 component2 component3])
+    end
+
+    context 'without a block' do
+      it 'makes a GET request for all components defined in the issue tracker' do
+        subject.list('mock_user', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'makes a GET request for all components defined in the issue tracker' do
+        subject.list('mock_user', 'mock_repo') { |component| component }
+      end
+    end
+  end
+
+  describe '#get' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_user/mock_repo/components/1',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a GET request for all components defined in the issue tracker' do
+      subject.get('mock_user', 'mock_repo', 1)
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/default_reviewers_spec.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+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,10 @@
+# frozen_string_literal: true
+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,53 @@
+# frozen_string_literal: true
+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,
+        '/1.0/repositories/mock_username/mock_repo/followers/',
+        {},
+        {}
+      ).and_return(%w[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,
+        '/1.0/user/follows',
+        {},
+        {}
+      ).and_return(%w[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,46 @@
+# frozen_string_literal: true
+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(%w[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,
+        '/1.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