bitbucket_rest_api2 0.2.0

data/spec/bitbucket_rest_api/client_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe BitBucket::Client do
+  let(:client) { described_class.new }
+
+  it "returns the a new object of the correct class" do
+    expect(client.issues).to be_a BitBucket::Issues
+    expect(client.repos).to be_a BitBucket::Repos
+    expect(client.users).to be_a BitBucket::Users
+    expect(client.user_api).to be_a BitBucket::User
+    expect(client.invitations).to be_a BitBucket::Invitations
+    expect(client.pull_requests).to be_a BitBucket::Repos::PullRequest
+    expect(client.oauth).to be_a BitBucket::Request::OAuth
+  end
+end

data/spec/bitbucket_rest_api/core_ext/array_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+require 'bitbucket_rest_api/core_ext/array'
+
+describe Array do
+  let(:array) { [:a, :b, :c, :d, { key: :value }] }
+
+  describe '#extract_options!' do
+    it 'selects a hash from the arguments list' do
+      expect(array.extract_options!).to eq({ key: :value })
+    end
+  end
+end

data/spec/bitbucket_rest_api/core_ext/hash_spec.rb

@@ -0,0 +1,49 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Hash do
+
+  before do
+    BitBucket.new
+    @hash = { :a => 1, :b => 2, :c => 'e'}
+    @serialized = "a=1&b=2&c=e"
+    @nested_hash = { 'a' => { 'b' => {'c' => 1 } } }
+    @symbols = { :a => { :b => { :c => 1 } } }
+  end
+
+  context '#symbolize_keys' do
+    it 'should respond to symbolize_keys' do
+      expect(@nested_hash).to respond_to :symbolize_keys
+    end
+  end
+
+  context '#symbolize_keys!' do
+    it 'should respond to symbolize_keys!' do
+      expect(@nested_hash).to respond_to :symbolize_keys!
+    end
+
+    it 'should convert nested keys to symbols' do
+      expect(@nested_hash.symbolize_keys!).to eq @symbols
+
+      @nested_hash_with_array = { 'a' => { 'b' => [{'c' => 1}] } }
+      expect(@nested_hash_with_array.symbolize_keys!).to eq({:a=>{:b=>[{:c=>1}]}})
+    end
+  end
+
+  context '#serialize' do
+    it 'should respond to serialize' do
+      expect(@nested_hash).to respond_to :serialize
+    end
+
+    it 'should serialize hash' do
+      expect(@hash.serialize).to eq @serialized
+    end
+  end
+
+  context '#deep_key?' do
+    it 'should find key inside nested hash' do
+      expect(@nested_hash.has_deep_key?('c')).to be_truthy
+    end
+  end
+end # Hash

data/spec/bitbucket_rest_api/deprecation_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe BitBucket do
+  let(:method) { 'create_repos'}
+  let(:alt_method) { 'repos.create'}
+
+  it { expect(described_class.constants).to include :DEPRECATION_PREFIX }
+
+  context '.deprecate' do
+    before do
+      BitBucket.deprecation_tracker = []
+    end
+
+    it 'tracks messages' do
+      expect(BitBucket).to receive(:warn).once()
+      BitBucket.deprecate(method)
+      BitBucket.deprecate(method)
+    end
+
+    it 'prints the message through Kernel' do
+      expect(BitBucket).to receive(:warn).once()
+      BitBucket.deprecate method
+    end
+  end
+
+  it 'prints the message through Kernel' do
+    expect(BitBucket).to receive(:warn)
+    BitBucket.warn_deprecation method
+  end
+end

data/spec/bitbucket_rest_api/error/bad_events_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe BitBucket::Error::BadEvents do
+  context 'an event that does not exist is passed in BitBucket::Repos::Webhooks#create or BitBucket::Repos::Webhooks#edit' do
+    it 'should raise an error with a message' do
+      expect { raise described_class.new('bad:event') }.
+      to raise_error(BitBucket::Error::BadEvents, "The event: 'bad:event', does not exist :(")
+    end
+  end
+end

data/spec/bitbucket_rest_api/error/blank_value_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe BitBucket::Error::BlankValue do
+  context 'an event that does not exist is passed in BitBucket::Repos::Webhooks#create or BitBucket::Repos::Webhooks#edit' do
+    it 'should raise an error with a message' do
+      expect { raise described_class.new('required_key') }.
+      to raise_error(
+        BitBucket::Error::BlankValue,
+        "The value for: 'required_key', cannot be blank :("
+      )
+    end
+  end
+end

data/spec/bitbucket_rest_api/error/no_events_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe BitBucket::Error::NoEvents do
+  context 'no events are passed in BitBucket::Repos::Webhooks#create or BitBucket::Repos::Webhooks#edit' do
+    it 'should raise an error with a message' do
+      expect { raise described_class.new }.
+      to raise_error(BitBucket::Error::NoEvents, "At least one event is required, none given :(")
+    end
+  end
+end

data/spec/bitbucket_rest_api/invitations_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe BitBucket::Invitations do
+  subject { described_class.new }
+
+  describe '#invitations' do
+    before do
+      expect(subject).to receive(:request).with(
+        :post,
+        "/2.0/invitations/mock_username/mock_repo/mock_email_address",
+        { :permission => 'read' },
+        {}
+      )
+    end
+
+    it 'sends a POST request for an invitation belonging to the given repo' do
+      subject.invite('mock_username', 'mock_repo', 'mock_email_address', 'read')
+    end
+  end
+end
+

data/spec/bitbucket_rest_api/issues/comments_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+describe BitBucket::Issues::Comments do
+  let(:comments) { described_class.new }
+
+  describe '.list' do
+    before do
+      expect(comments).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/issues/mock_issue_id/comments/',
+        {},
+        {}
+      ).and_return(['comment1', 'comment2', 'comment3'])
+    end
+
+    context 'without a block' do
+      it 'should make a GET request for the given issue' do
+        comments.list('mock_username', 'mock_repo', 'mock_issue_id')
+      end
+    end
+
+    context 'with a block' do
+      it 'should make a GET request for the given issue' do
+        comments.list('mock_username', 'mock_repo', 'mock_issue_id') { |comment| comment }
+      end
+    end
+  end
+
+  describe '.get' do
+    before do
+      expect(comments).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/issues/comments/mock_comment_id',
+        {},
+        {}
+      )
+    end
+
+    it 'should make a GET request for the given comment' do
+      comments.get('mock_username', 'mock_repo', 'mock_comment_id')
+    end
+  end
+
+  describe '.create' do
+    before do
+      expect(comments).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_username/mock_repo/issues/mock_issue_id/comments/',
+        {'content' => 'mock_comment'},
+        {}
+      )
+    end
+
+    it 'should send a POST request for the given issue' do
+      comments.create('mock_username', 'mock_repo', 'mock_issue_id', 'content' => 'mock_comment')
+    end
+  end
+
+  describe '.edit' do
+    before do
+      expect(comments).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_username/mock_repo/issues/comments/mock_comment_id',
+        {'content' => 'new_mock_comment'},
+        {}
+      )
+    end
+
+    it 'should send a PUT request for the given comment' do
+      comments.edit('mock_username', 'mock_repo', 'mock_comment_id', 'content' => 'new_mock_comment')
+    end
+  end
+
+  describe '.delete' do
+    before do
+      expect(comments).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_username/mock_repo/issues/comments/mock_comment_id',
+        {},
+        {}
+      )
+    end
+
+    it 'should make a DELETE request for the given comment' do
+      comments.delete('mock_username', 'mock_repo', 'mock_comment_id')
+    end
+  end
+end
+

data/spec/bitbucket_rest_api/issues/components_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+describe BitBucket::Issues::Components do
+  subject { described_class.new }
+
+  describe '#list' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/issues/components',
+        {},
+        {}
+      ).and_return(['component1', 'component2', 'component3'])
+    end
+
+    context 'without a block' do
+      it 'makes a GET request for the components belonging to the given repo' do
+        subject.list('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'makes a GET request for the components belonging to the given repo' do
+        subject.list('mock_username', 'mock_repo') { |component| component }
+      end
+    end
+  end
+
+  describe '#get' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/issues/components/mock_component_id',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a GET request for the given component belonging to the given repo' do
+      subject.get('mock_username', 'mock_repo', 'mock_component_id')
+    end
+  end
+
+  describe '#create' do
+    before do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_username/mock_repo/issues/components',
+        { 'name' => 'mock_name' },
+        {}
+      )
+    end
+
+    it 'makes a POST request for a new component that will belong to the given repo' do
+      subject.create('mock_username', 'mock_repo', name: 'mock_name')
+    end
+  end
+
+  describe '#update' do
+    before do
+      expect(subject).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_username/mock_repo/issues/components/mock_component_id',
+        { 'name' => 'mock_name' },
+        {}
+      )
+    end
+
+    it 'makes a PUT request for the given component belonging to the given repo' do
+      subject.update('mock_username', 'mock_repo', 'mock_component_id', name: 'mock_name')
+    end
+  end
+
+  describe '#delete' do
+    before do
+      expect(subject).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_username/mock_repo/issues/components/mock_component_id',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a PUT request for the given component belonging to the given repo' do
+      subject.delete('mock_username', 'mock_repo', 'mock_component_id')
+    end
+  end
+end

data/spec/bitbucket_rest_api/issues/milestones_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+describe BitBucket::Issues::Milestones do
+  subject { described_class.new }
+
+  describe '#list' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/issues/milestones',
+        {},
+        {}
+      ).and_return(['milsetone1', 'milestone2', 'milestone3'])
+    end
+
+    context 'without a block' do
+      it 'makes a GET request for the milestones belonging to the given repo' do
+        subject.list('mock_username', 'mock_repo')
+      end
+    end
+
+    context 'with a block' do
+      it 'makes a GET request for the milestones belonging to the given repo' do
+        subject.list('mock_username', 'mock_repo') { |milestone| milestone }
+      end
+    end
+  end
+
+  describe '#get' do
+    before do
+      expect(subject).to receive(:request).with(
+        :get,
+        '/2.0/repositories/mock_username/mock_repo/issues/milestones/mock_milestone_id',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a GET request for the given milestone belonging to the given repo' do
+      subject.get('mock_username', 'mock_repo', 'mock_milestone_id')
+    end
+  end
+
+  describe '#create' do
+    before do
+      expect(subject).to receive(:request).with(
+        :post,
+        '/2.0/repositories/mock_username/mock_repo/issues/milestones',
+        { 'name' => 'mock_name' },
+        {}
+      )
+    end
+
+    it 'makes a POST request for a new milestone that will belong to the given repo' do
+      subject.create('mock_username', 'mock_repo', name: 'mock_name')
+    end
+  end
+
+  describe '#update' do
+    before do
+      expect(subject).to receive(:request).with(
+        :put,
+        '/2.0/repositories/mock_username/mock_repo/issues/milestones/mock_milestone_id',
+        { 'name' => 'mock_name' },
+        {}
+      )
+    end
+
+    it 'makes a PUT request for the given milestone belonging to the given repo' do
+      subject.update('mock_username', 'mock_repo', 'mock_milestone_id', name: 'mock_name')
+    end
+  end
+
+  describe '#delete' do
+    before do
+      expect(subject).to receive(:request).with(
+        :delete,
+        '/2.0/repositories/mock_username/mock_repo/issues/milestones/mock_milestone_id',
+        {},
+        {}
+      )
+    end
+
+    it 'makes a PUT request for the given milestone belonging to the given repo' do
+      subject.delete('mock_username', 'mock_repo', 'mock_milestone_id')
+    end
+  end
+end

data/spec/bitbucket_rest_api/issues_spec.rb

@@ -0,0 +1,90 @@
+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,
+        '/2.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,
+        '/2.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,
+        '/2.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,
+        '/2.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,
+        '/2.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