git_helper 1.3.0 → 3.0.0

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_helper/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

data/spec/git_helper/local_code_spec.rb

@@ -0,0 +1,231 @@
+require 'spec_helper'
+require 'git_helper'
+
+describe GitHelper::LocalCode do
+  let(:response) { double(:response, readline: true, to_i: 5) }
+  let(:local_codeent) { double(:local_code, ask: response) }
+  let(:ssh_remote) { 'origin\tgit@github.com:emmasax4/git_helper.git (fetch)' }
+  let(:https_remote) { 'origin\thttps://github.com/emmasax4/git_helper.git (fetch)' }
+  let(:github_remotes) { ['origin\tgit@github.com:emmasax4/git_helper.git (fetch)', 'origin\thttps://github.com/emmasax4/git_helper.git (fetch)' ] }
+  let(:gitlab_remotes) { ['origin\tgit@gitlab.com:emmasax4/git_helper.git (fetch)', 'origin\thttps://gitlab.com/emmasax4/git_helper.git (fetch)' ] }
+
+
+  subject { GitHelper::LocalCode.new }
+
+  before do
+    allow(subject).to receive(:system).and_return(nil)
+  end
+
+  describe '#checkout_default' do
+    it 'should make a system call' do
+      expect(subject).to receive(:system)
+      subject.checkout_default
+    end
+  end
+
+  describe '#forget_local_commits' do
+    it 'should make a system call' do
+      expect(subject).to receive(:system).exactly(2).times
+      subject.forget_local_commits
+    end
+
+    it 'should return nil' do
+      expect(subject.forget_local_commits).to eq(nil)
+    end
+  end
+
+  describe '#empty_commit' do
+    it 'should make a system call' do
+      expect(subject).to receive(:system)
+      subject.empty_commit
+    end
+  end
+
+  describe '#clean_branches' do
+    it 'should make a system call' do
+      expect(subject).to receive(:system).exactly(4).times
+      subject.clean_branches
+    end
+  end
+
+  describe '#new_branch' do
+    it 'should make a system call' do
+      expect(subject).to receive(:system).exactly(4).times
+      subject.new_branch('branch_name')
+    end
+  end
+
+  describe '#change_remote' do
+    it 'should return a string' do
+      expect(subject.change_remote('name', 'url')).to be_a(String)
+    end
+  end
+
+  describe '#remotes' do
+    it 'should return an array of strings' do
+      expect(subject.remotes).to be_a(Array)
+      expect(subject.remotes.first).to be_a(String)
+    end
+  end
+
+  describe '#remote_name' do
+    it 'should be a string' do
+      expect(subject.remote_name(ssh_remote)).to be_a(String)
+    end
+  end
+
+  describe '#ssh_remote' do
+    it 'should come out true if ssh' do
+      expect(subject.ssh_remote?(ssh_remote)).to eq(true)
+    end
+
+    it 'should come out false if https' do
+      expect(subject.ssh_remote?(https_remote)).to eq(false)
+    end
+  end
+
+  describe '#https_remote' do
+    it 'should come out false if ssh' do
+      expect(subject.https_remote?(ssh_remote)).to eq(false)
+    end
+
+    it 'should come out true if https' do
+      expect(subject.https_remote?(https_remote)).to eq(true)
+    end
+  end
+
+  describe '#remote_project' do
+    it 'should return just the plain project if ssh' do
+      expect(subject.remote_project(ssh_remote)).to eq('git_helper')
+    end
+
+    it 'should return just the plain project if https' do
+      expect(subject.remote_project(https_remote)).to eq('git_helper')
+    end
+  end
+
+  describe '#remote_source' do
+    it 'should return just the plain project if ssh' do
+      expect(subject.remote_source(ssh_remote)).to eq('github.com')
+    end
+
+    it 'should return just the plain project if https' do
+      expect(subject.remote_source(https_remote)).to eq('github.com')
+    end
+  end
+
+  describe '#github_repo' do
+    it 'should return true if github' do
+      allow(subject).to receive(:remotes).and_return(github_remotes)
+      expect(subject.github_repo?).to eq(true)
+    end
+
+    it 'should return false if gitlab' do
+      allow(subject).to receive(:remotes).and_return(gitlab_remotes)
+      expect(subject.github_repo?).to eq(false)
+    end
+  end
+
+  describe '#gitlab_project' do
+    it 'should return true if gitlab' do
+      allow(subject).to receive(:remotes).and_return(gitlab_remotes)
+      expect(subject.gitlab_project?).to eq(true)
+    end
+
+    it 'should return false if github' do
+      allow(subject).to receive(:remotes).and_return(github_remotes)
+      expect(subject.gitlab_project?).to eq(false)
+    end
+  end
+
+  describe '#project_name' do
+    it 'should return a string' do
+      expect(subject.project_name).to be_a(String)
+    end
+
+    it 'should equal this project name' do
+      allow_any_instance_of(String).to receive(:scan).and_return([['emmasax4/git_helper']])
+      expect(subject.project_name).to eq('emmasax4/git_helper')
+    end
+  end
+
+  describe '#branch' do
+    it 'should return a string' do
+      expect(subject.branch).to be_a(String)
+    end
+  end
+
+  describe '#default_branch' do
+    it 'should return a string' do
+      expect(subject.default_branch).to be_a(String)
+    end
+  end
+
+  describe '#template_options' do
+    let(:template_identifiers) do
+      {
+        nested_directory_name: 'PULL_REQUEST_TEMPLATE',
+        non_nested_file_name: 'pull_request_template'
+      }
+    end
+
+    it 'should return an array' do
+      expect(subject.template_options(template_identifiers)).to be_a(Array)
+    end
+
+    it 'should call Dir.glob and File.join' do
+      expect(Dir).to receive(:glob).and_return(['.github/pull_request_template.md']).at_least(:once)
+      expect(File).to receive(:join).at_least(:once)
+      subject.template_options(template_identifiers)
+    end
+  end
+
+  describe '#read_template' do
+    it 'should call File.open' do
+      expect(File).to receive(:open).and_return(double(read: true))
+      subject.read_template('.gitignore')
+    end
+  end
+
+  describe '#generate_title' do
+    it 'should return a title based on the branch' do
+      branch = 'jira-123-test-branch'
+      expect(subject.generate_title(branch)).to eq('JIRA-123 Test branch')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'jira_123_test_branch'
+      expect(subject.generate_title(branch)).to eq('JIRA-123 Test branch')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'jira-123_test_branch'
+      expect(subject.generate_title(branch)).to eq('JIRA-123 Test branch')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'test_branch'
+      expect(subject.generate_title(branch)).to eq('Test branch')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'test-branch'
+      expect(subject.generate_title(branch)).to eq('Test branch')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'test'
+      expect(subject.generate_title(branch)).to eq('Test')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'some_other_words_in_this_test_branch'
+      expect(subject.generate_title(branch)).to eq('Some other words in this test branch')
+    end
+
+    it 'should return a title based on the branch' do
+      branch = 'some-other-words-in-this-test-branch'
+      expect(subject.generate_title(branch)).to eq('Some other words in this test branch')
+    end
+  end
+end