pt-flow 0.2 → 0.2.1

data/spec/lib/pt-flow/ui_spec.rb

@@ -1,43 +1,94 @@
 require 'spec_helper'
 
-module PT::Flow
-  describe UI do
+describe PT::Flow::UI do
 
-    let(:project) { double('Project', id: 1, use_https: false) }
-    let(:task) { double('Task', id: 1, name: 'Do this', current_state: 'Unscheduled', estimate: 1, errors: []) }
-    let(:prompt) { double('HighLine') }
+  let(:endpoint) { "http://www.pivotaltracker.com/services/v3" }
+  let(:prompt) { double('HighLine') }
 
-    before do
-      PivotalTracker::Project.stub(all: [], find: project)
-      PivotalTracker::Story.stub(find: task)
-      HighLine.stub(new: prompt)
+  before do
+    HighLine.stub(new: prompt)
+    stub_request(:get, /projects$/).to_return(body: fixture_file('projects.xml'))
+    stub_request(:get, /stories\?/).to_return(body: fixture_file('stories.xml'))
+    stub_request(:any, /stories\/\d+/).to_return(body: fixture_file('story.xml'))
+  end
+
+  describe '#start' do
+    context 'given an unestimated story' do
+      it "shows lists of stories - choosing one asks you to estimate, starts/assigns the story on pt and checks out a new branch." do
+        prompt.should_receive(:ask).with("Please select a task to start working on (1-14, 'q' to exit)".bold).and_return('2')
+        prompt.should_receive(:ask).with("How many points do you estimate for it? (0,1,2,3)".bold).and_return('3')
+
+        PT::Flow::UI.new %w{ start }
+
+        WebMock.should have_requested(:get, "#{endpoint}/projects/102622/stories?filter=current_state:unscheduled,unstarted,started")
+        WebMock.should have_requested(:put, "#{endpoint}/projects/102622/stories/4459994").with(body: /<estimate>3<\/estimate>/)
+        WebMock.should have_requested(:put, "#{endpoint}/projects/102622/stories/4459994").with(body: /<owned_by>Jon Mischo<\/owned_by>/)
+        WebMock.should have_requested(:put, "#{endpoint}/projects/102622/stories/4459994").with(body: /<current_state>started<\/current_state>/)
+
+        current_branch.should == 'master-4459994'
+      end
     end
 
-    describe '#start' do
-      before do
-        project.stub(stories: double('stories', all: [task]))
+    context 'given an already estimated story' do
+      it "does not prompt to estimate" do
+        prompt.should_receive(:ask).once.with("Please select a task to start working on (1-14, 'q' to exit)".bold).and_return('3')
+        PT::Flow::UI.new %w{ start }
       end
+    end
 
-      context 'given no args' do
-        it "shows lists of tasks - choosing one starts/assigns the task on pt and checks out a new branch." do
-          prompt.should_receive(:ask).with("Please select a task to start working on (1-1, 'q' to exit)".bold).and_return('1')
-          task.should_receive(:update).with(owned_by: 'Jens Balvig').and_return(task)
-          task.should_receive(:update).with(current_state: 'started').and_return(task)
-          UI.any_instance.should_receive(:`).with('git checkout -B 1')
+    context 'when run from a feature branch' do
+      before { system('git checkout -B new_feature') }
 
-          ui = UI.new %w{ start }
-        end
+      it "creates an appropriately namespaced branch" do
+        prompt.should_receive(:ask).and_return('3')
+        PT::Flow::UI.new %w{ start }
+        current_branch.should == 'new_feature-4460038'
       end
-      context 'given a number' do
-        it "starts/assigns the task on pt and checks out a new branch." do
-          task.should_receive(:update).with(owned_by: 'Jens Balvig').and_return(task)
-          task.should_receive(:update).with(current_state: 'started').and_return(task)
-          UI.any_instance.should_receive(:`).with('git checkout -B 1')
-
-          ui = UI.new %w{ start 1 }
-        end
+    end
+
+    context 'when run from an existing story branch' do
+      before { system('git checkout -B new_feature-12345') }
+
+      it "creates a branch within the same namespace" do
+        prompt.should_receive(:ask).and_return('3')
+        PT::Flow::UI.new %w{ start }
+        current_branch.should == 'new_feature-4460038'
       end
     end
+  end
 
+  describe '#finish' do
+    before do
+      #TODO: Stubbed  endpoint ALWAYS returns story  4459994, need a way to check it is actually getting the right id from the branch
+      system('git checkout -B new_feature-4459994')
+      system('git remote add origin git@github.com:balvig/pt-flow.git')
+    end
+
+    it "pushes the current branch to origin, flags the story as finished, and opens a github pull request" do
+      PT::Flow::UI.any_instance.should_receive(:run).with('git push origin new_feature-4459994')
+      PT::Flow::UI.any_instance.should_receive(:run).with("open 'https://github.com/balvig/pt-flow/pull/new/new_feature...new_feature-4459994?title=Unestimated Feature [#4459994]&body=http://www.pivotaltracker.com/story/show/4459994'")
+      PT::Flow::UI.new %w{ finish }
+      WebMock.should have_requested(:put, "#{endpoint}/projects/102622/stories/4459994").with(body: /<current_state>finished<\/current_state>/)
+    end
+  end
+
+  describe '#deliver' do
+    before do
+      #TODO: Stubbed  endpoint ALWAYS returns story  4459994, need a way to check it is actually getting the right id from the branch
+      system('git checkout -B new_feature-4459994')
+    end
+
+    it "pushes the current branch to origin, flags the story as finished, and opens a github pull request" do
+      PT::Flow::UI.any_instance.should_receive(:run).with('git fetch')
+      PT::Flow::UI.any_instance.should_receive(:run).with('git checkout new_feature')
+      PT::Flow::UI.any_instance.should_receive(:run).with('git pull --rebase origin new_feature')
+      PT::Flow::UI.any_instance.should_receive(:run).with('git merge new_feature-4459994')
+      PT::Flow::UI.any_instance.should_receive(:run).with('git push origin new_feature')
+      PT::Flow::UI.any_instance.should_receive(:run).with('git push origin :new_feature-4459994')
+      PT::Flow::UI.any_instance.should_receive(:run).with('git branch -d new_feature-4459994')
+
+      PT::Flow::UI.new %w{ deliver }
+      WebMock.should have_requested(:put, "#{endpoint}/projects/102622/stories/4459994").with(body: /<current_state>delivered<\/current_state>/)
+    end
   end
 end

data/spec/spec_helper.rb

@@ -1,4 +1,3 @@
-require 'bundler'
 require 'pt-flow'
 
 PT::UI.send(:remove_const, :GLOBAL_CONFIG_PATH)
@@ -6,6 +5,8 @@ PT::UI.send(:remove_const, :LOCAL_CONFIG_PATH)
 PT::UI::GLOBAL_CONFIG_PATH = File.join(File.dirname(__FILE__), 'fixtures', 'global_config.yml')
 PT::UI::LOCAL_CONFIG_PATH = File.join(File.dirname(__FILE__), 'fixtures', 'local_config.yml')
 
+Dir[File.join(File.dirname(__FILE__),'support', '**', '*.rb')].each {|f| require f}
+
 RSpec.configure do |config|
   config.mock_with :rspec
 end

data/spec/support/dummy.rb

@@ -0,0 +1,14 @@
+RSpec.configure do |config|
+  dummy_path = File.join(File.dirname(__FILE__), '..', 'fixtures', 'dummy')
+
+  config.before(:each) do
+    FileUtils.rm_rf(dummy_path) if Dir.exists?(dummy_path)
+    Dir.mkdir(dummy_path)
+    Dir.chdir(dummy_path)
+    system('git init .')
+    system('touch dummy.txt')
+    system('git add .')
+    system('git commit -m"init commit"')
+  end
+
+end

data/spec/support/fixture_macros.rb

@@ -0,0 +1,13 @@
+module FixtureMacros
+  def fixture_file(filename)
+    File.new(fixture_file_path(filename))
+  end
+
+  def fixture_file_path(filename)
+    File.join(File.dirname(__FILE__), '..', 'fixtures', filename)
+  end
+end
+
+RSpec.configure do |config|
+  config.include(FixtureMacros)
+end

data/spec/support/git_macros.rb

@@ -0,0 +1,9 @@
+module GitMacros
+  def current_branch
+    `git rev-parse --abbrev-ref HEAD`.strip
+  end
+end
+
+RSpec.configure do |config|
+  config.include(GitMacros)
+end

data/spec/support/webmock.rb

@@ -0,0 +1,5 @@
+require 'webmock/rspec'
+
+class Pivotal
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pt-flow
 version: !ruby/object:Gem::Version
-  version: '0.2'
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pt
-  requirement: &70237517292560 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70237517292560
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70237509107220 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +37,31 @@ dependencies:
         version: '2.9'
   type: :development
   prerelease: false
-  version_requirements: *70237509107220
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.9'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: guard
-  requirement: &70237509108900 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +69,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70237509108900
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: &70237513253180 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +85,12 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70237513253180
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Some extra methods for the pt gem to use in our dev flow.
 email:
 - jens@balvig.com
@@ -74,10 +110,38 @@ files:
 - lib/pt-flow/ui.rb
 - lib/pt-flow/version.rb
 - pt-flow.gemspec
+- spec/fixtures/activity.xml
+- spec/fixtures/backup_stories.xml
+- spec/fixtures/bugs.xml
+- spec/fixtures/created_note.xml
+- spec/fixtures/created_story.xml
+- spec/fixtures/features.xml
 - spec/fixtures/global_config.yml
+- spec/fixtures/iterations_all.xml
+- spec/fixtures/iterations_backlog.xml
+- spec/fixtures/iterations_current.xml
+- spec/fixtures/iterations_current_backlog.xml
+- spec/fixtures/iterations_done.xml
 - spec/fixtures/local_config.yml
+- spec/fixtures/memberships.xml
+- spec/fixtures/notes.xml
+- spec/fixtures/project.xml
+- spec/fixtures/project_activity.xml
+- spec/fixtures/projects.xml
+- spec/fixtures/stories.xml
+- spec/fixtures/story-4459994.xml
+- spec/fixtures/story-4460038.xml
+- spec/fixtures/story-4460598.xml
+- spec/fixtures/story-4473735.xml
+- spec/fixtures/story.xml
+- spec/fixtures/tasks.xml
+- spec/fixtures/update_tasks.xml
 - spec/lib/pt-flow/ui_spec.rb
 - spec/spec_helper.rb
+- spec/support/dummy.rb
+- spec/support/fixture_macros.rb
+- spec/support/git_macros.rb
+- spec/support/webmock.rb
 homepage: ''
 licenses: []
 post_install_message: 
@@ -92,7 +156,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3947336664115380543
+      hash: -3508141726274605870
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -101,15 +165,43 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3947336664115380543
+      hash: -3508141726274605870
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.11
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
 summary: Some extra methods for the pt gem to use in our dev flow.
 test_files:
+- spec/fixtures/activity.xml
+- spec/fixtures/backup_stories.xml
+- spec/fixtures/bugs.xml
+- spec/fixtures/created_note.xml
+- spec/fixtures/created_story.xml
+- spec/fixtures/features.xml
 - spec/fixtures/global_config.yml
+- spec/fixtures/iterations_all.xml
+- spec/fixtures/iterations_backlog.xml
+- spec/fixtures/iterations_current.xml
+- spec/fixtures/iterations_current_backlog.xml
+- spec/fixtures/iterations_done.xml
 - spec/fixtures/local_config.yml
+- spec/fixtures/memberships.xml
+- spec/fixtures/notes.xml
+- spec/fixtures/project.xml
+- spec/fixtures/project_activity.xml
+- spec/fixtures/projects.xml
+- spec/fixtures/stories.xml
+- spec/fixtures/story-4459994.xml
+- spec/fixtures/story-4460038.xml
+- spec/fixtures/story-4460598.xml
+- spec/fixtures/story-4473735.xml
+- spec/fixtures/story.xml
+- spec/fixtures/tasks.xml
+- spec/fixtures/update_tasks.xml
 - spec/lib/pt-flow/ui_spec.rb
 - spec/spec_helper.rb
+- spec/support/dummy.rb
+- spec/support/fixture_macros.rb
+- spec/support/git_macros.rb
+- spec/support/webmock.rb