git_reflow 0.8.9 → 0.9.3

data/spec/lib/git_reflow/git_server/bit_bucket_spec.rb

@@ -1,81 +1,81 @@
-require 'spec_helper'
-
-describe GitReflow::GitServer::BitBucket do
-  let(:user)         { 'reenhanced' }
-  let(:password)     { 'shazam' }
-  let(:repo)         { 'repo' }
-  let(:api_key)      { 'a1b2c3d4e5f6g7h8i9j0' }
-  let(:hostname)     { 'hostname.local' }
-  let(:api_endpoint) { 'https://bitbucket.org/api/1.0' }
-  let(:site)         { 'https://bitbucket.org' }
-  let(:remote_url)   { "git@bitbucket.org:#{user}/#{repo}.git" }
-
-  before do
-    allow_any_instance_of(HighLine).to receive(:ask) do |terminal, question|
-      values = {
-        "Please enter your BitBucket username: " => user
-      }
-     return_value = values[question]
-     question = ""
-     return_value
-    end
-  end
-
-  describe '#initialize(options)' do
-    subject { GitReflow::GitServer::BitBucket.new({}) }
-
-    it 'sets the reflow git server provider to BitBucket in the git config' do
-      expect(GitReflow::Config).to receive(:set).once.with('reflow.git-server', 'BitBucket', local: false)
-      subject
-    end
-
-    context 'storing git config settings only for this project' do
-      subject { GitReflow::GitServer::BitBucket.new(project_only: true) }
-
-      it 'sets the enterprise site and api as the site and api endpoints for the BitBucket provider in the git config' do
-        expect(GitReflow::Config).to receive(:set).once.with('reflow.git-server', 'BitBucket', local: true)
-        subject
-      end
-    end
-
-  end
-
-  describe '#authenticate' do
-    let(:bitbucket)      { GitReflow::GitServer::BitBucket.new( { }) }
-    let!(:bitbucket_api) { BitBucket.new }
-    subject              { bitbucket.authenticate }
-
-    context 'already authenticated' do
-      it "notifies the user of successful setup" do
-        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', 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_said "\nYour BitBucket account was already setup with:"
-        expect { subject }.to have_said "\tUser Name: #{user}"
-      end
-    end
-
-    context 'not yet authenticated' do
-      context 'with valid BitBucket credentials' do
-        before do
-          allow(GitReflow::Config).to receive(:get).and_return('')
-          allow(GitReflow::Config).to receive(:set)
-          allow(GitReflow::Config).to receive(:set).with('bitbucket.api-key', reload: true).and_return(api_key)
-          allow(GitReflow::Config).to receive(:get).with('bitbucket.api-key', reload: true).and_return('')
-          allow(GitReflow::Config).to receive(:get).with('remote.origin.url').and_return(remote_url)
-          allow(GitReflow::Config).to receive(:get).with('reflow.local-projects').and_return('')
-          allow(bitbucket).to receive(:connection).and_return double(repos: double(all: []))
-        end
-
-        it "prompts me to setup an API key" do
-          expect { subject }.to have_said "\nIn order to connect your BitBucket account,"
-          expect { subject }.to have_said "you'll need to generate an API key for your team"
-          expect { subject }.to have_said "Visit https://bitbucket.org/account/user/reenhanced/api-key/, to generate it\n"
-        end
-      end
-    end
-  end
-
-end
+require 'spec_helper'
+
+describe GitReflow::GitServer::BitBucket do
+  let(:user)         { 'reenhanced' }
+  let(:password)     { 'shazam' }
+  let(:repo)         { 'repo' }
+  let(:api_key)      { 'a1b2c3d4e5f6g7h8i9j0' }
+  let(:hostname)     { 'hostname.local' }
+  let(:api_endpoint) { 'https://bitbucket.org/api/1.0' }
+  let(:site)         { 'https://bitbucket.org' }
+  let(:remote_url)   { "git@bitbucket.org:#{user}/#{repo}.git" }
+
+  before do
+    allow_any_instance_of(HighLine).to receive(:ask) do |terminal, question|
+      values = {
+        "Please enter your BitBucket username: " => user
+      }
+     return_value = values[question]
+     question = ""
+     return_value
+    end
+  end
+
+  describe '#initialize(options)' do
+    subject { GitReflow::GitServer::BitBucket.new({}) }
+
+    it 'sets the reflow git server provider to BitBucket in the git config' do
+      expect(GitReflow::Config).to receive(:set).once.with('reflow.git-server', 'BitBucket', local: false)
+      subject
+    end
+
+    context 'storing git config settings only for this project' do
+      subject { GitReflow::GitServer::BitBucket.new(project_only: true) }
+
+      it 'sets the enterprise site and api as the site and api endpoints for the BitBucket provider in the git config' do
+        expect(GitReflow::Config).to receive(:set).once.with('reflow.git-server', 'BitBucket', local: true)
+        subject
+      end
+    end
+
+  end
+
+  describe '#authenticate' do
+    let(:bitbucket)      { GitReflow::GitServer::BitBucket.new( { }) }
+    let!(:bitbucket_api) { BitBucket.new }
+    subject              { bitbucket.authenticate }
+
+    context 'already authenticated' do
+      it "notifies the user of successful setup" do
+        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', 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_said "\nYour BitBucket account was already setup with:"
+        expect { subject }.to have_said "\tUser Name: #{user}"
+      end
+    end
+
+    context 'not yet authenticated' do
+      context 'with valid BitBucket credentials' do
+        before do
+          allow(GitReflow::Config).to receive(:get).and_return('')
+          allow(GitReflow::Config).to receive(:set)
+          allow(GitReflow::Config).to receive(:set).with('bitbucket.api-key', reload: true).and_return(api_key)
+          allow(GitReflow::Config).to receive(:get).with('bitbucket.api-key', reload: true).and_return('')
+          allow(GitReflow::Config).to receive(:get).with('remote.origin.url').and_return(remote_url)
+          allow(GitReflow::Config).to receive(:get).with('reflow.local-projects').and_return('')
+          allow(bitbucket).to receive(:connection).and_return double(repos: double(all: []))
+        end
+
+        it "prompts me to setup an API key" do
+          expect { subject }.to have_said "\nIn order to connect your BitBucket account,"
+          expect { subject }.to have_said "you'll need to generate an API key for your team"
+          expect { subject }.to have_said "Visit https://bitbucket.org/account/user/reenhanced/api-key/, to generate it\n"
+        end
+      end
+    end
+  end
+
+end

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

@@ -86,21 +86,21 @@ describe GitReflow::GitServer::GitHub::PullRequest do
           pull_request: {
             number: existing_pull_request.number,
             comments: [{author: comment_author}],
-            reviews: []
+            reviews: [{author: existing_pull_request.user.login}]
           },
           issue: {
             number: existing_pull_request.number,
-            comments: [{author: comment_author}]
+            comments: [{author: comment_author}, {author: existing_pull_request.user.login}]
           }
         )
       end
-      specify { expect(subject).to eql(existing_pull_comments.to_a + existing_issue_comments.to_a) }
+      specify { expect(subject).to eql(existing_pull_comments.to_a + existing_issue_comments.to_a - [existing_pull_request.user.login]) }
     end
 
     context "Testing Nil Comments" do
       before do
         stub_request(:get, "https://api.github.com/repos/reenhanced/repo/pulls/2/comments?access_token=a1b2c3d4e5f6g7h8i9j0").
-         with(:headers => {'Accept'=>'application/vnd.github.v3+json,application/vnd.github.beta+json;q=0.5,application/json;q=0.1', 'Accept-Charset'=>'utf-8', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'Authorization'=>'token a1b2c3d4e5f6g7h8i9j0', 'User-Agent'=>'Github API Ruby Gem 0.15.0'}).
+         with(:headers => {'Accept'=>'application/vnd.github.v3+json,application/vnd.github.beta+json;q=0.5,application/json;q=0.1', 'Accept-Charset'=>'utf-8', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'Authorization'=>'token a1b2c3d4e5f6g7h8i9j0', 'User-Agent'=>'Github API Ruby Gem 0.18.2'}).
          to_return(:status => 200, :body => "", :headers => {})
 
         FakeGitHub.new(
@@ -133,7 +133,7 @@ describe GitReflow::GitServer::GitHub::PullRequest do
           number: existing_pull_request.number,
           owner: existing_pull_request.user.login,
           comments: [{author: 'tito'}, {author: 'bobby'}, {author: 'ringo'}],
-          reviews:  [{author: 'nature-boy'}]
+          reviews:  [{author: 'ringo'}, {author: 'nature-boy'}]
         },
         issue: {
           number: existing_pull_request.number,
@@ -484,16 +484,16 @@ describe GitReflow::GitServer::GitHub::PullRequest do
       allow(GitReflow.git_server).to receive(:connection).and_return(github_api)
       allow(GitReflow.git_server).to receive(:get_build_status).and_return(Struct.new(:state, :description, :target_url).new())
       allow_any_instance_of(GitReflow::GitServer::PullRequest).to receive(:commit_message_for_merge).and_return('Bingo')
-      allow_any_instance_of(GitReflow).to receive(:append_to_squashed_commit_message).and_return(true)
+      allow_any_instance_of(GitReflow).to receive(:append_to_merge_commit_message).and_return(true)
     end
 
     context "and force-merging" do
       let(:inputs) do
         {
-          base:      "base_branch",
-          title:     "title",
-          message:   "message",
-          skip_lgtm: true
+          base:    "base_branch",
+          title:   "title",
+          message: "message",
+          force:   true
         }
       end
 

data/spec/lib/git_reflow/git_server/git_hub_spec.rb

@@ -78,7 +78,7 @@ describe GitReflow::GitServer::GitHub do
     subject                     { github.authenticate }
 
     before  do
-      allow(GitReflow::GitServer::GitHub).to receive(:user).and_return('reenhanced')
+      allow(GitReflow::GitServer::GitHub).to receive(:user).and_return(user)
       allow(github_api).to receive(:oauth).and_return(github_authorizations)
       allow(github_api).to receive_message_chain(:oauth, :all).and_return([])
       allow(github).to receive(:run).with('hostname', loud: false).and_return(hostname)
@@ -151,12 +151,86 @@ describe GitReflow::GitServer::GitHub do
           body: { error: "GET https://api.github.com/authorizations: 401 Bad credentials" }
         }}
 
+        before do
+          allow(GitReflow::Config).to receive(:get).and_call_original
+          allow(GitReflow::Config).to receive(:get).with('github.oauth-token').and_return(oauth_token_hash[:token])
+          allow(Github::Client).to receive(:new).and_return(github_api)
+          allow(github_authorizations).to receive(:authenticated?).and_return(true)
+          allow(github_api.oauth).to receive(:create).with({ scopes: ['repo'], note: "git-reflow (#{hostname})" }).and_return(oauth_token_hash)
+
+          stub_request(:get, %r{/user}).
+            to_return(
+              body: Fixture.new('authentication_failure.json').to_s,
+              status: 401,
+              headers: {'content-type' => 'application/json; charset=utf-8', status: 'Unauthorized'},
+            )
+        end
+
+        it "notifies the user of successful setup" do
+          expect { subject }.to have_said "Your GitHub account was successfully setup!", :success
+        end
+
+        it "creates a new GitHub oauth token" do
+          expect(github_api.oauth).to receive(:create).and_return(oauth_token_hash)
+          subject
+        end
+
+        it "creates git config keys for github connections" do
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all github.site \"#{github_site}\"", blocking: false
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all github.endpoint \"#{github_api_endpoint}\"", blocking: false
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all github.oauth-token \"#{oauth_token_hash[:token]}\"", blocking: false
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all reflow.git-server \"GitHub\"", blocking: false
+        end
+
+      end
+    end
+
+    context 'already authenticated' do
+      let(:oauth_token) { "abc123" }
+
+      before do
+        allow(GitReflow::Config).to receive(:get).and_call_original
+        allow(GitReflow::Config).to receive(:get).with('github.oauth-token').and_return(oauth_token)
+      end
+
+      context "and authentication token is still valid" do
+        before do
+          stub_request(:get, %r{/user}).
+            to_return(
+              body: Fixture.new('users/user.json').to_s,
+              status: 200,
+              headers: { content_type: "application/json; charset=utf-8" }
+            )
+
+          allow(Github::Client).to receive(:new).and_return(github_api)
+          allow(github_api).to receive(:oauth).and_return(github_authorizations)
+          allow(github_authorizations).to receive(:authenticated?).and_return(true)
+          allow(github_api.oauth).to receive(:create).with({ scopes: ['repo'], note: "git-reflow (#{hostname})" }).and_return(oauth_token_hash)
+        end
+
+        it "resolves all missing git-reflow configurations" do
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all github.site \"#{github_site}\"", blocking: false
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all github.endpoint \"#{github_api_endpoint}\"", blocking: false
+          expect{ subject }.to have_run_command_silently "git config -f #{GitReflow::Config::CONFIG_FILE_PATH} --replace-all reflow.git-server \"GitHub\"", blocking: false
+          expect { subject }.to have_said "Your GitHub account was already setup with: "
+          expect { subject }.to have_said "\tUser Name: #{user}"
+          expect { subject }.to have_said "\tEndpoint: #{github_api_endpoint}"
+        end
+      end
+
+      context "and authentication token is expired" do
+        let(:unauthorized_error_response) {{
+          response_headers: {'content-type' => 'application/json; charset=utf-8', status: 'Unauthorized'},
+          method: 'GET',
+          status: '401',
+          body: { error: "GET https://api.github.com/authorizations: 401 Bad credentials" }
+        }}
+
         before do
           allow(Github::Client).to receive(:new).and_raise Github::Error::Unauthorized.new(unauthorized_error_response)
         end
 
-        it "notifies user of invalid login details" do
-          expect { subject }.to have_said "Github Authentication Error: #{Github::Error::Unauthorized.new(unauthorized_error_response).inspect}", :error
+        it "requests a new oauth token" do
         end
       end
     end

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

@@ -364,7 +364,7 @@ describe GitReflow::GitServer::PullRequest do
     subject { pr.merge! inputs }
 
     before do
-      allow(GitReflow).to receive(:append_to_squashed_commit_message)
+      allow(GitReflow).to receive(:append_to_merge_commit_message)
       allow(pr).to receive(:commit_message_for_merge).and_return(commit_message_for_merge)
     end
 
@@ -376,7 +376,7 @@ describe GitReflow::GitServer::PullRequest do
       it "updates both feature and destination branch and squash-merges feature into base branch" do
         expect(GitReflow).to receive(:update_current_branch)
         expect(GitReflow).to receive(:fetch_destination).with(pr.base_branch_name)
-        expect(GitReflow).to receive(:append_to_squashed_commit_message).with(pr.commit_message_for_merge)
+        expect(GitReflow).to receive(:append_to_merge_commit_message).with(pr.commit_message_for_merge)
         expect { subject }.to have_run_commands_in_order [
           "git checkout #{pr.base_branch_name}",
           "git pull origin #{pr.base_branch_name}",
@@ -470,6 +470,12 @@ describe GitReflow::GitServer::PullRequest do
       before  { allow(pr).to receive(:approvals).and_return(['sally', 'joey']) }
       specify { expect(subject).to include "\nLGTM given by: @sally, @joey\n" }
     end
+
+    context "with custom merge commit message template" do
+      before { allow(GitReflow).to receive(:merge_commit_template).and_return("Super cool changes") }
+      specify { expect(subject).to include "Super cool changes" }
+      specify { expect(subject).to_not include "\nMerges ##{pr.number}\n" }
+    end
   end
 
   context "#cleanup_feature_branch?" do
@@ -490,7 +496,7 @@ describe GitReflow::GitServer::PullRequest do
         it     { should be_truthy }
       end
 
-      context "and user choose not to cleanup" do
+      context "and user chooses not to cleanup" do
         before { expect(pr).to receive(:ask).with('Would you like to push this branch to your remote repo and cleanup your feature branch? ').and_return('no') }
         it     { should be_falsy }
       end

data/spec/lib/git_reflow/git_server_spec.rb

@@ -1,101 +1,101 @@
-require 'spec_helper'
-
-describe GitReflow::GitServer do
-  let(:connection_options) { nil }
-
-  subject { GitReflow::GitServer.connect connection_options }
-
-  before do
-    allow(GitReflow::GitServer::GitHub).to receive(:new)
-
-    module GitReflow::GitServer
-      class DummyHub < Base
-        def initialize(options)
-          "Initialized with #{options}"
-        end
-
-        def authenticate(options={})
-        end
-
-        def connection
-          'Connected!'
-        end
-      end
-    end
-  end
-
-  describe '.connect(options)' do
-    it 'initializes a new GitHub server provider by default' do
-      stubbed_github = Class.new
-      allow(stubbed_github).to receive(:authenticate)
-      expect(GitReflow::GitServer::GitHub).to receive(:new).and_return(stubbed_github)
-      subject
-    end
-
-    context 'provider is specified' do
-      let(:connection_options) { {provider: 'DummyHub'}.merge(expected_server_options) }
-      let(:expected_server_options) {{ basic_auth: 'user:pass', end_point: 'https://api.example.com' }}
-
-      it 'initializes any server provider that has been implemented' do
-        dummy_hub = GitReflow::GitServer::DummyHub.new({})
-        expect(GitReflow::GitServer::DummyHub).to receive(:new).with(expected_server_options).and_return(dummy_hub)
-        expect(subject).to eq(dummy_hub)
-        expect($says).not_to include 'GitServer not setup for: DummyHub'
-      end
-    end
-
-    context 'provider not yet implemented' do
-      let(:connection_options) {{ provider: 'GitLab' }}
-      it { expect{ subject }.to have_said "Error connecting to GitLab: GitServer not setup for \"GitLab\"", :error }
-    end
-  end
-
-  describe '.current_provider' do
-    subject { GitReflow::GitServer.current_provider }
-
-    before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server', local: true).and_return(nil) }
-
-    context 'Reflow setup to use GitHub' do
-      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GitHub') }
-      it     { is_expected.to eq(GitReflow::GitServer::GitHub) }
-    end
-
-    context 'Reflow has not yet been setup' do
-      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('') }
-      it     { is_expected.to be_nil }
-      it     { expect{ subject }.to have_said "Reflow hasn't been setup yet.  Run 'git reflow setup' to continue", :notice }
-    end
-
-    context 'an unknown server provider is stored in the git config' do
-      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GittyUp') }
-
-      it { is_expected.to be_nil }
-      it { expect{ subject }.to have_said "GitServer not setup for \"GittyUp\"", :error }
-    end
-  end
-
-  describe '.connection' do
-    subject { GitReflow::GitServer.connection }
-
-    before do
-      allow(GitReflow::Config).to receive(:get).with('reflow.git-server', local: true).and_return(nil)
-      allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return(nil)
-    end
-
-    it { is_expected.to be_nil }
-
-    context "with a valid provider" do
-      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GitHub') }
-      it 'calls connection on the provider' do
-        expect(GitReflow::GitServer::GitHub).to receive(:connection)
-        subject
-      end
-    end
-
-    context "with an invalid provider" do
-      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GittyUp') }
-      it     { is_expected.to be_nil }
-      it     { expect{ subject }.to have_said "GitServer not setup for \"GittyUp\"", :error }
-    end
-  end
-end
+require 'spec_helper'
+
+describe GitReflow::GitServer do
+  let(:connection_options) { nil }
+
+  subject { GitReflow::GitServer.connect connection_options }
+
+  before do
+    allow(GitReflow::GitServer::GitHub).to receive(:new)
+
+    module GitReflow::GitServer
+      class DummyHub < Base
+        def initialize(options)
+          "Initialized with #{options}"
+        end
+
+        def authenticate(options={})
+        end
+
+        def connection
+          'Connected!'
+        end
+      end
+    end
+  end
+
+  describe '.connect(options)' do
+    it 'initializes a new GitHub server provider by default' do
+      stubbed_github = Class.new
+      allow(stubbed_github).to receive(:authenticate)
+      expect(GitReflow::GitServer::GitHub).to receive(:new).and_return(stubbed_github)
+      subject
+    end
+
+    context 'provider is specified' do
+      let(:connection_options) { {provider: 'DummyHub'}.merge(expected_server_options) }
+      let(:expected_server_options) {{ basic_auth: 'user:pass', end_point: 'https://api.example.com' }}
+
+      it 'initializes any server provider that has been implemented' do
+        dummy_hub = GitReflow::GitServer::DummyHub.new({})
+        expect(GitReflow::GitServer::DummyHub).to receive(:new).with(expected_server_options).and_return(dummy_hub)
+        expect(subject).to eq(dummy_hub)
+        expect($says).not_to include 'GitServer not setup for: DummyHub'
+      end
+    end
+
+    context 'provider not yet implemented' do
+      let(:connection_options) {{ provider: 'GitLab' }}
+      it { expect{ subject }.to have_said "Error connecting to GitLab: GitServer not setup for \"GitLab\"", :error }
+    end
+  end
+
+  describe '.current_provider' do
+    subject { GitReflow::GitServer.current_provider }
+
+    before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server', local: true).and_return(nil) }
+
+    context 'Reflow setup to use GitHub' do
+      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GitHub') }
+      it     { is_expected.to eq(GitReflow::GitServer::GitHub) }
+    end
+
+    context 'Reflow has not yet been setup' do
+      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('') }
+      it     { is_expected.to be_nil }
+      it     { expect{ subject }.to have_said "Reflow hasn't been setup yet.  Run 'git reflow setup' to continue", :notice }
+    end
+
+    context 'an unknown server provider is stored in the git config' do
+      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GittyUp') }
+
+      it { is_expected.to be_nil }
+      it { expect{ subject }.to have_said "GitServer not setup for \"GittyUp\"", :error }
+    end
+  end
+
+  describe '.connection' do
+    subject { GitReflow::GitServer.connection }
+
+    before do
+      allow(GitReflow::Config).to receive(:get).with('reflow.git-server', local: true).and_return(nil)
+      allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return(nil)
+    end
+
+    it { is_expected.to be_nil }
+
+    context "with a valid provider" do
+      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GitHub') }
+      it 'calls connection on the provider' do
+        expect(GitReflow::GitServer::GitHub).to receive(:connection)
+        subject
+      end
+    end
+
+    context "with an invalid provider" do
+      before { allow(GitReflow::Config).to receive(:get).with('reflow.git-server').and_return('GittyUp') }
+      it     { is_expected.to be_nil }
+      it     { expect{ subject }.to have_said "GitServer not setup for \"GittyUp\"", :error }
+    end
+  end
+end