zz_bitbucket_rest_api 0.1.8

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,
+        '/1.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,
+        '/1.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,
+        '/1.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,
+        '/1.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,
+        '/1.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/repositories',
+        {"pagelen" => 100},
+        {}
+      ).and_return(values: ['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,
+        '/1.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

data/spec/bitbucket_rest_api/request/jsonize_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe BitBucket::Request::Jsonize do
+  let(:jsonize) { described_class.new(lambda { |env| env }) }
+  before do
+    @env = {
+      body: {key1: 'val1'},
+      request_headers: {
+        'Content-Type' => 'application/json; charset=utf-8'
+      }
+    }
+  end
+
+  it "converts the body to json" do
+    expected = {body: "{\"key1\":\"val1\"}", request_headers: {"Content-Type"=>"application/json; charset=utf-8"}}
+    expect(jsonize.call(@env)).to eq expected
+  end
+end

data/spec/bitbucket_rest_api/request/oauth_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+require 'rack/test'
+
+describe BitBucket::Request::OAuth do
+  include Rack::Test::Methods
+
+
+  let(:app) { ->(env) { [200, env, "app"] } }
+
+  let (:middleware) { BitBucket::Request::OAuth.new(app) }
+
+  let(:request) { Rack::MockRequest.new(middleware) }
+
+  it "add url key to env hash with URI value" do
+    query_string = "key1=val1&key2=val2"
+    code, env = middleware.call Rack::MockRequest.env_for("/?#{query_string}", {method: :post})
+    expect(code).to eq 200
+    expect(env[:url].query).to eq query_string
+  end
+
+  it "creates a empty hash if query of URI is empty" do
+    code, env = middleware.call Rack::MockRequest.env_for("/", {method: :get})
+    expect(code).to eq 200
+    expect(middleware.query_params(env[:url])).to eq({})
+  end
+end
+

data/spec/bitbucket_rest_api/request_spec.rb

@@ -0,0 +1,81 @@
+require 'spec_helper'
+require 'bitbucket_rest_api/request'
+
+describe BitBucket::Request do
+  let(:fake_api) { (Class.new { include BitBucket::Request })}
+  let(:faraday_connection) { Faraday.new(:url => 'https://api.bitbucket.org') }
+
+  describe "request" do
+    it "raises an ArgumentError if an unsupported HTTP verb is used" do
+      expect { fake_api.new.request(:i_am_a_teapot, {}, '/') }.to raise_error(ArgumentError)
+    end
+
+    context "with a connection" do
+      before do
+        (fake_api).any_instance.stubs(:connection).returns(faraday_connection)
+        (fake_api).any_instance.stubs(:new_access_token).returns("12345")
+      end
+
+      it "supports get" do
+        stub_request(:get, "https://api.bitbucket.org/1.0/endpoint").
+         with(:headers => {
+          'Accept' => '*/*',
+          'Accept-Encoding' => 'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
+          'Authorization' => 'Bearer 12345',
+          'User-Agent' => 'Faraday v0.9.2'
+          })
+
+        fake_api.new.request(:get, '/1.0/endpoint', {}, {})
+      end
+
+      it "supports put" do
+        stub_request(:put, "https://api.bitbucket.org/1.0/endpoint").
+         with(:body => "{\"data\":{\"key\":\"value\"}}",
+          :headers => {
+            'Accept' => '*/*',
+            'Content-Type'=>'application/x-www-form-urlencoded',
+            'Authorization' => 'Bearer 12345',
+            'User-Agent' => 'Faraday v0.9.2'
+          })
+
+        fake_api.new.request(:put, '/1.0/endpoint', { 'data' => { 'key' => 'value'} }, {})
+      end
+
+      it "supports patch" do
+        stub_request(:patch, "https://api.bitbucket.org/1.0/endpoint").
+         with(:body => "{\"data\":{\"key\":\"value\"}}",
+          :headers => {
+            'Accept' => '*/*',
+            'Content-Type'=>'application/x-www-form-urlencoded',
+            'Authorization' => 'Bearer 12345',
+            'User-Agent' => 'Faraday v0.9.2'
+          })
+
+        fake_api.new.request(:patch, '/1.0/endpoint', { 'data' => { 'key' => 'value'} }, {})
+      end
+
+      it "supports delete" do
+        stub_request(:delete, "https://api.bitbucket.org/1.0/endpoint").
+         with(:headers => {
+          'Accept' => '*/*',
+          'Authorization' => 'Bearer 12345',
+          'User-Agent' => 'Faraday v0.9.2'
+          })
+        fake_api.new.request(:delete, '/1.0/endpoint', {}, {})
+      end
+
+      it "supports post" do
+        stub_request(:post, "https://api.bitbucket.org/1.0/endpoint").
+         with(:body => "{\"data\":{\"key\":\"value\"}}",
+          :headers => {
+            'Accept' => '*/*',
+            'Content-Type'=>'application/x-www-form-urlencoded',
+            'Authorization' => 'Bearer 12345',
+            'User-Agent' => 'Faraday v0.9.2'
+          })
+
+        fake_api.new.request(:post, '/1.0/endpoint', { 'data' => { 'key' => 'value'} }, {})
+      end
+    end
+  end
+end

data/spec/bitbucket_rest_api/response/jsonize_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe BitBucket::Response::Jsonize do
+  let(:jsonize) { described_class.new }
+  before do
+    @body = "{\"key1\":\"val1\"}"
+  end
+
+  it "parses the json and returns a hash" do
+    expect(jsonize.parse(@body)).to eq({"key1"=>"val1"})
+  end
+end

data/spec/bitbucket_rest_api/response/mashify_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe BitBucket::Response::Mashify do
+  # let(:mashify) { described_class.new }
+  describe 'parse' do
+    before do
+      @mashify = BitBucket::Response::Mashify.new
+      @string = "Fantastic week!"
+      @array = ['Monday', 'Tuesday']
+      @hash = {one: 'one', two: 'two', three: 'three'}
+      @array_with_hash = ['banana', 'apple', {:third => 'mango'}]
+    end
+    it 'parses a hash an returns a hashie mash' do
+      hashie_mash = @mashify.parse(@hash)
+      expect(hashie_mash.one).to eq("one")
+    end
+
+    it 'parses a hash that is within an array' do
+      array_hashie_mash = @mashify.parse(@array_with_hash)
+      expect(array_hashie_mash[2].third).to eq("mango")
+    end
+
+    it 'returns same object if the object does not contain a hash' do
+      string = @mashify.parse(@string)
+      array = @mashify.parse(@array)
+
+      expect(string).to eq(@string)
+      expect(array.length).to eq(2)
+      expect(array[0]).to eq("Monday")
+    end
+  end
+end

data/spec/bitbucket_rest_api/response/raise_error_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe BitBucket::Response::RaiseError do
+  describe '.on_complete' do
+    before do
+      @raise_error = BitBucket::Response::RaiseError.new
+    end
+
+    it 'raises a BadRequest error on 400 status code' do
+      expect{ @raise_error.on_complete({status: 400}) }.to raise_error BitBucket::Error::BadRequest
+    end
+
+    it 'raises an Unauthorized error on 401 status code' do
+      expect{ @raise_error.on_complete({status: 401}) }.to raise_error BitBucket::Error::Unauthorized
+    end
+
+    it 'raises a Forbidden error on 403 status code' do
+      expect{ @raise_error.on_complete({status: 403}) }.to raise_error BitBucket::Error::Forbidden
+    end
+
+    it 'raises a NotFound error on 404 status code' do
+      expect{ @raise_error.on_complete({status: 404}) }.to raise_error BitBucket::Error::NotFound
+    end
+
+    it 'raises an UnprocessableEntity error on 422 status code' do
+      expect{ @raise_error.on_complete({status: 422}) }.to raise_error BitBucket::Error::UnprocessableEntity
+    end
+
+    it 'raises an InternalServerError error on 500 status code' do
+      expect{ @raise_error.on_complete({status: 500}) }.to raise_error BitBucket::Error::InternalServerError
+    end
+
+    it 'raises a ServiceUnavailable error on 503 status code' do
+      expect{ @raise_error.on_complete({status: 503}) }.to raise_error BitBucket::Error::ServiceUnavailable
+    end
+
+    it 'raises a ServiceError when another status code' do
+      expect{ @raise_error.on_complete({status: 501}) }.to raise_error BitBucket::Error::ServiceError
+    end
+  end
+end

data/spec/bitbucket_rest_api/teams_spec.rb

@@ -0,0 +1,135 @@
+require 'spec_helper'
+
+describe BitBucket::Teams do
+  let(:team) { described_class.new }
+
+  describe '.list' do
+    before do
+      expect(team).to receive(:request).with(
+        :get,
+        '/2.0/teams/?role=member',
+        {},
+        {}
+      ).and_return({"values" => ['team1', 'team2', 'team3']})
+    end
+
+    context 'without a block' do
+      it 'sends a GET request for the teams of which the user is a member' do
+        team.list(:member)
+      end
+    end
+
+    context 'with a block' do
+      it 'sends a GET request for the teams of which the user is a member' do
+        team.list(:member) { |team| team }
+      end
+    end
+  end
+
+  describe '.profile' do
+    before do
+      expect(team).to receive(:request).with(
+        :get,
+        '/2.0/teams/team_name',
+        {},
+        {}
+      )
+    end
+
+    it 'sends a GET request for the profile for the team' do
+      team.profile('team_name')
+    end
+  end
+
+  describe '.members' do
+    before do
+      expect(team).to receive(:request).with(
+        :get,
+        '/2.0/teams/team_name/members',
+        {},
+        {}
+      ).and_return({"values" => ['member1', 'member2', 'member3']})
+    end
+
+    context "without a block" do
+      it 'sends a GET request for the members of the team' do
+        team.members('team_name')
+      end
+    end
+
+    context "with a block" do
+      it 'sends a GET request for the members of the team' do
+        team.members('team_name') { |member| member }
+      end
+    end
+  end
+
+  describe '.followers' do
+    before do
+      expect(team).to receive(:request).with(
+        :get,
+        '/2.0/teams/team_name/followers',
+        {},
+        {}
+      ).and_return({"values" => ['follower1', 'follower2', 'follower3']})
+    end
+
+    context "without a block" do
+      it 'sends a GET request for the followers of the team' do
+        team.followers('team_name')
+      end
+    end
+
+    context "with a block" do
+      it 'sends a GET request for the followers of the team' do
+        team.followers('team_name') { |follower| follower }
+      end
+    end
+  end
+
+  describe '.following' do
+    before do
+      expect(team).to receive(:request).with(
+        :get,
+        '/2.0/teams/team_name/following',
+        {},
+        {}
+      ).and_return({"values" => ['following1', 'following2', 'following3']})
+    end
+
+    context "without a block" do
+      it 'sends a GET request for accounts the team is following' do
+        team.following('team_name')
+      end
+    end
+
+    context "with a block" do
+      it 'sends a GET request for accounts the team is following' do
+        team.following('team_name') { |followee| followee }
+      end
+    end
+  end
+
+  describe '.repos' do
+    before do
+      expect(team).to receive(:request).with(
+        :get,
+        '/2.0/repositories/team_name',
+        {},
+        {}
+      ).and_return({"values" => ['repo1', 'repo2', 'repo3']})
+    end
+
+    context "without a block" do
+      it 'sends a GET request for the repos for the team' do
+        team.repos('team_name')
+      end
+    end
+
+    context "with a block" do
+      it 'sends a GET request for the repos for the team' do
+        team.repos('team_name') { |repo| repo }
+      end
+    end
+  end
+end