tinybucket 1.3.0 → 1.4.0

data/spec/lib/tinybucket/api/comments_api_spec.rb

@@ -1,132 +0,0 @@
-require 'spec_helper'
-
-RSpec.describe Tinybucket::Api::CommentsApi do
-  include ApiResponseMacros
-
-  let(:request_path) { nil }
-  let(:owner) { 'test_owner' }
-  let(:slug) { 'test_repo' }
-  let(:options) { {} }
-
-  let(:commit_hash) { '1' }
-  let(:commit) do
-    m = Tinybucket::Model::Commit.new({})
-    m.repo_owner = owner
-    m.repo_slug  = slug
-    m.hash = commit_hash
-    m
-  end
-
-  let(:pullreq_id) { '1' }
-  let(:pullreq) do
-    m = Tinybucket::Model::PullRequest.new({})
-    m.repo_owner = owner
-    m.repo_slug  = slug
-    m.id = pullreq_id
-    m
-  end
-
-  let(:commented_to) { nil }
-  let(:api) do
-    api = Tinybucket::Api::CommentsApi.new
-    api.repo_owner = owner
-    api.repo_slug = slug
-    api.commented_to = commented_to if commented_to.present?
-    api
-  end
-
-  it { expect(api).to be_a_kind_of(Tinybucket::Api::BaseApi) }
-
-  before { stub_apiresponse(:get, request_path) if request_path }
-
-  describe '#list' do
-    subject { api.list(options) }
-    let(:commented_to) { commit }
-
-    context 'when without owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without commented_to' do
-      let(:commented_to) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with unknown commented_to' do
-      let(:commented_to) { {} }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when commented_to is commit' do
-      let(:commented_to) { commit }
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{commit_hash}/comments"
-      end
-      it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-    end
-
-    context 'when commented_to is pullreq' do
-      let(:commented_to) { pullreq }
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/#{pullreq_id}/comments"
-      end
-      it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-    end
-  end
-
-  describe '#find' do
-    subject { api.find(comment_id, options) }
-    let(:commented_to) { commit }
-    let(:comment_id) { '1' }
-
-    context 'when without owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without commented_to' do
-      let(:commented_to) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with unknown commented_to' do
-      let(:commented_to) { {} }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when commented_to is commit' do
-      let(:commented_to) { commit }
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{commit_hash}" \
-          "/comments/#{comment_id}"
-      end
-      it 'return Comment model' do
-        expect(subject).to be_an_instance_of(Tinybucket::Model::Comment)
-        expect(subject.commented_to).to eq(commit)
-      end
-    end
-
-    context 'when commented_to is pull_request' do
-      let(:commented_to) { pullreq }
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/#{pullreq_id}" \
-          "/comments/#{comment_id}"
-      end
-      it 'return Comment model' do
-        expect(subject).to be_an_instance_of(Tinybucket::Model::Comment)
-        expect(subject.commented_to).to eq(pullreq)
-      end
-    end
-  end
-end

data/spec/lib/tinybucket/api/commits_api_spec.rb

@@ -1,152 +0,0 @@
-require 'spec_helper'
-
-RSpec.describe Tinybucket::Api::CommitsApi do
-  include ApiResponseMacros
-
-  let(:request_path) { nil }
-  let(:owner) { 'test_owner' }
-  let(:slug)  { 'test_repo' }
-  let(:options) { {} }
-
-  let(:api) do
-    api = Tinybucket::Api::CommitsApi.new
-    api.repo_owner = owner
-    api.repo_slug  = slug
-    api
-  end
-
-  it { expect(api).to be_a_kind_of(Tinybucket::Api::BaseApi) }
-
-  let(:request_method) { :get }
-  let(:stub_options) { nil }
-
-  before do
-    if request_path
-      opts = stub_options.present? ? stub_options : {}
-      stub_apiresponse(request_method, request_path, opts)
-    end
-  end
-
-  describe 'list' do
-    subject { api.list(options) }
-
-    context 'without owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'without slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'with owner and slug' do
-      let(:request_path) { "/repositories/#{owner}/#{slug}/commits" }
-      it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-    end
-  end
-
-  describe 'find' do
-    let(:revision) { '1' }
-    subject { api.find(revision, options) }
-
-    context 'without owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'without slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'with owner and slug' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{revision}"
-      end
-      it { expect(subject).to be_instance_of(Tinybucket::Model::Commit) }
-    end
-  end
-
-  describe 'approve' do
-    let(:revision) { '1' }
-    let(:request_method) { :post }
-
-    subject { api.approve(revision, options) }
-
-    context 'without owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'without slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when pull request is not yet approved' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{revision}/approve"
-      end
-
-      it { expect(subject).to be_truthy }
-    end
-
-    context 'when pull request is already approved.' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{revision}/approve"
-      end
-
-      let(:stub_options) do
-        {
-          status_code: 409,
-          message: JSON.generate(
-            error: { message: 'You already approved this pull request.' }
-          )
-        }
-      end
-      it { expect(subject).to be_truthy }
-    end
-  end
-
-  describe 'unapprove' do
-    let(:revision) { '1' }
-    let(:request_method) { :delete }
-    subject { api.unapprove(revision) }
-
-    context 'without owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'without slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when commit is not yet approved' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{revision}/approve"
-      end
-
-      let(:stub_options) do
-        {
-          status_code: 404,
-          message: JSON.generate(
-            error: { message: 'You haven\'t approve this pull request.' }
-          )
-        }
-      end
-      it { expect(subject).to be_truthy }
-    end
-
-    context 'when commit is already approved' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/commit/#{revision}/approve"
-      end
-      let(:stub_options) { { status_code: 204, message: '' } }
-
-      it { expect(subject).to be_truthy }
-    end
-  end
-end

data/spec/lib/tinybucket/api/diff_api_spec.rb

@@ -1,5 +0,0 @@
-require 'spec_helper'
-
-RSpec.describe Tinybucket::Api::DiffApi do
-  pending 'TODO: add specs'
-end

data/spec/lib/tinybucket/api/pull_requests_api_spec.rb

@@ -1,259 +0,0 @@
-require 'spec_helper'
-
-RSpec.describe Tinybucket::Api::PullRequestsApi do
-  include ApiResponseMacros
-
-  let(:api) do
-    api = Tinybucket::Api::PullRequestsApi.new
-    api.repo_owner = owner
-    api.repo_slug = slug
-    api
-  end
-
-  let(:owner) { 'test_owner' }
-  let(:slug) { 'test_repo' }
-  let(:request_path) { nil }
-  let(:request_method) { :get }
-  let(:stub_options) { nil }
-
-  it { expect(api).to be_a_kind_of(Tinybucket::Api::BaseApi) }
-
-  before do
-    if request_path
-      opts = stub_options.present? ? stub_options : {}
-      stub_apiresponse(request_method, request_path, opts)
-    end
-  end
-
-  describe 'list' do
-    let(:options) { {} }
-
-    subject { api.list(options) }
-
-    context 'without repo_owner and repo_slug' do
-      let(:owner) { nil }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'without repo_owner' do
-      let(:owner) { nil }
-      let(:slug) { 'test_repo' }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'without repo_slug' do
-      let(:owner) { 'test_owner' }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'with repo_owner and repo_slug' do
-      let(:request_path) { "/repositories/#{owner}/#{slug}/pullrequests" }
-      it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-    end
-
-    context 'when request with state' do
-      let(:options) { { state: state } }
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests?state=#{state}"
-      end
-
-      context 'when state is OPEN' do
-        let(:state) { 'open' }
-        it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-      end
-
-      context 'when state is DECLINED' do
-        let(:state) { 'declined' }
-        it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-      end
-
-      context 'when state is MERGED' do
-        let(:state) { 'merged' }
-        it { expect(subject).to be_an_instance_of(Tinybucket::Model::Page) }
-      end
-    end
-  end
-
-  describe 'find' do
-    let(:pr_id) { 1 }
-    subject { api.find(pr_id) }
-
-    context 'when without repo_owner and repo_slug' do
-      let(:owner) { nil }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with repo_owner and repo_slug' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/#{pr_id}"
-      end
-
-      it 'return pull request model' do
-        expect(subject).to be_an_instance_of(Tinybucket::Model::PullRequest)
-      end
-    end
-  end
-
-  describe 'commits' do
-    let(:pr_id) { 1 }
-    subject { api.commits(pr_id) }
-
-    context 'when without repo_owner and repo_slug' do
-      let(:owner) { nil }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with repo_owner and repo_slug' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/1/commits"
-      end
-      it 'return page model which contains commit models' do
-        expect(subject).to be_an_instance_of(Tinybucket::Model::Page)
-      end
-    end
-  end
-
-  describe 'approve' do
-    let(:pr_id) { 1 }
-    let(:request_method) { :post }
-    subject { api.approve(pr_id) }
-
-    context 'when without repo_owner and repo_slug' do
-      let(:owner) { nil }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with repo_owner and repo_slug' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/1/approve"
-      end
-      context 'when pull request is not yet approved.' do
-        it { expect(subject).to be_truthy }
-      end
-      context 'when pull request is already approved.' do
-        let(:stub_options) do
-          {
-            status_code: 409,
-            message: JSON.generate(
-              error: { message: 'You already approved this pull request.' }
-            )
-          }
-        end
-        it { expect(subject).to be_truthy }
-      end
-    end
-  end
-
-  describe 'unapprove' do
-    let(:pr_id) { 1 }
-    let(:request_method) { :delete }
-    subject { api.unapprove(pr_id) }
-
-    context 'when without repo_owner and repo_slug' do
-      let(:owner) { nil }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with repo_owner and repo_slug' do
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/1/approve"
-      end
-
-      context 'when pull request is approved' do
-        # In this case, bitbucket api respond '204 No content'.
-        let(:stub_options) { { status_code: 204, message: '' } }
-        it { expect(subject).to be_truthy }
-      end
-
-      context 'when pull request is not approved yet.' do
-        # In this case, bitbucket api respond '404 NotFound'.
-        let(:stub_options) do
-          {
-            status_code: 404,
-            message: JSON.generate(
-              error: { message: 'You haven\'t approve this pull request.' }
-            )
-          }
-        end
-        it { expect(subject).to be_truthy }
-      end
-    end
-  end
-
-  describe 'diff' do
-    let(:pr_id) { 1 }
-    subject { api.diff(pr_id) }
-
-    context 'when without repo_owner and repo_slug' do
-      let(:owner) { nil }
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_owner' do
-      let(:owner) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when without repo_slug' do
-      let(:slug) { nil }
-      it { expect { subject }.to raise_error(ArgumentError) }
-    end
-
-    context 'when with repo_owner and repo_slug' do
-      let(:stub_options) do
-        { content_type: 'plain/text' }
-      end
-      let(:request_path) do
-        "/repositories/#{owner}/#{slug}/pullrequests/1/diff"
-      end
-      it { expect(subject).to be_instance_of(String) }
-    end
-  end
-end