pgit 0.0.2

data/spec/pgit/current_branch_spec.rb

@@ -0,0 +1,72 @@
+require 'pgit'
+
+describe 'PGit::CurrentBranch' do
+  describe '#name' do
+    it 'should return the current branch' do
+      git_branch_command = 'git branch'
+      fake_current_branch = 'master'
+      fake_branches = <<-FAKE_BRANCHES
+        some-branch
+        * master
+        some-other-branch
+      FAKE_BRANCHES
+
+      allow(PGit::CurrentBranch).to receive(:`).with(git_branch_command).and_return(fake_branches)
+      current_branch_name = PGit::CurrentBranch.new.name
+
+      expect(current_branch_name).to eq fake_current_branch
+    end
+  end
+
+  describe '#story_id' do
+    describe 'current branch has no story id' do
+      it 'should return nil' do
+        git_branch_command = 'git branch'
+        fake_current_branch = 'master'
+        fake_branches = <<-FAKE_BRANCHES
+        some-branch
+        * master
+        some-other-branch
+        FAKE_BRANCHES
+
+        allow_any_instance_of(PGit::CurrentBranch).to receive(:`).with(git_branch_command).and_return(fake_branches)
+        current_branch_story_id = PGit::CurrentBranch.new.story_id
+
+        expect(current_branch_story_id).to be_nil
+      end
+    end
+
+    describe 'current branch has story id' do
+      it 'should return the story id' do
+        git_branch_command = 'git branch'
+        fake_current_branch = 'master'
+        fake_branches = <<-FAKE_BRANCHES
+        some-branch
+        * some-feature-branch-123456
+        some-other-branch
+        FAKE_BRANCHES
+
+        allow_any_instance_of(PGit::CurrentBranch).to receive(:`).with(git_branch_command).and_return(fake_branches)
+        current_branch_story_id = PGit::CurrentBranch.new.story_id
+
+        expect(current_branch_story_id).to eq '123456'
+      end
+    end
+  end
+
+  describe 'when there is error or fatal in the result' do
+    it 'should raise the error' do
+      git_branch_command = 'git branch'
+      fake_current_branch = 'master'
+      fake_branches = <<-FAKE_BRANCHES
+        fatal: Not a git repository
+          (or any of the parent directories): .git
+      FAKE_BRANCHES
+
+
+      allow_any_instance_of(PGit::CurrentBranch).to receive(:`).with(git_branch_command).and_return(fake_branches)
+
+      expect{PGit::CurrentBranch.new}.to raise_error(fake_branches)
+    end
+  end
+end

data/spec/pgit/current_project_spec.rb

@@ -0,0 +1,80 @@
+require 'pgit'
+# .edderic-dotfiles/config.yml
+# projects
+#   - path:                  ~/Therapy-Exercises-Online
+#     id:                    12345
+#     api_token: asoeuhot
+#
+def successful_setup
+  fake_project_1 =  { "path" => "/Therapy-Exercises-Online/some_other_project",
+                      "id" => 12345,
+                      "api_token" => "astoeuh" }
+  fake_project_2 =  { "path" => "~/Therapy-Exercises-Online",
+                      "id" => 19191,
+                      "api_token" => "astoeuh" }
+  fake_project_list = [ fake_project_1, fake_project_2 ]
+  fake_yaml = { "projects" => fake_project_list }
+  fake_pwd = "/Therapy-Exercises-Online/some_other_project/some_subdirectory"
+  allow(Dir).to receive(:pwd).and_return(fake_pwd)
+
+  fake_configuration = double('configuration', to_yaml: fake_yaml)
+end
+
+describe 'PGit::CurrentProject' do
+  describe '#new(config_yaml)' do
+    describe 'none of the projects listed matches the working directory' do
+      it 'should throw an error' do
+        error_message = "None of the project paths matches the working directory"
+        fake_project_1 =  { "path" => "~/some-non-matching-path",
+                            "id" => 12345,
+                            "api_token" => "astoeuh" }
+        fake_project_2 =  { "path" => "~/some-other-non-matching-path",
+                            "id" => 19191,
+                            "api_token" => "astoeuh" }
+        fake_project_list = [ fake_project_1, fake_project_2 ]
+        fake_yaml = { "projects" => fake_project_list }
+        fake_pwd = "/Therapy-Exercises-Online/some_other_project/some_subdirectory"
+        fake_configuration = double('configuration', to_yaml: fake_yaml)
+        allow(Dir).to receive(:pwd).and_return(fake_pwd)
+
+        expect{ PGit::CurrentProject.new(fake_configuration.to_yaml) }.to raise_error(error_message)
+      end
+    end
+
+  end
+
+  describe '#pwd' do
+    describe 'more than one of the projects listed matches the working directory' do
+      it "should return the more specific directory" do
+        fake_configuration = successful_setup
+
+        current_project = PGit::CurrentProject.new(fake_configuration.to_yaml)
+        working_directory = current_project.pwd
+
+        expect(working_directory).to eq "/Therapy-Exercises-Online/some_other_project"
+      end
+    end
+  end
+
+  describe '#id' do
+    it 'should return the correct pivotal tracker project_id' do
+      fake_configuration = successful_setup
+
+      current_project = PGit::CurrentProject.new(fake_configuration.to_yaml)
+      project_id = current_project.id
+
+      expect(project_id).to eq 12345
+    end
+  end
+
+  describe '#api_token' do
+    it 'should return the api_token associated to the current project' do
+      fake_configuration = successful_setup
+
+      current_project = PGit::CurrentProject.new(fake_configuration.to_yaml)
+      api_token = current_project.api_token
+
+      expect(api_token).to eq 'astoeuh'
+    end
+  end
+end

data/spec/pgit/installer_spec.rb

@@ -0,0 +1,158 @@
+require 'pgit'
+
+describe 'PGit::Installer' do
+  describe '~/pgit.rc.yml exists' do
+    it 'should raise an error' do
+      global_opts = {}
+      opts = { }
+      args = {}
+      message = "Error: ~/.pgit.rc.yml already exists"
+      default_file_path = "~/.pgit.rc.yml"
+      file_expanded_path = "/home/edderic/.pgit.rc.yml"
+
+      allow(File).to receive(:expand_path).
+        with(".")
+      allow(File).to receive(:expand_path).
+        with(default_file_path).and_return(file_expanded_path)
+      allow(File).to receive(:exists?).
+        with(file_expanded_path).and_return(true)
+
+      expect do
+        PGit::Installer.new(global_opts, opts, args)
+      end.to raise_error message
+    end
+  end
+
+  describe '~/pgit.rc.yml does not exist' do
+    it 'should ask to continue or not' do
+      global_opts = {}
+      opts = { }
+      args = {}
+
+      message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
+        "Continue? [Y/n]"
+      confirmation_message = "Saving example pgit config in ~/.pgit.rc.yml..."
+      edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
+      answer = instance_double("String", chomp: 'y')
+      expanded_path = "/home/edderic/.pgit.rc.yml"
+      fake_writable_file = double('File')
+      allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
+      allow(File).to receive(:exists?).with(expanded_path).and_return(false)
+      allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
+      allow(STDIN).to receive(:gets).and_return(answer)
+      allow_any_instance_of(PGit::Installer).to receive(:puts).with(message)
+      allow_any_instance_of(PGit::Installer).to receive(:puts).with(confirmation_message)
+      allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
+      installer = PGit::Installer.new(global_opts, opts, args)
+
+      expect(installer).to have_received(:puts).with(message)
+    end
+
+    describe 'user answers "y"' do
+      it "should show the save message" do
+        global_opts = {}
+        opts = { }
+        args = {}
+
+        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
+          "Continue? [Y/n]"
+        save_message = "Saving example pgit config in ~/.pgit.rc.yml..."
+        expanded_path = "/home/edderic/.pgit.rc.yml"
+        edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
+        fake_writable_file = double('File')
+        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
+        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
+        allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(save_message)
+
+        answer = instance_double("String", chomp: 'Y')
+        allow(STDIN).to receive(:gets).and_return(answer)
+
+        installer = PGit::Installer.new(global_opts, opts, args)
+
+        expect(installer).to have_received(:puts).with(save_message)
+      end
+
+      it 'should save the file under ~/.pgit.rc.yml' do
+        global_opts = {}
+        opts = { }
+        args = {}
+
+        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
+          "Continue? [Y/n]"
+        save_message = "Saving example pgit config in ~/.pgit.rc.yml..."
+        expanded_path = "/home/edderic/.pgit.rc.yml"
+        fake_writable_file = double('File')
+        edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
+        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
+        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
+        allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(save_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
+
+        answer = instance_double("String", chomp: 'Y')
+        allow(STDIN).to receive(:gets).and_return(answer)
+
+        installer = PGit::Installer.new(global_opts, opts, args)
+
+        expect(File).to have_received(:open).with(expanded_path, 'w')
+      end
+
+      it 'should ask the user to edit the config file' do
+        global_opts = {}
+        opts = { }
+        args = {}
+
+        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
+          "Continue? [Y/n]"
+        save_message = "Saving example pgit config in ~/.pgit.rc.yml..."
+        edit_message = "Saved! Please edit ~/.pgit.rc.yml and add the proper Pivotal Tracker API tokens, id, and file paths for each project"
+        expanded_path = "/home/edderic/.pgit.rc.yml"
+        fake_writable_file = double('File')
+        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
+        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
+        allow(File).to receive(:open).with(expanded_path, 'w').and_return(fake_writable_file)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(edit_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(save_message)
+
+        answer = instance_double("String", chomp: 'Y')
+        allow(STDIN).to receive(:gets).and_return(answer)
+
+        installer = PGit::Installer.new(global_opts, opts, args)
+
+        expect(installer).to have_received(:puts).with(edit_message)
+      end
+    end
+
+    describe 'user answers with "n"' do
+      it "should show the abort message" do
+        message = "Aborting installation..."
+        global_opts = {}
+        opts = { }
+        args = {}
+
+        first_message = "*** Installing example pgit configuration file under ~/.pgit.rc.yml. " +
+          "Continue? [Y/n]"
+        expanded_path = "/home/edderic/.pgit.rc.yml"
+        allow(File).to receive(:expand_path).with("~/.pgit.rc.yml").and_return(expanded_path)
+        allow(File).to receive(:exists?).with(expanded_path).and_return(false)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(first_message)
+        allow_any_instance_of(PGit::Installer).to receive(:puts).with(message)
+
+        answer = instance_double("String", chomp: 'n')
+        allow(STDIN).to receive(:gets).and_return(answer)
+
+        installer = PGit::Installer.new(global_opts, opts, args)
+
+        expect(installer).to have_received(:puts).with(message)
+      end
+    end
+
+    describe 'user answers with something els' do
+    end
+  end
+end

data/spec/pgit/pgit_spec.rb

@@ -0,0 +1,33 @@
+describe 'pgit' do
+  describe 'story_branch' do
+    it 'should output the help info for story_branch command' do
+      pgit_expanded_path = File.expand_path('bin/pgit')
+
+      result = `bundle exec #{pgit_expanded_path} story_branch`
+      story_branch_matches = result.match(/story_branch/)
+      name_match = result.match(/NAME/)
+
+      expect(story_branch_matches.length).to be >= 1
+      expect(name_match.length).to be >= 1
+    end
+  end
+
+  # describe 'install' do
+    # it 'should ask the user some questions' do
+      # pgit_expanded_path = File.expand_path('bin/pgit')
+      # message = "This will save the config file on ~/.pgit.rc.yml"
+      # allow(Kernel).to receive(:puts).with message
+#
+      # result = `bundle exec #{pgit_expanded_path} install`
+      # expect(Kernel).to have_received(:puts).with message
+    # end
+  # end
+
+  # describe 'commit' do
+    # describe 'on branch some-story-branch-1234' do
+      # it 'should prepend the commit with [#1234]' do
+#
+      # end
+    # end
+  # end
+end

data/spec/pgit/story_branch/application_spec.rb

@@ -0,0 +1,48 @@
+require 'pgit'
+
+describe 'PGit::StoryBranch::Application' do
+  describe '#new' do
+    describe 'options hash has start: 1234,
+             and "config" => is some config' do
+      it 'should call #start on an instance of StoryBranch' do
+        global_opts = {}
+        opts = { start: 1234 }
+        args = {}
+        fake_story_branch = instance_double('PGit::StoryBranch')
+        fake_config_yaml = double('config_yaml')
+        fake_configuration = instance_double('PGit::Configuration', to_yaml: fake_config_yaml)
+        allow(fake_story_branch).to receive(:start)
+        fake_get_request_results = double('json_response')
+        fake_story_id = double('fake_story_id')
+
+        fake_story = double('PGit::Story', id: fake_story_id)
+        fake_story_class = class_double('PGit::Story', get: fake_get_request_results)
+        fake_current_project = double('current_project')
+        fake_name_parser = instance_double(PGit::StoryBranch::NameParser)
+        allow(PGit::CurrentProject).to receive(:new).with(fake_config_yaml).and_return(fake_current_project)
+        allow(PGit::StoryBranch::NameParser).to receive(:new).with(fake_story).and_return(fake_name_parser)
+        allow(PGit::Story).to receive(:get).with(1234, fake_current_project).and_return(fake_story)
+        allow(PGit::StoryBranch).to receive(:new).with(fake_name_parser).and_return(fake_story_branch)
+        allow(PGit::Configuration).to receive(:new).and_return(fake_configuration)
+
+        PGit::StoryBranch::Application.new(global_opts, opts, args)
+
+        expect(fake_story_branch).to have_received(:start)
+      end
+    end
+
+    describe 'options passed in have keys that only point to nil or false' do
+      it 'should show the helpfile' do
+        global_opts = {}
+        opts = { start: nil, finish: false }
+        args = {}
+        story_branch_help_call = 'pgit story_branch --help'
+        allow_any_instance_of(PGit::StoryBranch::Application).to receive(:`).with(story_branch_help_call)
+
+        story_branch = PGit::StoryBranch::Application.new(global_opts, opts, args)
+
+        expect(story_branch).to have_received(:`).with(story_branch_help_call)
+      end
+    end
+  end
+end

data/spec/pgit/story_branch/name_parser_spec.rb

@@ -0,0 +1,42 @@
+require 'pgit'
+
+describe 'PGit::StoryBranch::NameParser' do
+  describe '#name' do
+    it 'should remove fluff words, remove periods,
+        replace spaces with dashes, suffix story_id' do
+
+      unparsed_name = "Deemphasize the Clue on the stimulus screen of multi-screen exercises."
+      story_id = '12345'
+      fake_story = double('PGit::Story', id: story_id, name: unparsed_name)
+
+      name_parser = PGit::StoryBranch::NameParser.new(fake_story)
+      parsed = name_parser.name
+
+      expect(parsed).to eq "deemphasize-clue-stimulus-screen-multiscreen-exercises-12345"
+    end
+
+    it 'should strip non-alpha-numerics like apostrophes' do
+      unparsed_name = "Some don't like putin's dictatorship"
+      story_id = '29292'
+
+      fake_story = double('PGit::Story', id: story_id, name: unparsed_name)
+
+      name_parser = PGit::StoryBranch::NameParser.new(fake_story)
+      parsed = name_parser.name
+
+      expect(parsed).to eq "some-dont-like-putins-dictatorship-29292"
+    end
+
+    it 'should remove all non-word characters' do
+      unparsed_name = "Some *@#   don't like ,putin's dictator-ship"
+      story_id = '29292'
+
+      fake_story = double('PGit::Story', id: story_id, name: unparsed_name)
+
+      name_parser = PGit::StoryBranch::NameParser.new(fake_story)
+      parsed = name_parser.name
+
+      expect(parsed).to eq "some-dont-like-putins-dictatorship-29292"
+    end
+  end
+end

data/spec/pgit/story_branch_spec.rb

@@ -0,0 +1,90 @@
+require 'pgit'
+
+describe 'PGit::StoryBranch' do
+  describe '#id' do
+    it 'should return the story_id' do
+      fake_parsed_branch_name = 'bring-me-da-coconut-1234'
+      fake_story_id = '1234'
+      name_parser = instance_double('PGit::StoryBranch::NameParser',
+                                    story_id: fake_story_id,
+                                    name: fake_parsed_branch_name)
+      allow(name_parser).to receive(:name).and_return(fake_parsed_branch_name)
+      story_branch = PGit::StoryBranch.new(name_parser)
+      story_branch_id = story_branch.id
+
+      expect(story_branch_id).to eq '1234'
+    end
+  end
+
+
+  describe '#start' do
+    it 'should call the backticks on the Kernel with the proper branch name' do
+      fake_parsed_json = { "name" => "Bring me the passengers",
+                           "id" => 555
+      }
+      fake_json_response = double('fake_json_response')
+      fake_story_id = 555
+      fake_story_name = "bring-me-passengers-555"
+      allow(JSON).to receive(:parse).with(fake_json_response).and_return(fake_parsed_json)
+      fake_name_parser = instance_double('PGit::StoryBranch::NameParser',
+                                         story_id: fake_story_id,
+                                         name: fake_story_name
+                                         )
+      checkout_branch_command = "git checkout -b bring-me-passengers-555"
+      pg_branch = PGit::StoryBranch.new(fake_name_parser)
+      allow(pg_branch).to receive(:`).with(checkout_branch_command)
+
+      pg_branch.start
+
+      expect(pg_branch).to have_received(:`).with(checkout_branch_command)
+    end
+  end
+
+  describe '#name' do
+    it 'should return the result of the parsed branch name' do
+      fake_parsed_json = { "name" => "Bring me the passengers",
+                           "id" => 555
+      }
+      fake_json_response = double('fake_json_response')
+      allow(JSON).to receive(:parse).with(fake_json_response).and_return(fake_parsed_json)
+      checkout_branch_command = "git checkout -b bring-me-passengers-555"
+      fake_parsed_branch_name = double('parsed_branch_name')
+      fake_name_parser = double('name_parser', name: fake_parsed_branch_name)
+
+
+      pg_branch = PGit::StoryBranch.new(fake_name_parser)
+
+      branch_name = pg_branch.name
+
+      expect(branch_name).to eq fake_parsed_branch_name
+    end
+  end
+
+  # describe '#new without any arguments' do
+    # describe 'on a branch without a story id' do
+      # describe '#id' do
+        # it 'should throw an error' do
+          # fake_current_branch_name = "master"
+          # error_message = "Error: #{fake_current_branch_name} does not have a story id at the end"
+          # allow(PGit::CurrentBranch).to receive(:name).and_return(fake_current_branch_name)
+#
+          # expect{PGit::StoryBranch.new}.to raise_error error_message
+        # end
+      # end
+    # end
+#
+    # describe 'on a branch with a story id' do
+      # describe '#id' do
+        # it 'should return the id' do
+          # fake_current_branch_name = "some-feature-branch-12345678"
+          # allow(PGit::CurrentBranch).to receive(:name).and_return(fake_current_branch_name)
+          # error_message = "Error: #{fake_current_branch_name} does not have a story id at the end"
+          # story_branch = PGit::StoryBranch.new
+          # story_branch_id = story_branch.id
+#
+          # expect(story_branch_id).to eq('12345678')
+        # end
+      # end
+    # end
+  # end
+end