bitbuckets 0.2.0

data/spec/bitbucket_rest_api/repos_spec.rb

@@ -0,0 +1,158 @@
+# frozen_string_literal: true
+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(%w[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: %w[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(%w[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,19 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+describe BitBucket::Request::Jsonize do
+  let(:jsonize) { described_class.new(->(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,26 @@
+# frozen_string_literal: true
+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,88 @@
+# frozen_string_literal: true
+require 'spec_helper'
+require 'bitbucket_rest_api/request'
+
+describe BitBucket::Request do
+  let(:fake_api) do
+    Class.new do
+      include BitBucket::Request
+
+      def connection(*_args)
+        Faraday.new(url: 'https://api.bitbucket.org')
+      end
+
+      def new_access_token
+        '12345'
+      end
+    end
+  end
+
+  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
+      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 v#{Faraday::VERSION}"
+                })
+
+        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 v#{Faraday::VERSION}"
+                })
+
+        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 v#{Faraday::VERSION}"
+                })
+
+        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 v#{Faraday::VERSION}"
+                })
+        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 v#{Faraday::VERSION}"
+                })
+
+        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,13 @@
+# frozen_string_literal: true
+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,33 @@
+# frozen_string_literal: true
+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 = %w[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,42 @@
+# frozen_string_literal: true
+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,136 @@
+# frozen_string_literal: true
+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' => %w[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' => %w[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' => %w[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' => %w[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' => %w[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