git_helper 1.2.0 → 2.0.2

data/spec/git_helper/code_request_spec.rb

@@ -0,0 +1,259 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::CodeRequest do
+  let(:highline_cli) { double(:highline_cli) }
+  let(:local_code) { double(:local_code, project_name: 'name', branch: 'branch') }
+  let(:process_project) { double(:process_project, create: :created, merge: :merged) }
+
+  subject { GitHelper::CodeRequest.new }
+
+  before do
+    allow(GitHelper::LocalCode).to receive(:new).and_return(local_code)
+    allow(GitHelper::HighlineCli).to receive(:new).and_return(highline_cli)
+  end
+
+  describe '#create' do
+    before do
+      allow(subject).to receive(:base_branch).and_return('base')
+      allow(subject).to receive(:new_code_request_title).and_return('Title')
+    end
+
+    it 'should call to process the project' do
+      expect(subject).to receive(:process_project).and_return(process_project)
+      subject.create
+    end
+
+    it 'should call create' do
+      allow(subject).to receive(:process_project).and_return(process_project)
+      expect(process_project).to receive(:create)
+      subject.create
+    end
+
+    it 'should call base_branch and new_code_request_title' do
+      expect(subject).to receive(:base_branch).and_return('base')
+      expect(subject).to receive(:new_code_request_title).and_return('Title')
+      allow(subject).to receive(:process_project).and_return(process_project)
+      allow(process_project).to receive(:create)
+      subject.create
+    end
+  end
+
+  describe '#merge' do
+    it 'should call to process the project' do
+      expect(subject).to receive(:process_project).and_return(process_project)
+      subject.merge
+    end
+
+    it 'should call merge' do
+      allow(subject).to receive(:process_project).and_return(process_project)
+      expect(process_project).to receive(:merge)
+      subject.merge
+    end
+  end
+
+  describe '#process_project' do
+    it 'should call the local code to see if it is a github or gitlab project' do
+      expect(local_code).to receive(:gitlab_project?).and_return(false)
+      expect(local_code).to receive(:github_repo?).and_return(true)
+      subject.send(:process_project)
+    end
+
+    context 'when github and gitlab remotes are found' do
+      it 'should ask for clarification' do
+        allow(local_code).to receive(:gitlab_project?).and_return(true)
+        allow(local_code).to receive(:github_repo?).and_return(true)
+        expect(subject).to receive(:ask_for_clarification)
+        subject.send(:process_project)
+      end
+    end
+
+    context 'when github' do
+      it 'should call the github_pull_request' do
+        allow(local_code).to receive(:gitlab_project?).and_return(false)
+        allow(local_code).to receive(:github_repo?).and_return(true)
+        expect(subject).to receive(:github_pull_request)
+        subject.send(:process_project)
+      end
+    end
+
+    context 'when gitlab' do
+      it 'should call the gitlab_merge_request' do
+        allow(local_code).to receive(:gitlab_project?).and_return(true)
+        allow(local_code).to receive(:github_repo?).and_return(false)
+        expect(subject).to receive(:gitlab_merge_request)
+        subject.send(:process_project)
+      end
+    end
+
+    context 'when no github or gitlab remotes are found' do
+      it 'should raise error' do
+        allow(local_code).to receive(:gitlab_project?).and_return(false)
+        allow(local_code).to receive(:github_repo?).and_return(false)
+        expect(subject).to receive(:exit)
+        subject.send(:process_project)
+      end
+    end
+  end
+
+  describe '#ask_for_clarification' do
+    it 'should ask the CLI' do
+      expect(highline_cli).to receive(:conflicting_remote_clarification).and_return('github')
+      subject.send(:ask_for_clarification)
+    end
+
+    context 'when response is github' do
+      it 'should return github_pull_request' do
+        allow(highline_cli).to receive(:conflicting_remote_clarification).and_return('github')
+        expect(subject).to receive(:github_pull_request)
+        subject.send(:ask_for_clarification)
+      end
+
+      it 'should return github_pull_request' do
+        allow(highline_cli).to receive(:conflicting_remote_clarification).and_return('Github')
+        expect(subject).to receive(:github_pull_request)
+        subject.send(:ask_for_clarification)
+      end
+    end
+
+    context 'when response is gitlab' do
+      it 'should return gitlab_merge_request' do
+        allow(highline_cli).to receive(:conflicting_remote_clarification).and_return('gitlab')
+        expect(subject).to receive(:gitlab_merge_request)
+        subject.send(:ask_for_clarification)
+      end
+
+      it 'should return gitlab_merge_request' do
+        allow(highline_cli).to receive(:conflicting_remote_clarification).and_return('Gitlab')
+        expect(subject).to receive(:gitlab_merge_request)
+        subject.send(:ask_for_clarification)
+      end
+    end
+
+    context 'when response is neither' do
+      it 'should raise an error' do
+        allow(highline_cli).to receive(:conflicting_remote_clarification).and_return('huh?')
+        expect(subject).to receive(:exit)
+        subject.send(:ask_for_clarification)
+      end
+    end
+  end
+
+  describe '#github_pull_request' do
+    it 'should call the GitHelper::GitHubPullRequest' do
+      expect(GitHelper::GitHubPullRequest).to receive(:new)
+      subject.send(:github_pull_request)
+    end
+  end
+
+  describe '#gitlab_merge_request' do
+    it 'should call the GitHelper::GitLabMergeRequest' do
+      expect(GitHelper::GitLabMergeRequest).to receive(:new)
+      subject.send(:gitlab_merge_request)
+    end
+  end
+
+  describe '#local_project' do
+    it 'should call the name of the local_code' do
+      expect(local_code).to receive(:project_name)
+      subject.send(:local_project)
+    end
+  end
+
+  describe '#default_branch' do
+    it 'should call the name of the local_code' do
+      expect(local_code).to receive(:default_branch)
+      subject.send(:default_branch)
+    end
+  end
+
+  describe '#base_branch' do
+    it 'should call the default branch' do
+      expect(subject).to receive(:default_branch)
+      allow(highline_cli).to receive(:base_branch_default?).at_least(:once)
+      allow(highline_cli).to receive(:base_branch).at_least(:once).and_return('base')
+      subject.send(:base_branch)
+    end
+
+    it 'should ask the CLI to ask the user' do
+      allow(subject).to receive(:default_branch)
+      expect(highline_cli).to receive(:base_branch_default?).at_least(:once)
+      allow(highline_cli).to receive(:base_branch).at_least(:once).and_return('base')
+      subject.send(:base_branch)
+    end
+
+    context 'if the user says no' do
+      it "definitely asks for the user's base branch" do
+        allow(subject).to receive(:default_branch)
+        expect(highline_cli).to receive(:base_branch_default?).at_least(:once).and_return(false)
+        expect(highline_cli).to receive(:base_branch).at_least(:once).and_return('base')
+        subject.send(:base_branch)
+      end
+    end
+
+    context 'if the user says yes' do
+      it "does not ask for the user's base branch" do
+        allow(subject).to receive(:default_branch)
+        expect(highline_cli).to receive(:base_branch_default?).at_least(:once).and_return(true)
+        expect(highline_cli).not_to receive(:base_branch)
+        subject.send(:base_branch)
+      end
+    end
+  end
+
+  describe '#autogenerated_title' do
+    it 'should generate a title based on the branch' do
+      expect(subject).to receive(:local_branch).and_return('branch')
+      expect(local_code).to receive(:generate_title)
+      subject.send(:autogenerated_title)
+    end
+  end
+
+  describe '#new_code_request_title' do
+    it 'should call autogenerated title method' do
+      expect(subject).to receive(:autogenerated_title)
+      allow(highline_cli).to receive(:accept_autogenerated_title?).at_least(:once)
+      allow(highline_cli).to receive(:title).at_least(:once).and_return('Title')
+      subject.send(:new_code_request_title)
+    end
+
+    it 'should ask the CLI to ask the user' do
+      allow(subject).to receive(:autogenerated_title)
+      expect(highline_cli).to receive(:accept_autogenerated_title?).at_least(:once)
+      allow(highline_cli).to receive(:title).at_least(:once).and_return('Title')
+      subject.send(:new_code_request_title)
+    end
+
+    context 'if the user says no' do
+      it "definitely asks for the user's title" do
+        allow(subject).to receive(:autogenerated_title)
+        expect(highline_cli).to receive(:accept_autogenerated_title?).at_least(:once).and_return(false)
+        expect(highline_cli).to receive(:title).at_least(:once).and_return('Title')
+        subject.send(:new_code_request_title)
+      end
+    end
+
+    context 'if the user says yes to original title' do
+      it "does not ask for the user's chosen title" do
+        allow(subject).to receive(:autogenerated_title)
+        expect(highline_cli).to receive(:accept_autogenerated_title?).at_least(:once).and_return(true)
+        expect(highline_cli).not_to receive(:title)
+        subject.send(:new_code_request_title)
+      end
+    end
+  end
+
+  describe '#local_code' do
+    it 'should call the octokit client' do
+      expect(GitHelper::LocalCode).to receive(:new).and_return(local_code)
+      subject.send(:local_code)
+    end
+  end
+
+  describe '#cli' do
+    it 'should call the octokit client' do
+      expect(GitHelper::HighlineCli).to receive(:new).and_return(highline_cli)
+      subject.send(:cli)
+    end
+  end
+end

data/spec/git_helper/empty_commit_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::EmptyCommit do
+  let(:local_code) { double(:local_code, empty_commit: :commit) }
+
+  subject { GitHelper::EmptyCommit.new }
+
+  it 'should call GitHelper::LocalCode' do
+    expect(GitHelper::LocalCode).to receive(:new).and_return(local_code)
+    subject.execute
+  end
+
+  it 'should call the empty_commit method from the local code class' do
+    allow(GitHelper::LocalCode).to receive(:new).and_return(local_code)
+    expect(local_code).to receive(:empty_commit)
+    subject.execute
+  end
+end

data/spec/git_helper/forget_local_commits_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::ForgetLocalCommits do
+  let(:local_code) { double(:local_code, forget_local_commits: :commit) }
+
+  subject { GitHelper::ForgetLocalCommits.new }
+
+  it 'should call GitHelper::LocalCode' do
+    expect(GitHelper::LocalCode).to receive(:new).and_return(local_code)
+    subject.execute
+  end
+
+  it 'should call the forget_local_commits method from the local code class' do
+    allow(GitHelper::LocalCode).to receive(:new).and_return(local_code)
+    expect(local_code).to receive(:forget_local_commits)
+    subject.execute
+  end
+end

data/spec/git_helper/git_config_reader_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::GitConfigReader do
+  let(:github_token) { '1234ASDF1234ASDF' }
+  let(:gitlab_token) { 'ASDF123ASDF1234' }
+  let(:config_file) {
+    {
+      github_user: 'github-user-name',
+      github_token: github_token,
+      gitlab_user: 'gitlab-user-name',
+      gitlab_token: gitlab_token
+    }
+  }
+
+  subject { GitHelper::GitConfigReader.new }
+
+  describe '#gitlab_token' do
+    it 'should locate the gitlab_token' do
+      expect(subject).to receive(:config_file).and_return(config_file)
+      expect(subject.gitlab_token).to eq(gitlab_token)
+    end
+
+    it 'should call the config file' do
+      expect(subject).to receive(:config_file).and_return(config_file)
+      subject.gitlab_token
+    end
+  end
+
+  describe '#github_token' do
+    it 'should locate the github_token' do
+      expect(subject).to receive(:config_file).and_return(config_file)
+      expect(subject.github_token).to eq(github_token)
+    end
+
+    it 'should call the config file' do
+      expect(subject).to receive(:config_file).and_return(config_file)
+      subject.github_token
+    end
+  end
+
+  describe '#config_file' do
+    it 'should yaml load the file path' do
+      expect(YAML).to receive(:load_file)
+      subject.send(:config_file)
+    end
+  end
+
+  describe '#git_config_file_path' do
+    it 'should look in the current directory' do
+      expect(Dir).to receive(:pwd).and_return('/Users/firstnamelastname/path/to/git_helper')
+      subject.send(:git_config_file_path)
+    end
+
+    it 'should return the base path with the git config file at the end' do
+      allow(Dir).to receive(:pwd).and_return('/Users/firstnamelastname/path/to/git_helper')
+      expect(subject.send(:git_config_file_path)).to eq('/Users/firstnamelastname/.git_config.yml')
+    end
+  end
+end

data/spec/git_helper/gitlab_client_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::GitLabClient do
+  let(:git_config_reader) { double(:git_config_reader, gitlab_token: :token) }
+
+  subject { GitHelper::GitLabClient.new }
+
+  before do
+    allow(GitHelper::GitConfigReader).to receive(:new).and_return(git_config_reader)
+  end
+
+  describe '#client' do
+    it 'should call the GitLab client to make a new client' do
+      expect(Gitlab).to receive(:client)
+      subject.client
+    end
+  end
+
+  describe '#git_config_reader' do
+    it 'should make a new git config reader' do
+      expect(GitHelper::GitConfigReader).to receive(:new)
+      subject.send(:git_config_reader)
+    end
+  end
+end

data/spec/git_helper/highline_cli_spec.rb

@@ -0,0 +1,215 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::HighlineCli do
+  let(:response) { double(:response, readline: true, to_i: 5) }
+  let(:highline_client) { double(:highline_cli, ask: response) }
+
+  subject { GitHelper::HighlineCli.new }
+
+  before do
+    allow(HighLine).to receive(:new).and_return(highline_client)
+  end
+
+  describe '#new_branch_name' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).with('New branch name?')
+      subject.new_branch_name
+    end
+
+    it 'should come out a string' do
+      expect(subject.new_branch_name).to be_a(String)
+    end
+  end
+
+  describe '#process_directory_remotes' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).and_return('y')
+      subject.process_directory_remotes?('directory')
+    end
+
+    it 'should be a boolean at the end' do
+      allow(subject).to receive(:ask).and_return('y')
+      expect([true, false]).to include(subject.process_directory_remotes?('directory'))
+    end
+
+    it 'should come out as a true boolean if somebody responds y' do
+      allow(subject).to receive(:ask).and_return('y')
+      expect(subject.process_directory_remotes?('directory')).to eq(true)
+    end
+
+    it 'should come out as a false boolean if somebody responds n' do
+      allow(subject).to receive(:ask).and_return('n')
+      expect(subject.process_directory_remotes?('directory')).to eq(false)
+    end
+
+    it 'should come out as true if somebody presses enter' do
+      allow(subject).to receive(:ask).and_return('')
+      expect(subject.accept_autogenerated_title?('directory')).to eq(true)
+    end
+  end
+
+  describe '#conflicting_remote_clarification' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).with('Found git remotes for both GitHub and GitLab. Would you like to proceed with GitLab or GitHub? (github/gitlab)').and_return('gitlab')
+      subject.conflicting_remote_clarification
+    end
+
+    it 'should come out a string' do
+      expect(subject.conflicting_remote_clarification).to be_a(String)
+    end
+  end
+
+  describe '#title' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).with('Title?')
+      subject.title
+    end
+
+    it 'should come out a string' do
+      expect(subject.title).to be_a(String)
+    end
+  end
+
+  describe '#base_branch' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).with('Base branch?')
+      subject.base_branch
+    end
+
+    it 'should come out a string' do
+      expect(subject.base_branch).to be_a(String)
+    end
+  end
+
+  describe '#code_request_id' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).with('Example Request ID?')
+      subject.code_request_id('Example')
+    end
+
+    it 'should come out a string' do
+      expect(subject.code_request_id('Example')).to be_a(String)
+    end
+  end
+
+  describe '#accept_autogenerated_title' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).and_return('y')
+      subject.accept_autogenerated_title?('Title Example')
+    end
+
+    it 'should be a boolean at the end' do
+      allow(subject).to receive(:ask).and_return('y')
+      expect([true, false]).to include(subject.accept_autogenerated_title?('Title Example'))
+    end
+
+    it 'should come out as a true boolean if somebody responds y' do
+      allow(subject).to receive(:ask).and_return('y')
+      expect(subject.accept_autogenerated_title?('Title Example')).to eq(true)
+    end
+
+    it 'should come out as a true boolean if somebody responds n' do
+      allow(subject).to receive(:ask).and_return('n')
+      expect(subject.accept_autogenerated_title?('Title Example')).to eq(false)
+    end
+
+    it 'should come out as a true boolean if somebody responds yes' do
+      allow(subject).to receive(:ask).and_return('yes')
+      expect(subject.accept_autogenerated_title?('Title Example')).to eq(true)
+    end
+
+    it 'should come out as a false boolean if somebody responds no' do
+      allow(subject).to receive(:ask).and_return('no')
+      expect(subject.accept_autogenerated_title?('Title Example')).to eq(false)
+    end
+
+    it 'should come out as true if somebody presses enter' do
+      allow(subject).to receive(:ask).and_return('')
+      expect(subject.accept_autogenerated_title?('Title Example')).to eq(true)
+    end
+  end
+
+  describe '#base_branch_default' do
+    it "should ask the subject's ask method" do
+      expect(subject).to receive(:ask).and_return('y')
+      subject.base_branch_default?('default_branch')
+    end
+
+    it 'should be a boolean at the end' do
+      allow(subject).to receive(:ask).and_return('y')
+      expect([true, false]).to include(subject.base_branch_default?('default_branch'))
+    end
+
+    it 'should come out as a true boolean if somebody responds y' do
+      allow(subject).to receive(:ask).and_return('y')
+      expect(subject.base_branch_default?('default_branch')).to eq(true)
+    end
+
+    it 'should come out as a true boolean if somebody responds n' do
+      allow(subject).to receive(:ask).and_return('n')
+      expect(subject.base_branch_default?('default_branch')).to eq(false)
+    end
+
+    it 'should come out as a true boolean if somebody responds yes' do
+      allow(subject).to receive(:ask).and_return('yes')
+      expect(subject.base_branch_default?('default_branch')).to eq(true)
+    end
+
+    it 'should come out as a false boolean if somebody responds no' do
+      allow(subject).to receive(:ask).and_return('no')
+      expect(subject.base_branch_default?('default_branch')).to eq(false)
+    end
+
+    it 'should come out as true if somebody presses enter' do
+      allow(subject).to receive(:ask).and_return('')
+      expect(subject.base_branch_default?('default_branch')).to eq(true)
+    end
+  end
+
+  describe '#merge_method' do
+    it "should ask the subject's ask_options method" do
+      expect(subject).to receive(:ask_options).and_return(3)
+      subject.merge_method(['1', '2', '3'])
+    end
+
+    it 'should return a string' do
+      allow(subject).to receive(:ask_options).and_return(2)
+      expect(subject.merge_method(['1', '2', '3'])).to be_a(String)
+    end
+  end
+
+  describe '#template_to_apply' do
+    it "should ask the subject's ask_options method" do
+      expect(subject).to receive(:ask_options).and_return(3)
+      subject.template_to_apply(['option1', 'option2', 'option3'], 'example type')
+    end
+
+    it 'should return a string' do
+      allow(subject).to receive(:ask_options).and_return(3)
+      expect(subject.template_to_apply(['option1', 'option2', 'option3'], 'example type')).to eq('None')
+    end
+  end
+
+  describe '#ask' do
+    it 'should ask the highline client ask'do
+      expect(highline_client).to receive(:ask)
+      subject.send(:ask, 'prompt goes here')
+    end
+
+    it 'should return a string' do
+      expect(subject.send(:ask, 'prompt goes here')).to be_a(String)
+    end
+  end
+
+  describe '#ask_options' do
+    it 'should ask the highline client ask'do
+      expect(highline_client).to receive(:ask)
+      subject.send(:ask_options, 'prompt goes here', ['1', '2', '3'])
+    end
+
+    it 'should return an integer' do
+      expect(subject.send(:ask_options, 'prompt goes here', ['1', '2', '3'])).to be_a(Integer)
+    end
+  end
+end