bitbucket_rest_api2 0.2.0

data/spec/bitbucket_rest_api/repos/sources_spec.rb

@@ -0,0 +1,77 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Sources do
+  let(:subject) { BitBucket::Repos::Sources.new }
+
+  describe '#list' do
+    context 'when some parameters are missing' do
+      it 'raises an error' do
+        expect do
+          subject.list(
+            'mock_username',
+            'mock_repo'
+          )
+        end.to raise_error(ArgumentError)
+      end
+    end
+
+    context 'when path parameter is empty' do
+      before do
+        expect(subject).to receive(:request).with(
+          :get,
+          '/2.0/repositories/mock_username/mock_repo/src/moch_sha/',
+          {},
+          {}
+        )
+      end
+
+      it 'sends a GET request for a list of all source files' do
+        subject.list('mock_username', 'mock_repo', 'moch_sha', '')
+      end
+    end
+
+    context 'when path parameter is defined' do
+      before do
+        expect(subject).to receive(:request).with(
+          :get,
+          '/2.0/repositories/mock_username/mock_repo/src/moch_sha/app/controller',
+          {},
+          {}
+        )
+      end
+
+      it 'send a GET request for a list of the source files under the specified path' do
+        subject.list('mock_username', 'mock_repo', 'moch_sha', 'app/controller')
+      end
+    end
+  end
+
+  describe '#get' do
+    context 'when some parameters are missing' do
+      it 'raises an error' do
+        expect do
+          subject.get(
+            'mock_username',
+            'mock_repo',
+            'moch_sha'
+          )
+        end.to raise_error(ArgumentError)
+      end
+    end
+
+    context 'when path parameter is defined' do
+      before do
+        expect(subject).to receive(:request).with(
+          :get,
+          '/2.0/repositories/mock_username/mock_repo/raw/moch_sha/app/assets/images/logo.jpg',
+          {},
+          {}
+        )
+      end
+
+      it "send a GET request for a source file's size and contents" do
+        subject.get('mock_username', 'mock_repo', 'moch_sha', 'app/assets/images/logo.jpg')
+      end
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos/webhooks_spec.rb

@@ -0,0 +1,245 @@
+require 'spec_helper'
+
+describe BitBucket::Repos::Webhooks, wip: true do
+  subject { described_class.new }
+  let(:post_put_params) do
+    {
+      'description' => 'mock_description',
+      'url' => 'mock_url',
+      'active' => true,
+      'events' => ['repo:push']
+    }
+  end
+
+  let(:missing_key_params) do
+    {
+      'description' => 'mock_description',
+      'active' => true,
+      'events' => ['repo:push']
+    }
+  end
+
+  let(:blank_value_params) do
+    {
+      'description' => '',
+      'url' => 'mock_url',
+      'active' => true,
+      'events' => ['repo:push']
+    }
+  end
+
+  let(:bad_event_params) do
+    {
+      'description' => 'mock_description',
+      'url' => 'mock_url',
+      'active' => true,
+      'events' => ['bad:event']
+    }
+  end
+
+  let(:missing_events_params) do
+    {
+      'description' => 'mock_description',
+      'url' => 'mock_url',
+      'active' => true,
+      'events' => []
+    }
+  end
+
+  describe '#create' do
+    context 'when required fields are missing from params' do
+      context 'when a required key is missing' do
+        it 'raises an instance of BitBucket::Error::RequiredParams' do
+          expect do
+            subject.create(
+              'mock_username',
+              'mock_repo',
+              missing_key_params
+            )
+          end.to raise_error(BitBucket::Error::RequiredParams)
+        end
+      end
+
+      context 'when values of required keys are blank' do
+        it 'raises an instance of BitBucket::Error::RequiredParams' do
+          expect do
+            subject.create(
+              'mock_username',
+              'mock_repo',
+              blank_value_params
+            )
+          end.to raise_error(
+            BitBucket::Error::BlankValue,
+            "The value for: 'description', cannot be blank :("
+          )
+        end
+      end
+    end
+
+    it 'validates the given events' do
+      allow(subject).to(receive(:request))
+
+      expect do
+        subject.create(
+          'mock_username',
+          'mock_repo',
+          bad_event_params
+
+        )
+      end.to raise_error(
+        BitBucket::Error::BadEvents,
+        "The event: 'bad:event', does not exist :("
+      )
+    end
+
+    it 'checks that at least one event is given' do
+      allow(subject).to(receive(:request))
+
+      expect do
+        subject.create(
+          'mock_username',
+          'mock_repo',
+          missing_events_params
+        )
+      end.to raise_error(
+        BitBucket::Error::NoEvents,
+        "At least one event is required, none given :("
+      )
+    end
+
+    it 'makes a POST request to create a specific webhook' do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_username/mock_repo/hooks',
+        post_put_params,
+        {}
+      )
+
+      subject.create(
+        'mock_username',
+        'mock_repo',
+        post_put_params
+      )
+    end
+  end
+
+  describe '#list' do
+    it 'makes a GET request for all the webhooks beloning to the given repo' do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/hooks',
+        {},
+        {}
+      )
+
+      subject.list('mock_username', 'mock_repo')
+    end
+  end
+
+  describe '#get' do
+    it 'makes a GET request for a specific webook' do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/hooks/mock_uuid',
+        {},
+        {}
+      )
+
+      subject.get('mock_username', 'mock_repo', 'mock_uuid')
+    end
+  end
+
+  describe '#edit' do
+    context 'when required fields are missing from params' do
+      context 'when a required key is missing' do
+        it 'raises an instance of BitBucket::Error::RequiredParams' do
+          expect do
+            subject.edit(
+              'mock_username',
+              'mock_repo',
+              'mock_uuid',
+              missing_key_params
+            )
+          end.to raise_error(BitBucket::Error::RequiredParams)
+        end
+      end
+
+      context 'when values of required keys are blank' do
+        it 'raises an instance of BitBucket::Error::RequiredParams' do
+          expect do
+            subject.edit(
+              'mock_username',
+              'mock_repo',
+              'mock_uuid',
+              blank_value_params
+            )
+          end.to raise_error(
+            BitBucket::Error::BlankValue,
+            "The value for: 'description', cannot be blank :("
+          )
+        end
+      end
+    end
+
+    it 'validates the existence of the given events' do
+      allow(subject).to(receive(:request))
+
+      expect do
+        subject.edit(
+          'mock_username',
+          'mock_repo',
+          'mock_uuid',
+          bad_event_params
+        )
+      end.to raise_error(
+        BitBucket::Error::BadEvents,
+        "The event: 'bad:event', does not exist :("
+      )
+    end
+
+    it 'validates that at least one event is given' do
+      allow(subject).to(receive(:request))
+
+      expect do
+        subject.edit(
+          'mock_username',
+          'mock_repo',
+          'mock_uuid',
+          missing_events_params
+        )
+      end.to raise_error(
+        BitBucket::Error::NoEvents,
+        "At least one event is required, none given :("
+      )
+    end
+
+    it 'makes a PUT request for the given webhook' do
+      expect(subject).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_username/mock_repo/hooks/mock_uuid',
+        post_put_params,
+        {}
+      )
+
+      subject.edit(
+        'mock_username',
+        'mock_repo',
+        'mock_uuid',
+        post_put_params
+      )
+    end
+  end
+
+  describe '#delete' do
+    it 'sends a DELETE request for the given webhook' do
+      expect(subject).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_username/mock_repo/hooks/mock_uuid',
+        {},
+        {}
+      )
+
+      subject.delete('mock_username', 'mock_repo', 'mock_uuid')
+    end
+  end
+end

data/spec/bitbucket_rest_api/repos_spec.rb

@@ -0,0 +1,157 @@
+require 'spec_helper'
+
+describe BitBucket::Repos do
+  let(:repo) { BitBucket::Repos.new }
+
+  # class_eval is setting global variables in api.rb, this is creating
+  # failed expectations when these variables are not reset to nil before
+  # the next test is run. Therefore we must clear them manually as
+  # for the user attribute below...
+  after do
+    repo.clear_user
+  end
+
+  describe '.create' do
+    before do
+      expect(repo).to receive(:request).with(
+        :post,
+        '/2.0/repositories/',
+        BitBucket::Repos::DEFAULT_REPO_OPTIONS.merge({ 'owner' => 'mock_owner', 'name' => 'mock_repo' }),
+        {}
+      )
+    end
+
+    it 'should send a POST request to create the repo' do
+      repo.create({ 'owner' => 'mock_owner', 'name' => 'mock_repo' })
+    end
+  end
+
+  describe '.delete' do
+    before do
+      expect(repo).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_username/mock_repo',
+        {},
+        {}
+      )
+    end
+
+    it 'should send a DELETE request for the given repo' do
+      repo.delete('mock_username', 'mock_repo')
+    end
+  end
+
+  # TODO: fix case where block_given? returns true
+  describe '.branches' do
+    before do
+      expect(repo).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/branches/',
+        {},
+        {}
+      ).and_return(['branch1', 'branch2', 'branch3'])
+    end
+
+    context 'without a block' do
+      it 'invokes the .request method' do
+        repo.branches('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'invokes the .request method' do
+        repo.branches('mock_username', 'mock_repo') { |branch| branch }
+      end
+    end
+  end
+
+  describe '.edit' do
+    before do
+      expect(repo).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_username/mock_repo/',
+        BitBucket::Repos::DEFAULT_REPO_OPTIONS.merge({ 'owner' => 'mock_owner' }),
+        {}
+      )
+    end
+
+    it 'should send a PUT request for the given repo' do
+      repo.edit('mock_username', 'mock_repo', { 'owner' => 'mock_owner' })
+    end
+  end
+
+  # TODO: make sure this gets documented in gem since it is not in official docs
+  describe '.get' do
+    before do
+      expect(repo).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo',
+        {},
+        {}
+      )
+    end
+
+    it 'should send a GET request for the given repo' do
+      repo.get('mock_username', 'mock_repo', {})
+    end
+  end
+
+  describe '.list' do
+    before do
+      expect(repo).to receive(:request).with(
+        :get,
+        '/2.0/user/repositories',
+        {},
+        {}
+      ).and_return(['repo1', 'repo2' ,'repo3'])
+    end
+
+    # FIXME: this method belongs in the User class!
+    context 'without a block' do
+      it 'should send a GET request for the authenticated users repos' do
+        repo.list
+      end
+    end
+
+    context 'with a block' do
+      it 'should send a GET request for the authenticated users repos' do
+        repo.list { |repo| repo }
+      end
+    end
+  end
+
+  describe '.tags' do
+    before do
+      expect(repo).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/tags/',
+        {},
+        {}
+      ).and_return(['tag1', 'tag2' ,'tag3'])
+    end
+
+    context 'without a block' do
+      it 'should send a GET request for the tags belonging to the given repo' do
+        repo.tags('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'should send a GET request for the tags belonging to the given repo' do
+        repo.tags('mock_username', 'mock_repo') { |tag| tag }
+      end
+    end
+  end
+
+  describe "getter methods" do
+    it "returns an object of the correct class" do
+      expect(repo.changesets).to be_a BitBucket::Repos::Changesets
+      expect(repo.keys).to be_a BitBucket::Repos::Keys
+      expect(repo.following).to be_a BitBucket::Repos::Following
+      expect(repo.commits).to be_a BitBucket::Repos::Commits
+      expect(repo.pull_request).to be_a BitBucket::Repos::PullRequest
+      expect(repo.forks).to be_a BitBucket::Repos::Forks
+      expect(repo.download).to be_a BitBucket::Repos::Download
+    end
+  end
+end