[#2] assign pivotal story if starting it.

inaka · May 29, 2015 · 90abc40 · 90abc40
1 parent 233a2f9
commit 90abc40
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/lib/assisted_workflow/addons/pivotal.rb b/lib/assisted_workflow/addons/pivotal.rb
@@ -43,6 +43,12 @@ def start_story(story, options = {})
       log "starting story ##{story.id}"
       options.delete(:estimate) if options[:estimate].nil?
       update_story!(story, options.merge(:current_state => "started"))
+
+      owner_ids = story.owner_ids
+      if owner_ids.empty? || !owner_ids.include?(@client.me.id)
+        log "assigning story ##{story.id}"
+        update_story!(story, :owner_ids => owner_ids.dup << @client.me.id)
+      end
     end
 
     def finish_story(story, options = {})

diff --git a/spec/assisted_workflow/addons/pivotal_spec.rb b/spec/assisted_workflow/addons/pivotal_spec.rb
@@ -15,6 +15,7 @@
     stub(TrackerApi::Client).new{ @client }
     @project = TrackerApi::Resources::Project.new(client: @client, id: 1)
     stub(@client).project(@configuration["project_id"]){ @project }
+    stub(@client).me{ TrackerApi::Resources::Me.new }
 
     @pivotal = AssistedWorkflow::Addons::Pivotal.new(nil, @configuration)
   end
@@ -66,7 +67,6 @@
         story_stub(:id => "100002", :project_id => @project.id)
       ]
     end
-    stub(@client).me{ TrackerApi::Resources::Me.new }
 
     stories = @pivotal.pending_stories(:include_started => true)
     stories.size.must_equal 2