git-pivotal-tracker-centro 1.0.0.rc1

data/spec/commands/base_spec.rb

@@ -0,0 +1,137 @@
+require 'spec_helper'
+
+describe Commands::Base do
+  
+  before(:each) do
+    @input = mock('input')
+    @output = mock('output')
+    
+    # stub out git config requests
+    Commands::Base.any_instance.stubs(:get).with { |v| v =~ /git config/ }.returns("")
+  end
+  
+  it "should set the api key with the -k option" do
+    @pick = Commands::Base.new("-k", "1234").with(@input, @output)
+    @pick.options[:api_token].should == "1234"
+  end
+  
+  it "should set the api key with the --api-token= option" do
+    @pick = Commands::Base.new("--api-key=1234").with(@input, @output)
+    @pick.options[:api_token].should == "1234"
+  end
+  
+  it "should set the project id with the -p option" do
+    @pick = Commands::Base.new("-p", "1").with(@input, @output)
+    @pick.options[:project_id].should == "1"
+  end
+  
+  it "should set the project id with the --project-id= option" do
+    @pick = Commands::Base.new("--project-id=1").with(@input, @output)
+    @pick.options[:project_id].should == "1"
+  end
+  
+  it "should set the full name with the -n option" do
+    @pick = Commands::Base.new("-n", "Jeff Tucker").with(@input, @output)
+    @pick.options[:full_name].should == "Jeff Tucker"
+  end
+  
+  it "should set the full name with the --full-name= option" do
+    @pick = Commands::Base.new(@input, @output,"--full-name=Jeff Tucker")
+    @pick.options[:full_name].should == "Jeff Tucker"
+  end
+  
+  it "should set the quiet flag with the -q option" do
+    @pick = Commands::Base.new("-q").with(@input, @output)
+    @pick.options[:quiet].should be_true
+  end
+  
+  it "should set the quiet flag with the --quiet option" do
+    @pick = Commands::Base.new("--quiet").with(@input, @output)
+    @pick.options[:quiet].should be_true
+  end
+  
+  it "should set the force flag with the -f option" do
+    @pick = Commands::Base.new("-f").with(@input, @output)
+    @pick.options[:force].should be_true
+  end
+
+  it "should set the force flag with the --force option" do
+    @pick = Commands::Base.new("--force").with(@input, @output)
+    @pick.options[:force].should be_true
+  end
+
+  it "should set the verbose flag with the -v option" do
+    @pick = Commands::Base.new("-v").with(@input, @output)
+    @pick.options[:verbose].should be_true
+  end
+  
+  it "should set the verbose flag with the --verbose option" do
+    @pick = Commands::Base.new("--verbose").with(@input, @output)
+    @pick.options[:verbose].should be_true
+  end
+  
+  it "should unset the verbose flag with the --no-verbose option" do
+    @pick = Commands::Base.new("--no-verbose").with(@input, @output)
+    @pick.options[:verbose].should be_false
+  end
+  
+  it "should respect verbose from git config if it's set true (case insensitive)" do
+    Commands::Base.any_instance.stubs(:get).with("git config --get pivotal.verbose").returns("truE")
+    @pick = Commands::Base.new
+    @pick.options[:verbose].should be_true
+  end
+
+  it "should respect verbose from git config if it's set true (case insensitive)" do
+    Commands::Base.any_instance.stubs(:get).with("git config --get pivotal.verbose").returns("falSe")
+    @pick = Commands::Base.new
+    @pick.options[:verbose].should be_false
+  end
+
+  it "should be verbose by default" do
+    Commands::Base.any_instance.stubs(:get).with("git config --get pivotal.verbose").returns("")
+    @pick = Commands::Base.new
+    @pick.options[:verbose].should be_true
+  end
+
+  it "should print a message if the API token is missing" do
+    @output.expects(:print).with("Pivotal Tracker API Token and Project ID are required\n")
+
+    @pick = Commands::Base.new("-p", "1").with(@input, @output)
+    @pick.run!
+  end
+  
+  it "should print a message if the project ID is missing" do
+    @output.expects(:print).with("Pivotal Tracker API Token and Project ID are required\n")
+
+    @pick = Commands::Base.new("-k", "1").with(@input, @output)
+    @pick.run!    
+  end
+  
+  it "should set use_ssl to true with --use-ssl" do
+    @pick = Commands::Base.new("--use-ssl").with(@input, @output)
+    @pick.options[:use_ssl].should be_true
+  end
+
+  it "should set use_ssl to true with -S" do
+    @pick = Commands::Base.new("-S").with(@input, @output)
+    @pick.options[:use_ssl].should be_true
+  end
+
+  it "should set use_ssl to false by default" do
+    @pick = Commands::Base.new("").with(@input, @output)
+    @pick.options[:use_ssl].should be_false
+  end
+
+  it "should respect use_ssl from git config if it's set true (case insensitive)" do
+    Commands::Base.any_instance.stubs(:get).with("git config --get pivotal.use-ssl").returns("truE")
+    @pick = Commands::Base.new
+    @pick.options[:use_ssl].should be_true
+  end
+
+  it "should respect use_ssl from git config if it's set to anything but true" do
+    Commands::Base.any_instance.stubs(:get).with("git config --get pivotal.use-ssl").returns("not true")
+    @pick = Commands::Base.new
+    @pick.options[:use_ssl].should be_false
+  end
+
+end

data/spec/commands/bug_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Commands::Bug do
+
+  before(:each) do
+    # stub out git config requests
+    Commands::Bug.any_instance.stubs(:get).with { |v| v =~ /git config/ }.returns("")
+
+    @bug = Commands::Bug.new
+  end
+  
+  it "should specify its story type" do
+    @bug.type.should == "bug"
+  end
+  
+  it "should specify a plural for its story types" do
+    @bug.plural_type.should == "bugs"
+  end
+  
+  it "should specify its branch suffix" do
+    @bug.branch_suffix.should == "bugfix"
+  end
+  
+end

data/spec/commands/chore_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Commands::Chore do
+
+  before(:each) do
+    # stub out git config requests
+    Commands::Chore.any_instance.stubs(:get).with { |v| v =~ /git config/ }.returns("")
+
+    @chore = Commands::Chore.new
+  end
+  
+  it "should specify its story type" do
+    @chore.type.should == "chore"
+  end
+  
+  it "should specify a plural for its story types" do
+    @chore.plural_type.should == "chores"
+  end
+  
+  it "should specify its branch suffix" do
+    @chore.branch_suffix.should == "chore"
+  end
+  
+end

data/spec/commands/feature_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Commands::Feature do
+
+  before(:each) do
+    # stub out git config requests
+    Commands::Feature.any_instance.stubs(:get).with { |v| v =~ /git config/ }.returns("")
+
+    @feature = Commands::Feature.new
+  end
+  
+  it "should specify its story type" do
+    @feature.type.should == "feature"
+  end
+  
+  it "should specify a plural for its story types" do
+    @feature.plural_type.should == "features"
+  end
+  
+  it "should specify its branch suffix" do
+    @feature.branch_suffix.should == "feature"
+  end
+  
+end

data/spec/commands/finish_spec.rb

@@ -0,0 +1,134 @@
+require 'spec_helper'
+
+describe Commands::Finish do
+
+  def mock_project_id
+    @mock_project_id ||= '4321'
+  end
+
+  def mock_projects
+    if @mock_projects.nil?
+      @mock_projects = mock("mock project")
+      @mock_projects.stubs(:stories).returns(mock_stories)
+    end
+    @mock_projects
+  end
+
+  def mock_stories
+    if @mock_stories.nil?
+      @mock_stories = mock("mock story list")
+      @mock_stories.stubs(:find).with(mock_story_id).returns(mock_story)
+    end
+    @mock_stories
+  end
+
+  def mock_story_id
+    @mock_story_id ||= 1234
+  end
+
+  def mock_story
+    if @mock_story.nil?
+      @mock_story = mock("mock story")
+    end
+    @mock_story
+  end
+
+  def branch_name
+    "#{mock_story_id}-feature-branch-name"
+  end
+
+  before(:each) do
+    # stub out git config requests
+    Commands::Finish.any_instance.stubs(:get).with { |v| v =~ /git config/ }.returns("")
+
+    PivotalTracker::Project.stubs(:find).returns(mock_projects)
+
+    @finish = Commands::Finish.new(nil, nil, "-p", mock_project_id)
+    @finish.stubs(:put)
+    @finish.stubs(:get_char).returns('y')
+  end
+
+  context "where the branch name does not contain a valid story id" do
+    before(:each) do
+      # stub out git status request to identify the branch
+      branch_name = "invalid-branch-name-without-story-id"
+      Commands::Finish.any_instance.stubs(:get).with { |v| v =~ /git status/ }.returns("# On branch #{branch_name}")
+      Commands::Finish.any_instance.stubs(:get).with { |v| v == "git symbolic-ref HEAD" }.returns(branch_name)
+    end
+
+    it "should return an exit status of one" do
+      @finish.run!.should == 1
+    end
+
+    it "should print an error message" do
+      @finish.expects(:put).with("Branch name must contain a Pivotal Tracker story id").once
+      @finish.run!
+    end
+  end
+
+  context "where the branch name does contain a valid story id" do
+    before(:each) do
+      # stub out git status request to identify the branch
+      Commands::Finish.any_instance.stubs(:get).with { |v| v =~ /git status/ }.returns("# On branch #{branch_name}")
+      Commands::Finish.any_instance.stubs(:get).with { |v| v == "git symbolic-ref HEAD" }.returns(branch_name)
+    end
+
+    it "should exit if the user does not wish to proceed" do
+      @finish.stubs(:get_char).returns('n')
+      @finish.expects(:sys).never
+      @finish.run!
+    end
+
+    it "should checkout the master branch, delete the local branch, and delete the remote branch" do
+      @finish.expects(:sys).with() { |value| value == "git checkout master" }
+      @finish.expects(:sys).with("git branch -d #{branch_name}").returns(true)
+      @finish.expects(:sys).with("git push origin :#{branch_name}")
+      mock_story.stubs(:story_type).returns("finished")
+      mock_story.stubs(:update)
+      @finish.run!
+    end
+
+    it "should attempt to update the story status to the stories finished_state" do
+      @finish.stubs(:sys).returns(true)
+      mock_story.stubs(:story_type).returns("finished")
+      mock_story.expects(:update).with(:current_state => "finished")
+      @finish.run!
+    end
+
+    it "should not delete the remote branch and update the story if the local branch could not be deleted" do
+      @finish.expects(:sys).with() { |value| value == "git checkout master" }
+      @finish.expects(:sys).with("git branch -d #{branch_name}").returns(false)
+      @finish.expects(:sys).with("git push origin :#{branch_name}").never
+      @finish.run!
+    end
+
+    context "and the story is successfully marked as finished in PT" do
+      before(:each) do
+        @finish.stubs(:sys).returns(true)
+        mock_story.stubs(:story_type).returns("finished")
+        mock_story.stubs(:update).with(:current_state => "finished").returns(true)
+      end
+
+      it "should succeed with an exist status of zero" do
+        @finish.run!.should == 0
+      end
+    end
+
+    context "and the story fails to be marked as finished in PT" do
+      before(:each) do
+        @finish.stubs(:sys).returns(true)
+        mock_story.stubs(:story_type).returns("finished")
+        mock_story.stubs(:update).with(:current_state => "finished").returns(false)
+      end
+
+      it "should fail with an exist status of one" do
+        @finish.run!.should == 1
+      end
+
+      it "should print an error message" do
+        @finish.expects(:put).with("Unable to mark Story #{mock_story_id} as finished").once
+        @finish.run!
+      end
+    end
+  end
+end

data/spec/commands/map_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe Commands::Map do
+  
+  describe "#[]" do
+    it "retrieves values when a specific key matches a regular expression" do
+      subject[/^\d+$/] = "w00t"
+      subject["1234"].should eq("w00t")
+      subject["9123423423"].should eq("w00t")
+      subject["a9123423423"].should be_nil
+      subject["1234f"].should be_nil
+    end
+  end
+end

data/spec/commands/start_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe Commands::Start do
+
+  describe '.for' do
+    it "returns the Command::Bug when the first argument is bug" do
+      Commands::Start.for("bug").should be_instance_of(Commands::Bug)
+    end
+
+    it "returns the Command::Chore when the first argument is chore" do
+      Commands::Start.for("chore").should be_instance_of(Commands::Chore)
+    end
+
+    it "returns the Command::Feature when the first argument is feature" do
+      Commands::Start.for("feature").should be_instance_of(Commands::Feature)
+    end
+
+    it "returns the Command::Card when the first argument is a card id" do
+      Commands::Start.for("123456").should be_instance_of(Commands::Card)
+    end
+
+    it "raises when the command is unknown card type" do
+      Commands::Start.expects(:display_usage_instructions_and_quit)
+      Commands::Start.for("unknown")
+    end
+  end
+  
+end

data/spec/factories.rb

@@ -0,0 +1,13 @@
+require File.join(File.dirname(__FILE__), 'factory')
+
+Factory.define :story, {
+  :id            => 1,
+  :current_state => :unstarted,
+  :story_type    => :feature,
+  :estimate      => 1,
+}
+
+Factory.define :project, {
+  :id   => 1,
+  :name => "Project"
+}

data/spec/factory.rb

@@ -0,0 +1,26 @@
+class Factory
+  class << self
+    def factories
+      @@factories ||= {}
+    end
+    
+    def define(type, attributes)
+      factories[type] = attributes
+    end
+  end
+end
+
+
+def Factory(type, attrs = {})
+  defaults = Factory.factories[type]
+  attrs = defaults.merge(attrs)
+  
+  markup = Builder::XmlMarkup.new
+  markup.__send__(type) do
+    attrs.each do |attribute, value|
+      markup.__send__(attribute, value.to_s)
+    end
+  end
+  
+  markup.target!
+end

data/spec/spec_helper.rb

@@ -0,0 +1,24 @@
+require 'mocha'
+require 'builder'
+
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# Require this file using `require "spec_helper.rb"` to ensure that it is only
+# loaded once.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+RSpec.configure do |config|
+  config.mock_with :mocha
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
+end
+
+specdir = File.expand_path(File.join(File.dirname(__FILE__), '..'))
+require File.join(specdir, 'lib','git-pivotal-tracker')
+
+require File.join(File.dirname(__FILE__), 'factories')
+
+def stub_connection_to_pivotal
+  RestClient::Resource.any_instance.stubs(:get).returns("")
+end