git_reflow 0.6.7 → 0.7.0

data/spec/fixtures/repositories/commit.json.erb

@@ -0,0 +1,53 @@
+{
+    "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e",
+    "commit": {
+        "url": "https://api.github.com/repos/<%= repo_owner %>/<%= repo_name %>/git/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+        "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e",
+        "author": {
+            "name": "Monalisa Octocat",
+            "email": "support@github.com",
+            "date": "<%= created_at || "2011-04-14T16:00:49Z" %>"
+        },
+        "committer": {
+            "name": "Monalisa Octocat",
+            "email": "support@github.com",
+            "date": "<%= created_at || "2011-04-14T16:00:49Z" %>"
+        },
+        "message": "Fix all the bugs",
+        "tree": {
+            "url": "https://api.github.com/repos/<%= repo_owner %>/<%= repo_name %>/tree/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+            "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e"
+        }
+    },
+    "author": {
+        "login": "<%= author %>",
+        "id": 1,
+        "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+        "url": "https://api.github.com/users/<%= author %>"
+    },
+    "committer": {
+        "login": "<%= author %>",
+        "id": 1,
+        "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+        "url": "https://api.github.com/users/<%= author %>"
+    },
+    "parents": [
+        {
+            "url": "https://api.github.com/repos/<%= repo_owner %>/<%= repo_name %>/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+            "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e"
+        }
+    ],
+    "stats": {
+        "additions": 104,
+        "deletions": 4,
+        "total": 108
+    },
+    "files": [
+        {
+            "filename": "file1.txt",
+            "additions": 10,
+            "deletions": 2,
+            "total": 12
+        }
+    ]
+}

data/spec/fixtures/repositories/commits.json.erb

@@ -0,0 +1,13 @@
+[
+  <% commits_json = [] %>
+  <% commits.each_with_index do |commit, index| %>
+    <% commits_json << Fixture.new('repositories/commit.json.erb',
+                         id:                  index,
+                         author:              commit[:author],
+                         repo_owner:          repo_owner,
+                         repo_name:           repo_name,
+                         created_at:          commit[:created_at] || Chronic.parse("1.minute.ago")
+                       ).to_s %>
+  <% end %>
+  <%= commits_json.join(", ") %>
+]

data/spec/git_reflow_spec.rb

@@ -14,7 +14,7 @@ describe GitReflow do
   let(:hostname)         { 'hostname.local' }
 
   let(:github_authorizations)  { Github::Client::Authorizations.new }
-  let(:existing_pull_requests) { JSON.parse(fixture('pull_requests/pull_requests.json').read).collect { |pull| Hashie::Mash.new(pull)} }
+  let(:existing_pull_requests) { Fixture.new('pull_requests/pull_requests.json').to_json_hashie }
   let(:existing_pull_request)  { GitReflow::GitServer::GitHub::PullRequest.new existing_pull_requests.first }
 
   before do
@@ -37,11 +37,13 @@ describe GitReflow do
     subject { GitReflow.status(base_branch) }
 
     before do
-      GitReflow.stub(:current_branch).and_return(feature_branch)
-      GitReflow.stub(:destination_branch).and_return(base_branch)
+      allow(GitReflow).to receive(:current_branch).and_return(feature_branch)
+      allow(GitReflow).to receive(:destination_branch).and_return(base_branch)
 
-      Github.stub(:new).and_return(github)
-      GitReflow.stub(:git_server).and_return(git_server)
+      allow(Github).to receive(:new).and_return(github)
+      allow(GitReflow).to receive(:git_server).and_return(git_server)
+      allow(git_server).to receive(:connection).and_return(github)
+      allow(git_server).to receive(:get_build_status).and_return(Struct.new(:state, :description, :target_url).new())
     end
 
     context 'with no existing pull request' do
@@ -51,10 +53,12 @@ describe GitReflow do
     end
 
     context 'with an existing pull request' do
-      before { git_server.stub(:find_open_pull_request).with({from: feature_branch, to: base_branch}).and_return(existing_pull_request) }
+      before do
+        git_server.stub(:find_open_pull_request).with({from: feature_branch, to: base_branch}).and_return(existing_pull_request)
+      end
 
       it 'displays a summary of the pull request and asks to open it in the browser' do
-        GitReflow.should_receive(:display_pull_request_summary).with(existing_pull_request)
+        existing_pull_request.should_receive(:display_pull_request_summary)
         GitReflow.should_receive(:ask_to_open_in_browser).with(existing_pull_request.html_url)
         subject
         $output.should include "Here's the status of your review:"
@@ -77,6 +81,7 @@ describe GitReflow do
     }
 
     let(:github) do
+      allow_any_instance_of(GitReflow::GitServer::GitHub::PullRequest).to receive(:build).and_return(Struct.new(:state, :description, :url).new)
       stub_github_with({
         :user         => user,
         :password     => password,
@@ -115,15 +120,15 @@ describe GitReflow do
     context "pull request exists" do
       before do
         GitReflow.stub(:push_current_branch)
-        github_error = Github::Error::UnprocessableEntity.new( eval(fixture('pull_requests/pull_request_exists_error.json').read) )
-        github.stub(:create_pull_request).with(inputs.except('state')).and_raise(github_error)
-        GitReflow.stub(:display_pull_request_summary)
+        github_error = Github::Error::UnprocessableEntity.new( eval(Fixture.new('pull_requests/pull_request_exists_error.json').to_s) )
+        github.should_receive(:find_open_pull_request).and_return(existing_pull_request)
+        existing_pull_request.stub(:display_pull_request_summary)
       end
 
       subject { GitReflow.review inputs }
 
       it "displays a pull request summary for the existing pull request" do
-        GitReflow.should_receive(:display_pull_request_summary)
+        existing_pull_request.should_receive(:display_pull_request_summary)
         subject
       end
 
@@ -138,6 +143,7 @@ describe GitReflow do
     let(:branch)                { 'new-feature' }
     let(:inputs)                { {} }
     let!(:github) do
+      allow_any_instance_of(GitReflow::GitServer::GitHub::PullRequest).to receive(:build).and_return(Struct.new(:state, :description, :url).new)
       stub_github_with({
         :user         => user,
         :password     => password,
@@ -173,16 +179,17 @@ describe GitReflow do
     context "and pull request exists for the feature branch to the destination branch" do
       before do
         github.stub(:get_build_status).and_return(build_status)
-        github.stub(:has_pull_request_comments?).and_return(true)
-        github.stub(:comment_authors_for_pull_request).and_return(['codenamev'])
+        github.should_receive(:find_open_pull_request).and_return(existing_pull_request)
+        existing_pull_request.stub(:has_comments?).and_return(true)
+        github.stub(:reviewers).and_return(['codenamev'])
       end
 
       context 'and build status is not "success"' do
         let(:build_status) { Hashie::Mash.new({ state: 'failure', description: 'Build resulted in failed test(s)' }) }
 
         before do
-          # just stubbing these in a locked state as the test is specific to this scenario
-          GitReflow.stub(:has_pull_request_comments?).and_return(true)
+          existing_pull_request.stub(:build).and_return(build_status)
+          existing_pull_request.stub(:has_comments?).and_return(true)
         end
 
         it "halts delivery and notifies user of a failed build" do
@@ -196,8 +203,8 @@ describe GitReflow do
 
         before do
           # stubbing unrelated results so we can just test that it made it insdide the conditional block
-          GitReflow.stub(:has_pull_request_comments?).and_return(true)
-          GitReflow.stub(:comment_authors_for_pull_request).and_return([])
+          existing_pull_request.stub(:has_comments?).and_return(true)
+          existing_pull_request.stub(:reviewers).and_return([])
           GitReflow.stub(:update_destination).and_return(true)
           GitReflow.stub(:merge_feature_branch).and_return(true)
           GitReflow.stub(:append_to_squashed_commit_message).and_return(true)
@@ -213,14 +220,14 @@ describe GitReflow do
 
         context 'and has comments' do
           before do
-            GitReflow.stub(:has_pull_request_comments?).and_return(true)
+            existing_pull_request.stub(:has_comments?).and_return(true)
           end
 
           context 'but there is a LGTM' do
             let(:lgtm_comment_authors) { ['nhance'] }
             before do
-              github.stub(:approvals).and_return(lgtm_comment_authors)
-              github.stub(:reviewers_pending_response).and_return([])
+              existing_pull_request.stub(:approvals).and_return(lgtm_comment_authors)
+              existing_pull_request.stub(:reviewers_pending_response).and_return([])
             end
 
             it "includes the pull request body in the commit message" do
@@ -236,7 +243,7 @@ describe GitReflow do
                 existing_pull_request.description = ''
                 github.stub(:find_open_pull_request).and_return(existing_pull_request)
                 GitReflow.stub(:get_first_commit_message).and_return(first_commit_message)
-                github.stub(:comment_authors_for_pull_request).and_return(lgtm_comment_authors)
+                existing_pull_request.stub(:reviewers).and_return(lgtm_comment_authors)
               end
 
               it "includes the first commit message for the new branch in the commit message of the merge" do
@@ -366,7 +373,7 @@ describe GitReflow do
 
           context 'but there are still unaddressed comments' do
             let(:open_comment_authors) { ['nhance', 'codenamev'] }
-            before { github.stub(:reviewers_pending_response).and_return(open_comment_authors) }
+            before { existing_pull_request.stub(:reviewers_pending_response).and_return(open_comment_authors) }
             it "notifies the user to get their code reviewed" do
               expect { subject }.to have_said "You still need a LGTM from: #{open_comment_authors.join(', ')}", :deliver_halted
             end
@@ -375,9 +382,9 @@ describe GitReflow do
 
         context 'but has no comments' do
           before do
-            github.stub(:has_pull_request_comments?).and_return(false)
-            github.stub(:approvals).and_return([])
-            github.stub(:reviewers_pending_response).and_return([])
+            existing_pull_request.stub(:has_comments?).and_return(false)
+            existing_pull_request.stub(:approvals).and_return([])
+            existing_pull_request.stub(:reviewers_pending_response).and_return([])
           end
 
           it "notifies the user to get their code reviewed" do

data/spec/lib/git_reflow/config_spec.rb

@@ -6,8 +6,24 @@ describe GitReflow::Config do
     it      { expect{ subject }.to have_run_command_silently 'git config --get chucknorris.roundhouse' }
 
     context "and getting all values" do
-      subject { GitReflow::Config.get('chucknorris.roundhouse', all: true) }
-      it      { expect{ subject }.to have_run_command_silently 'git config --get-all chucknorris.roundhouse' }
+      subject { GitReflow::Config.get('chucknorris.roundhouse-kick', all: true) }
+      it      { expect{ subject }.to have_run_command_silently 'git config --get-all chucknorris.roundhouse-kick' }
+
+      context "and checking locally only" do
+        subject { GitReflow::Config.get('chucknorris.jump', local: true) }
+        it      { expect{ subject }.to have_run_command_silently 'git config --local --get chucknorris.jump' }
+      end
+    end
+
+    context "and checking for updates" do
+      before  { GitReflow::Config.get('checknorris.roundhouse') }
+      subject { GitReflow::Config.get('chucknorris.roundhouse') }
+      it      { expect{ subject }.to_not have_run_command_silently 'git config --get chucknorris.roundhouse-kick' }
+    end
+
+    context "and checking locally only" do
+      subject { GitReflow::Config.get('chucknorris.smash', local: true) }
+      it      { expect{ subject }.to have_run_command_silently 'git config --local --get chucknorris.smash' }
     end
   end
 
@@ -23,20 +39,20 @@ describe GitReflow::Config do
 
   describe ".unset(key)" do
     subject { GitReflow::Config.unset('chucknorris.roundhouse') }
-    it      { expect{ subject }.to have_run_command_silently 'git config --global --unset chucknorris.roundhouse ' }
+    it      { expect{ subject }.to have_run_command_silently 'git config --global --unset-all chucknorris.roundhouse ' }
 
     context "for multi-value keys" do
       subject { GitReflow::Config.unset('chucknorris.roundhouse', value: 'to the face') }
-      it      { expect{ subject }.to have_run_command_silently 'git config --global --unset chucknorris.roundhouse "to the face"' }
+      it      { expect{ subject }.to have_run_command_silently 'git config --global --unset-all chucknorris.roundhouse "to the face"' }
     end
 
     context "for current project only" do
       subject { GitReflow::Config.unset('chucknorris.roundhouse', local: true) }
-      it      { expect{ subject }.to have_run_command_silently 'git config --unset chucknorris.roundhouse ' }
+      it      { expect{ subject }.to have_run_command_silently 'git config --unset-all chucknorris.roundhouse ' }
 
       context "for multi-value keys" do
         subject { GitReflow::Config.unset('chucknorris.roundhouse', value: 'to the face', local: true) }
-        it      { expect{ subject }.to have_run_command_silently 'git config --unset chucknorris.roundhouse "to the face"' }
+        it      { expect{ subject }.to have_run_command_silently 'git config --unset-all chucknorris.roundhouse "to the face"' }
       end
     end
   end

data/spec/lib/git_server/bit_bucket_spec.rb

@@ -25,7 +25,7 @@ describe GitReflow::GitServer::BitBucket do
     subject { GitReflow::GitServer::BitBucket.new({}) }
 
     it 'sets the reflow git server provider to BitBucket in the git config' do
-      GitReflow::Config.should_receive(:set).once.with('reflow.git-server', 'BitBucket')
+      GitReflow::Config.should_receive(:set).once.with('reflow.git-server', 'BitBucket', local: false)
       subject
     end
 
@@ -47,10 +47,11 @@ describe GitReflow::GitServer::BitBucket do
 
     context 'already authenticated' do
       it "notifies the user of successful setup" do
-        GitReflow::Config.should_receive(:set).with('reflow.git-server', 'BitBucket')
+        allow(GitReflow::Config).to receive(:set).with('reflow.git-server', 'BitBucket', local: false)
         allow(GitReflow::Config).to receive(:get).with('remote.origin.url').and_return(remote_url)
-        allow(GitReflow::Config).to receive(:get).with('bitbucket.user').and_return(user)
-        allow(GitReflow::Config).to receive(:get).with('bitbucket.api-key', reload: true).and_return(api_key)
+        allow(GitReflow::Config).to receive(:get).with('bitbucket.user', local: false).and_return(user)
+        allow(GitReflow::Config).to receive(:get).with('bitbucket.api-key', reload: true, local: false).and_return(api_key)
+        allow(GitReflow::Config).to receive(:get).with('reflow.local-projects', all: true).and_return('')
         expect { subject }.to have_output "\nYour BitBucket account was already setup with:"
         expect { subject }.to have_output "\tUser Name: #{user}"
       end
@@ -77,34 +78,4 @@ describe GitReflow::GitServer::BitBucket do
     end
   end
 
-  xdescribe '#create_pull_request(options)' do
-    let(:title)          { 'Fresh title' }
-    let(:body)           { 'Funky body' }
-    let(:current_branch) { 'new-feature' }
-
-    it 'creates a pull request using the remote user and repo' do
-    end
-  end
-
-  xdescribe '#find_pull_request(from, to)' do
-  end
-
-  xdescribe '#pull_request_comments(pull_request)' do
-  end
-
-  xdescribe '#has_pull_request_comments?(pull_request)' do
-  end
-
-  xdescribe '#get_build_status(sha)' do
-  end
-
-  xdescribe '#find_authors_of_open_pull_request_comments(pull_request)' do
-  end
-
-  xdescribe '#comment_authors_for_pull_request(pull_request, options = {})' do
-  end
-
-  xdescribe '#get_commited_time(commit_sha)' do
-  end
-
 end

data/spec/lib/git_server/git_hub/pull_request_spec.rb

@@ -0,0 +1,319 @@
+require 'spec_helper'
+
+describe GitReflow::GitServer::GitHub::PullRequest do
+  let(:user)                    { 'reenhanced' }
+  let(:password)                { 'shazam' }
+  let(:repo)                    { 'repo' }
+  let(:oauth_token_hash)        { Hashie::Mash.new({ token: 'a1b2c3d4e5f6g7h8i9j0'}) }
+  let(:hostname)                { 'hostname.local' }
+  let(:github_site)             { 'https://github.com' }
+  let(:github_api_endpoint)     { 'https://api.github.com' }
+  let(:enterprise_site)         { 'https://github.gittyup.com' }
+  let(:enterprise_api)          { 'https://github.gittyup.com/api/v3' }
+  let(:github)                  { stub_github_with }
+  let!(:github_api)             { github.connection }
+  let(:existing_pull_request)   { Fixture.new('pull_requests/external_pull_request.json').to_json_hashie }
+  let(:existing_pull_requests)  { Fixture.new('pull_requests/pull_requests.json').to_json_hashie }
+  let(:existing_pull_commits)   { Fixture.new('pull_requests/commits.json').to_json_hashie }
+  let(:comment_author)          { 'octocat' }
+  let(:existing_pull_comments)  {
+    Fixture.new('pull_requests/comments.json.erb',
+                repo_owner: user,
+                repo_name: repo,
+                comments: [{author: comment_author}],
+                pull_request_number: existing_pull_request.number).to_json_hashie }
+  let(:existing_issue_comments) {
+    Fixture.new('issues/comments.json.erb',
+                repo_owner: user,
+                repo_name: repo,
+                comments: [{author: comment_author}],
+                pull_request_number: existing_pull_request.number).to_json_hashie }
+
+  subject { GitReflow::GitServer::GitHub::PullRequest.new(existing_pull_request) }
+
+
+  before do
+    stub_command_line_inputs({
+      "Please enter your GitHub username: "                                                 => user,
+      "Please enter your GitHub password (we do NOT store this): "                          => password,
+      "Please enter your Enterprise site URL (e.g. https://github.company.com):"            => enterprise_site,
+      "Please enter your Enterprise API endpoint (e.g. https://github.company.com/api/v3):" => enterprise_api
+    })
+
+    github.class.stub(:remote_user).and_return(user)
+    github.class.stub(:remote_repo_name).and_return(repo)
+  end
+
+  describe '#initialize(options)' do
+    specify { expect(subject.number).to eql(existing_pull_request.number) }
+    specify { expect(subject.description).to eql(existing_pull_request.body) }
+    specify { expect(subject.html_url).to eql(existing_pull_request.html_url) }
+    specify { expect(subject.feature_branch_name).to eql(existing_pull_request.head.label) }
+    specify { expect(subject.base_branch_name).to eql(existing_pull_request.base.label) }
+    specify { expect(subject.build_status).to eql('success') }
+    specify { expect(subject.source_object).to eql(existing_pull_request) }
+  end
+
+  describe '#commit_author' do
+    before do
+      stub_request(:get, %r{#{GitReflow.git_server.class.api_endpoint}/repos/#{user}/#{repo}/pulls/#{existing_pull_request.number}/commits}).
+        with(query: {"access_token" => "a1b2c3d4e5f6g7h8i9j0"}).
+        to_return(:body => Fixture.new("pull_requests/commits.json").to_s, status: 201, headers: {content_type: "application/json; charset=utf-8"})
+    end
+    specify { expect(subject.commit_author).to eql("#{existing_pull_commits.first.commit.author.name} <#{existing_pull_commits.first.commit.author.email}>") }
+  end
+
+  describe '#comments' do
+    before do
+      FakeGitHub.new(
+        repo_owner: user,
+        repo_name: repo,
+        pull_request: {
+          number: existing_pull_request.number,
+          comments: [{author: comment_author}]
+        },
+        issue: {
+          number: existing_pull_request.number,
+          comments: [{author: comment_author}]
+        })
+    end
+
+    specify { expect(subject.comments).to eql(existing_pull_comments.to_a + existing_issue_comments.to_a) }
+  end
+
+  describe '#reviewers' do
+    before do
+      allow(existing_pull_request.user).to receive(:login).and_return('ringo')
+
+      FakeGitHub.new(
+        repo_owner: user,
+        repo_name: repo,
+        pull_request: {
+          number: existing_pull_request.number,
+          owner: existing_pull_request.user.login,
+          comments: [{author: 'tito'}, {author: 'bobby'}, {author: 'ringo'}]
+        },
+        issue: {
+          number: existing_pull_request.number,
+          comments: [{author: 'ringo'}, {author: 'randy'}]
+        })
+    end
+
+    specify { expect(subject.reviewers).to eql(['tito', 'bobby', 'randy']) }
+  end
+
+  describe '#approvals' do
+    context "no comments" do
+      before do
+        FakeGitHub.new(
+          repo_owner: user,
+          repo_name: repo,
+          pull_request: {
+            number: existing_pull_request.number,
+            owner: existing_pull_request.head.user.login,
+            comments: []
+          })
+      end
+
+      specify { expect(subject.approvals).to eq([]) }
+    end
+
+    context "single reviewer without approval" do
+      before do
+        FakeGitHub.new(
+          repo_owner: user,
+          repo_name: repo,
+          pull_request: {
+            number: existing_pull_request.number,
+            owner: existing_pull_request.head.user.login,
+            comments: [{author: 'tito', body: 'This is some funky stuff'}]
+          })
+      end
+
+      specify { expect(subject.approvals).to eq([]) }
+    end
+
+    context "single reviewer with approval" do
+      before do
+        FakeGitHub.new(
+          repo_owner:   user,
+          repo_name:    repo,
+          pull_request: {
+            number:   existing_pull_request.number,
+            owner:    existing_pull_request.head.user.login,
+            comments: [{author: 'tito', body: 'LGTM'}]
+          })
+      end
+
+      specify { expect(subject.approvals).to eq(['tito']) }
+
+      context "but a new commit has been introduced" do
+        before do
+          FakeGitHub.new(
+            repo_owner:   user,
+            repo_name:    repo,
+            commits: [
+              {
+                author: user,
+                pull_request_number: existing_pull_request.number,
+                created_at: Chronic.parse("1 second ago")
+              }
+            ],
+            pull_request: {
+              number:   existing_pull_request.number,
+              owner:    existing_pull_request.head.user.login,
+              comments: [{author: 'tito', body: 'LGTM', created_at: Chronic.parse('1 minute ago')}]
+            })
+        end
+
+        specify { expect(subject.approvals).to eq([]) }
+      end
+    end
+
+    context "multiple reviewers with only one approval" do
+      before do
+        FakeGitHub.new(
+          repo_owner:   user,
+          repo_name:    repo,
+          pull_request: {
+            number:   existing_pull_request.number,
+            owner:    existing_pull_request.head.user.login,
+            comments: [{author: 'tito', body: 'LGTM'}, {author: 'ringo', body: 'Needs more cowbell.'}]
+          })
+      end
+
+      specify { expect(subject.approvals).to eq(['tito']) }
+    end
+
+    context "multiple reviewers with all approvals" do
+      before do
+        FakeGitHub.new(
+          repo_owner:   user,
+          repo_name:    repo,
+          pull_request: {
+            number:   existing_pull_request.number,
+            owner:    existing_pull_request.head.user.login,
+            comments: [{author: 'tito', body: 'lgtm'}, {author: 'ringo', body: ':+1:'}]
+          })
+      end
+
+      specify { expect(subject.approvals).to eq(['tito', 'ringo']) }
+
+      context "but a new commit has been introduced" do
+        before do
+          FakeGitHub.new(
+            repo_owner:   user,
+            repo_name:    repo,
+            commits: [
+              {
+                author:              user,
+                pull_request_number: existing_pull_request.number,
+                created_at:          Chronic.parse("1 second ago")
+              }
+            ],
+            pull_request: {
+              number:   existing_pull_request.number,
+              owner:    existing_pull_request.head.user.login,
+              comments: [{author: 'tito', body: 'lgtm', created_at: Chronic.parse('1 minute ago')}, {author: 'ringo', body: ':+1:', created_at: Chronic.parse('1 minute ago')}]
+            })
+        end
+
+        specify { expect(subject.approvals).to eq([]) }
+      end
+    end
+
+  end
+
+  describe '#last_comment' do
+      before do
+        FakeGitHub.new(
+          repo_owner:   user,
+          repo_name:    repo,
+          pull_request: {
+            number:   existing_pull_request.number,
+            owner:    existing_pull_request.head.user.login,
+            comments: [{author: 'tito', body: 'lgtm'}, {author: 'ringo', body: 'Cha cha cha'}]
+          })
+      end
+
+      specify { expect(subject.last_comment).to eq('"Cha cha cha"') }
+  end
+
+  describe '#build' do
+    let(:build) { Fixture.new('repositories/statuses.json').to_json_hashie.first }
+
+    context "with an existing build" do
+      specify { expect(subject.build.state).to eq(build.state) }
+      specify { expect(subject.build.description).to eq(build.description) }
+      specify { expect(subject.build.url).to eq(build.target_url) }
+    end
+
+    context "no build found" do
+      before  { allow(GitReflow.git_server).to receive(:get_build_status).and_return(nil) }
+      specify { expect(subject.build.state).to eq(nil) }
+      specify { expect(subject.build.description).to eq(nil) }
+      specify { expect(subject.build.url).to eq(nil) }
+    end
+  end
+
+
+  describe '.create(options)' do
+    let(:title) { 'Bone Saw is ready!' }
+    let(:body)  { 'Snap into a Slim Jim!' }
+    let(:base_branch_name) { 'base-branch' }
+    let(:pull_request_response) do
+      Fixture.new('pull_requests/pull_request.json.erb',
+                  number:      2,
+                  title:       title,
+                  body:        body,
+                  base_branch: base_branch_name,
+                  repo_owner:  user,
+                  repo_name:   repo)
+    end
+
+    subject { GitReflow::GitServer::GitHub::PullRequest.create(title: title, body: body, base: base_branch_name) }
+
+    before do
+      stub_request(:post, %r{/repos/#{user}/#{repo}/pulls}).
+        to_return(body: pull_request_response.to_s, status: 201, headers: {content_type: "application/json; charset=utf-8"})
+    end
+
+    specify { expect(subject.class.to_s).to eql('GitReflow::GitServer::GitHub::PullRequest') }
+    specify { expect(subject.title).to eql(title) }
+    specify { expect(subject.description).to eql(body) }
+    specify { expect(subject.base_branch_name).to eql(base_branch_name) }
+  end
+
+  describe '.find_open(options)' do
+    let(:feature_branch) { 'new-feature' }
+    let(:base_branch)    { 'base-branch' }
+
+    subject { GitReflow::GitServer::GitHub::PullRequest.find_open(from: feature_branch, to: base_branch) }
+
+    before do
+      allow(GitReflow.git_server.class).to receive(:current_branch).and_return(feature_branch)
+      FakeGitHub.new(
+        repo_owner:   user,
+        repo_name:    repo,
+        pull_request: {
+          number:   existing_pull_request.number,
+          owner:    existing_pull_request.head.user.login,
+          base_branch: base_branch,
+          feature_branch: feature_branch
+        })
+    end
+
+    specify { expect(subject.class.to_s).to eql('GitReflow::GitServer::GitHub::PullRequest') }
+    specify { expect(subject.number).to eql(existing_pull_request.number) }
+
+    context "without any options" do
+      let(:base_branch) { 'master' }
+      subject           { GitReflow::GitServer::GitHub::PullRequest.find_open() }
+      it "defaults to the current branch as the feature branch and 'master' as the base branch" do
+        expect(subject.class.to_s).to eql('GitReflow::GitServer::GitHub::PullRequest')
+        expect(subject.number).to eql(existing_pull_request.number)
+      end
+    end
+  end
+
+end