pivotal-to-trello 0.1.2 → 0.2.0

data/lib/pivotal_to_trello/pivotal_wrapper.rb

@@ -3,32 +3,31 @@ require 'pivotal-tracker'
 module PivotalToTrello
   # Interface to the Pivotal Tracker API.
   class PivotalWrapper
-
     # Constructor
     def initialize(token)
       ::PivotalTracker::Client.token = token
+      ::PivotalTracker::Client.use_ssl = true
     end
 
     # Returns a hash of available projects keyed on project ID.
     def project_choices
-      ::PivotalTracker::Project.all.inject({}) do |hash, project|
+      ::PivotalTracker::Project.all.each_with_object({}) do |project, hash|
         hash[project.id] = project.name
-        hash
+
       end
     end
 
     # Returns all stories for the given project.
     def stories(project_id)
-      project(project_id).stories.all
+      project(project_id).stories.all.sort_by(&:created_at)
     end
 
     private
 
-      # Returns the Pivotal project that we're exporting.
-      def project(project_id)
-        @projects             ||= {}
-        @projects[project_id] ||= ::PivotalTracker::Project.find(project_id)
-      end
-
+    # Returns the Pivotal project that we're exporting.
+    def project(project_id)
+      @projects             ||= {}
+      @projects[project_id] ||= ::PivotalTracker::Project.find(project_id)
+    end
   end
-end
+end

data/lib/pivotal_to_trello/trello_wrapper.rb

@@ -3,7 +3,6 @@ require 'trello'
 module PivotalToTrello
   # Interface to the Trello API.
   class TrelloWrapper
-
     # Constructor
     def initialize(key, token)
       Trello.configure do |config|
@@ -18,15 +17,13 @@ module PivotalToTrello
       card ||= begin
         puts "Creating a card for #{pivotal_story.story_type} '#{pivotal_story.name}'."
         card = Trello::Card.create(
-          :name    => pivotal_story.name,
-          :desc    => pivotal_story.description,
-          :list_id => list_id
+          name:    pivotal_story.name,
+          desc:    pivotal_story.description,
+          list_id: list_id,
         )
 
-        pivotal_story.notes.all.each do |note|
-          card.add_comment("[#{note.author}] #{note.text.to_s.strip}") unless note.text.to_s.strip.empty?
-        end
-
+        create_notes(card, pivotal_story)
+        create_tasks(card, pivotal_story)
         card
       end
 
@@ -38,9 +35,8 @@ module PivotalToTrello
 
     # Returns a hash of available boards, keyed on board ID.
     def board_choices
-      Trello::Board.all.inject({}) do |hash, board|
+      Trello::Board.all.each_with_object({}) do |board, hash|
         hash[board.id] = board.name
-        hash
       end
     end
 
@@ -49,9 +45,8 @@ module PivotalToTrello
       # Cache the list to improve performance.
       @lists           ||= {}
       @lists[board_id] ||= begin
-        choices = Trello::Board.find(board_id).lists.inject({}) do |hash, list|
+        choices = Trello::Board.find(board_id).lists.each_with_object({}) do |list, hash|
           hash[list.id] = list.name
-          hash
         end
         choices        = Hash[choices.sort_by { |_, v| v }]
         choices[false] = "[don't import these stories]"
@@ -64,17 +59,21 @@ module PivotalToTrello
     # Returns a list of all cards in the given list, keyed on name.
     def cards_for_list(list_id)
       @cards          ||= {}
-      @cards[list_id] ||= Trello::List.find(list_id).cards.inject({}) do |hash, card|
+      @cards[list_id] ||= Trello::List.find(list_id).cards.each_with_object({}) do |card, hash|
         hash[card_hash(card.name, card.desc)] = card
-        hash
       end
 
       @cards[list_id]
     end
 
     # Adds the given label to the card.
-    def add_label(card, label)
-      card.add_label(label) unless card.labels.collect { |label| label.color }.include?(label)
+    def add_label(card, label_name, label_color)
+      @labels                ||= {}
+      @labels[card.board_id] ||= Trello::Board.find(card.board_id).labels
+      label                    = @labels[card.board_id].find { |l| l.name == label_name && l.color == label_color }
+      label                  ||= Trello::Label.create(name: label_name, board_id: card.board_id, color: label_color)
+
+      card.add_label(label) unless card.labels.detect { |l| l.id == label.id }
     end
 
     # Returns a list of colors that can be used to label cards.
@@ -86,22 +85,42 @@ module PivotalToTrello
         'purple' => 'Purple',
         'red'    => 'Red',
         'yellow' => 'Yellow',
-        false    => '[none]'
+        false    => '[none]',
       }
     end
 
     private
 
-      # Returns a unique identifier for this list/name/description combination.
-      def card_hash(name, description)
-        Digest::SHA1.hexdigest("#{name}_#{description}")
+    # Copies notes from the pivotal story to the card.
+    def create_notes(card, pivotal_story)
+      pivotal_story.notes.all.each do |note|
+        card.add_comment("[#{note.author}] #{note.text.to_s.strip}") unless note.text.to_s.strip.empty?
       end
+    end
+
+    # Copies notes from the pivotal story to the card.
+    def create_tasks(card, pivotal_story)
+      tasks = pivotal_story.tasks.all
+      return if tasks.empty?
 
-      # Returns a card with the given name and description if it exists in the given list, nil otherwise.
-      def get_card(list_id, name, description)
-        key = card_hash(name, description)
-        cards_for_list(list_id)[key] if !cards_for_list(list_id)[key].nil?
+      checklist = Trello::Checklist.create(name: 'Tasks', card_id: card.id)
+      card.add_checklist(checklist)
+
+      tasks.each do |task|
+        puts " - Creating task '#{task.description}'"
+        checklist.add_item(task.description, task.complete)
       end
+    end
 
+    # Returns a unique identifier for this list/name/description combination.
+    def card_hash(name, description)
+      Digest::SHA1.hexdigest("#{name}_#{description}")
+    end
+
+    # Returns a card with the given name and description if it exists in the given list, nil otherwise.
+    def get_card(list_id, name, description)
+      key = card_hash(name, description)
+      cards_for_list(list_id)[key] unless cards_for_list(list_id)[key].nil?
+    end
   end
-end
+end

data/pivotal-to-trello.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: pivotal-to-trello 0.1.2 ruby lib
+# stub: pivotal-to-trello 0.2.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "pivotal-to-trello"
-  s.version = "0.1.2"
+  s.version = "0.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
-  s.authors = ["Dave Perrett"]
-  s.date = "2014-01-13"
+  s.authors = ["Dave Perrett", "Erik Frederiksen", "Kenneth Kalmer"]
+  s.date = "2017-08-10"
   s.description = "Pulls stories from Pivotal Tracker and imports them into Trello"
   s.email = "hello@daveperrett.com"
   s.executables = ["pivotal-to-trello"]
@@ -21,6 +21,8 @@ Gem::Specification.new do |s|
   ]
   s.files = [
     ".document",
+    ".rubocop.yml",
+    "Dockerfile",
     "Gemfile",
     "LICENSE.txt",
     "README.markdown",
@@ -40,34 +42,37 @@ Gem::Specification.new do |s|
   ]
   s.homepage = "http://github.com/recurser/pivotal-to-trello"
   s.licenses = ["MIT"]
-  s.rubygems_version = "2.2.1"
+  s.rubygems_version = "2.4.5"
   s.summary = "Pivotal Tracker to Trello exporter"
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<highline>, [">= 1.6.20"])
-      s.add_runtime_dependency(%q<ruby-trello>, [">= 1.1.0"])
-      s.add_runtime_dependency(%q<pivotal-tracker>, [">= 0.5.12"])
+      s.add_runtime_dependency(%q<highline>, ["~> 1.7.8"])
+      s.add_runtime_dependency(%q<ruby-trello>, ["~> 2.0.0"])
+      s.add_runtime_dependency(%q<pivotal-tracker>, ["~> 0.5.13"])
       s.add_development_dependency(%q<rspec>, [">= 2.14.1"])
       s.add_development_dependency(%q<rdoc>, [">= 4.1.1"])
-      s.add_development_dependency(%q<jeweler>, [">= 2.0.0"])
+      s.add_development_dependency(%q<jeweler>, [">= 2.3.7"])
+      s.add_development_dependency(%q<rubocop>, [">= 0.49.1"])
     else
-      s.add_dependency(%q<highline>, [">= 1.6.20"])
-      s.add_dependency(%q<ruby-trello>, [">= 1.1.0"])
-      s.add_dependency(%q<pivotal-tracker>, [">= 0.5.12"])
+      s.add_dependency(%q<highline>, ["~> 1.7.8"])
+      s.add_dependency(%q<ruby-trello>, ["~> 2.0.0"])
+      s.add_dependency(%q<pivotal-tracker>, ["~> 0.5.13"])
       s.add_dependency(%q<rspec>, [">= 2.14.1"])
       s.add_dependency(%q<rdoc>, [">= 4.1.1"])
-      s.add_dependency(%q<jeweler>, [">= 2.0.0"])
+      s.add_dependency(%q<jeweler>, [">= 2.3.7"])
+      s.add_dependency(%q<rubocop>, [">= 0.49.1"])
     end
   else
-    s.add_dependency(%q<highline>, [">= 1.6.20"])
-    s.add_dependency(%q<ruby-trello>, [">= 1.1.0"])
-    s.add_dependency(%q<pivotal-tracker>, [">= 0.5.12"])
+    s.add_dependency(%q<highline>, ["~> 1.7.8"])
+    s.add_dependency(%q<ruby-trello>, ["~> 2.0.0"])
+    s.add_dependency(%q<pivotal-tracker>, ["~> 0.5.13"])
     s.add_dependency(%q<rspec>, [">= 2.14.1"])
     s.add_dependency(%q<rdoc>, [">= 4.1.1"])
-    s.add_dependency(%q<jeweler>, [">= 2.0.0"])
+    s.add_dependency(%q<jeweler>, [">= 2.3.7"])
+    s.add_dependency(%q<rubocop>, [">= 0.49.1"])
   end
 end
 

data/spec/pivotal_to_trello/core_spec.rb

@@ -6,32 +6,32 @@ describe 'Core' do
   let(:trello)  { mock_trello_wrapper }
 
   before(:each) do
-    IO.any_instance.stub(:puts)
-    core.stub(:pivotal => pivotal)
-    core.stub(:trello  => trello)
-
-    core.stub(:prompt_selection).with('Which Pivotal project would you like to export?', pivotal.project_choices).and_return('pivotal_project_id')
-    core.stub(:prompt_selection).with('Which Trello board would you like to import into?', trello.board_choices).and_return('trello_board_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'icebox' stories into?", trello.list_choices).and_return('icebox_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'current' stories into?", trello.list_choices).and_return('current_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'finished' stories into?", trello.list_choices).and_return('finished_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'delivered' stories into?", trello.list_choices).and_return('delivered_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'accepted' stories into?", trello.list_choices).and_return('accepted_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'rejected' stories into?", trello.list_choices).and_return('rejected_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'backlog' bugs into?", trello.list_choices).and_return('bug_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'backlog' chores into?", trello.list_choices).and_return('chore_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'backlog' features into?", trello.list_choices).and_return('feature_list_id')
-    core.stub(:prompt_selection).with("Which Trello list would you like to put 'backlog' releases into?", trello.list_choices).and_return('release_list_id')
-    core.stub(:prompt_selection).with('What color would you like to label bugs with?', trello.label_choices).and_return('bug_label')
-    core.stub(:prompt_selection).with('What color would you like to label features with?', trello.label_choices).and_return('feature_label')
-    core.stub(:prompt_selection).with('What color would you like to label chores with?', trello.label_choices).and_return('chore_label')
-    core.stub(:prompt_selection).with('What color would you like to label releases with?', trello.label_choices).and_return('release_label')
+    allow_any_instance_of(IO).to receive(:puts)
+    allow(core).to receive_messages(pivotal: pivotal)
+    allow(core).to receive_messages(trello: trello)
+
+    allow(core).to receive(:prompt_selection).with('Which Pivotal project would you like to export?', pivotal.project_choices).and_return('pivotal_project_id')
+    allow(core).to receive(:prompt_selection).with('Which Trello board would you like to import into?', trello.board_choices).and_return('trello_board_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'icebox' stories into?", trello.list_choices).and_return('icebox_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'current' stories into?", trello.list_choices).and_return('current_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'finished' stories into?", trello.list_choices).and_return('finished_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'delivered' stories into?", trello.list_choices).and_return('delivered_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'accepted' stories into?", trello.list_choices).and_return('accepted_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'rejected' stories into?", trello.list_choices).and_return('rejected_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'backlog' bugs into?", trello.list_choices).and_return('bug_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'backlog' chores into?", trello.list_choices).and_return('chore_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'backlog' features into?", trello.list_choices).and_return('feature_list_id')
+    allow(core).to receive(:prompt_selection).with("Which Trello list would you like to put 'backlog' releases into?", trello.list_choices).and_return('release_list_id')
+    allow(core).to receive(:prompt_selection).with('What color would you like to label bugs with?', trello.label_choices).and_return('bug_label')
+    allow(core).to receive(:prompt_selection).with('What color would you like to label features with?', trello.label_choices).and_return('feature_label')
+    allow(core).to receive(:prompt_selection).with('What color would you like to label chores with?', trello.label_choices).and_return('chore_label')
+    allow(core).to receive(:prompt_selection).with('What color would you like to label releases with?', trello.label_choices).and_return('release_label')
   end
 
   context '#import!' do
     it 'prompts the user for details' do
       core.import!
-      core.options.should == mock_options
+      expect(core.options).to eq(mock_options)
     end
 
     describe 'story handling' do
@@ -39,68 +39,68 @@ describe 'Core' do
       let(:story) { mock_pivotal_story }
 
       before(:each) do
-        pivotal.should_receive(:stories).and_return([story])
-        trello.stub(:add_label => true, :create_card => card)
+        expect(pivotal).to receive(:stories).and_return([story])
+        allow(trello).to receive_messages(add_label: true, create_card: card)
       end
 
       it 'handles accepted stories' do
-        story.stub(:current_state => 'accepted')
-        trello.should_receive(:create_card).with(core.options.accepted_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'accepted')
+        expect(trello).to receive(:create_card).with(core.options.accepted_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles rejected stories' do
-        story.stub(:current_state => 'rejected')
-        trello.should_receive(:create_card).with(core.options.rejected_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'rejected')
+        expect(trello).to receive(:create_card).with(core.options.rejected_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles finished stories' do
-        story.stub(:current_state => 'finished')
-        trello.should_receive(:create_card).with(core.options.finished_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'finished')
+        expect(trello).to receive(:create_card).with(core.options.finished_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles delivered stories' do
-        story.stub(:current_state => 'delivered')
-        trello.should_receive(:create_card).with(core.options.delivered_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'delivered')
+        expect(trello).to receive(:create_card).with(core.options.delivered_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles unstarted features' do
-        story.stub(:current_state => 'unstarted', :story_type => 'feature')
-        trello.should_receive(:create_card).with(core.options.feature_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'unstarted', story_type: 'feature')
+        expect(trello).to receive(:create_card).with(core.options.feature_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles unstarted chores' do
-        story.stub(:current_state => 'unstarted', :story_type => 'chore')
-        trello.should_receive(:create_card).with(core.options.chore_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'unstarted', story_type: 'chore')
+        expect(trello).to receive(:create_card).with(core.options.chore_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles unstarted bugs' do
-        story.stub(:current_state => 'unstarted', :story_type => 'bug')
-        trello.should_receive(:create_card).with(core.options.bug_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'unstarted', story_type: 'bug')
+        expect(trello).to receive(:create_card).with(core.options.bug_list_id, story).and_return(card)
         core.import!
       end
 
       it 'handles unstarted releases' do
-        story.stub(:current_state => 'unstarted', :story_type => 'release')
-        trello.should_receive(:create_card).with(core.options.release_list_id, story).and_return(card)
+        allow(story).to receive_messages(current_state: 'unstarted', story_type: 'release')
+        expect(trello).to receive(:create_card).with(core.options.release_list_id, story).and_return(card)
         core.import!
       end
 
       it 'labels stories' do
-        story.stub(:story_type => 'bug')
-        trello.should_receive(:add_label).with(card, core.options.bug_label)
+        allow(story).to receive_messages(story_type: 'bug')
+        expect(trello).to receive(:add_label).with(card, 'bug', core.options.bug_label)
         core.import!
       end
 
       it 'ignores nil labels' do
-        core.options.stub(:bug_label => nil)
-        story.stub(:story_type => 'bug')
-        trello.should_not_receive(:add_label)
+        allow(core.options).to receive_messages(bug_label: nil)
+        allow(story).to receive_messages(story_type: 'bug')
+        expect(trello).not_to receive(:add_label)
         core.import!
       end
     end

data/spec/pivotal_to_trello/pivotal_wrapper_spec.rb

@@ -5,26 +5,28 @@ describe 'PivotalWrapper' do
 
   context '#initialize' do
     it 'sets the pivotal token' do
-      ::PivotalTracker::Client.should_receive(:token=).with('token')
+      expect(::PivotalTracker::Client).to receive(:token=).with('token')
       PivotalToTrello::PivotalWrapper.new('token')
     end
   end
 
   context '#project_choices' do
     it 'returns a hash of pivotal projects' do
-      project = OpenStruct.new(:id => 'id', :name => 'My Project')
-      ::PivotalTracker::Project.should_receive(:all).and_return([project])
-      wrapper.project_choices.should == { 'id' => 'My Project'}
+      project = OpenStruct.new(id: 'id', name: 'My Project')
+      expect(::PivotalTracker::Project).to receive(:all).and_return([project])
+      expect(wrapper.project_choices).to eq('id' => 'My Project')
     end
   end
 
   context '#stories' do
-    it 'returns an array of pivotal stories' do
-      story   = mock_pivotal_story
+    it 'returns a sorted array of pivotal stories' do
+      first_story = mock_pivotal_story(created_at: Time.now - 10_000)
+      last_story  = mock_pivotal_story(created_at: Time.now + 10_000)
       project = double(PivotalTracker::Project)
-      ::PivotalTracker::Project.should_receive(:find).with('project_id').and_return(project)
-      project.stub_chain(:stories, :all).and_return([story])
-      wrapper.stories('project_id').should == [story]
+      expect(::PivotalTracker::Project).to receive(:find).with('project_id').and_return(project)
+      allow(project).to receive_message_chain(:stories, :all).and_return([last_story, first_story])
+      expect(wrapper.stories('project_id').first).to eq(first_story)
+      expect(wrapper.stories('project_id').last).to eq(last_story)
     end
   end
-end
+end

data/spec/pivotal_to_trello/trello_wrapper_spec.rb

@@ -4,15 +4,15 @@ describe 'TrelloWrapper' do
   let(:wrapper) { PivotalToTrello::TrelloWrapper.new('key', 'token') }
 
   before(:each) do
-    IO.any_instance.stub(:puts)
+    allow_any_instance_of(IO).to receive(:puts)
   end
 
   context '#initialize' do
     it 'sets the auth credentials' do
       config = double
-      Trello.should_receive(:configure).and_yield(config)
-      config.should_receive(:developer_public_key=).with('key')
-      config.should_receive(:member_token=).with('token')
+      expect(Trello).to receive(:configure).and_yield(config)
+      expect(config).to receive(:developer_public_key=).with('key')
+      expect(config).to receive(:member_token=).with('token')
       PivotalToTrello::TrelloWrapper.new('key', 'token')
     end
   end
@@ -22,93 +22,115 @@ describe 'TrelloWrapper' do
 
     it 'creates a new card if none exists' do
       story = mock_pivotal_story
-      wrapper.should_receive(:get_card).and_return(nil)
-      Trello::Card.should_receive(:create).with(
-        :name    => story.name,
-        :desc    => story.description,
-        :list_id => 'list_id'
+      expect(wrapper).to receive(:get_card).and_return(nil)
+      expect(Trello::Card).to receive(:create).with(
+        name:    story.name,
+        desc:    story.description,
+        list_id: 'list_id',
       ).and_return(card)
-      wrapper.create_card('list_id', story).should == card
+      expect(wrapper.create_card('list_id', story)).to eq(card)
     end
 
     it 'does not create a new card if one exists with the same name' do
-      story = mock_pivotal_story(:name => 'My Card')
-      Trello::List.stub_chain(:find, :cards).and_return([card])
-      Trello::Card.should_not_receive(:create)
-      wrapper.create_card('list_id', story).should == card
+      story = mock_pivotal_story(name: 'My Card')
+      allow(Trello::List).to receive_message_chain(:find, :cards).and_return([card])
+      expect(Trello::Card).not_to receive(:create)
+      expect(wrapper.create_card('list_id', story)).to eq(card)
     end
 
     it 'creates a new card if one exists with a different name' do
-      story = mock_pivotal_story(:name => 'My Foo')
-      Trello::List.stub_chain(:find, :cards).and_return([card])
-      Trello::Card.should_receive(:create).with(
-        :name    => story.name,
-        :desc    => story.description,
-        :list_id => 'list_id'
+      story = mock_pivotal_story(name: 'My Foo')
+      allow(Trello::List).to receive_message_chain(:find, :cards).and_return([card])
+      expect(Trello::Card).to receive(:create).with(
+        name:    story.name,
+        desc:    story.description,
+        list_id: 'list_id',
       ).and_return(card)
-      wrapper.create_card('list_id', story).should == card
+      expect(wrapper.create_card('list_id', story)).to eq(card)
     end
 
     it 'adds comments' do
-      note  = OpenStruct.new(:text => 'My Note', :author => 'John Smith')
+      note  = OpenStruct.new(text: 'My Note', author: 'John Smith')
       story = mock_pivotal_story
-      story.stub_chain(:notes, :all).and_return([note])
-      wrapper.should_receive(:get_card).and_return(nil)
-      Trello::Card.should_receive(:create).and_return(card)
-      card.should_receive(:add_comment).with('[John Smith] My Note')
-      wrapper.create_card('list_id', story).should == card
+      allow(story).to receive_message_chain(:notes, :all).and_return([note])
+      expect(wrapper).to receive(:get_card).and_return(nil)
+      expect(Trello::Card).to receive(:create).and_return(card)
+      expect(card).to receive(:add_comment).with('[John Smith] My Note')
+      expect(wrapper.create_card('list_id', story)).to eq(card)
+    end
+
+    it 'adds tasks' do
+      task      = OpenStruct.new(description: 'My Task', complete: false)
+      story     = mock_pivotal_story
+      checklist = double(Trello::Checklist)
+      allow(story).to receive_message_chain(:tasks, :all).and_return([task])
+      expect(wrapper).to receive(:get_card).and_return(nil)
+      expect(Trello::Card).to receive(:create).and_return(card)
+      expect(Trello::Checklist).to receive(:create).with(name: 'Tasks', card_id: card.id).and_return(checklist)
+      expect(card).to receive(:add_checklist).with(checklist)
+      expect(checklist).to receive(:add_item).with(task.description, task.complete)
+      expect(wrapper.create_card('list_id', story)).to eq(card)
     end
   end
 
   context '#board_choices' do
     it 'returns a hash of Trello boards' do
-      board = OpenStruct.new(:id => 'id', :name => 'My Board')
-      Trello::Board.should_receive(:all).and_return([board])
-      wrapper.board_choices.should == { 'id' => 'My Board'}
+      board = OpenStruct.new(id: 'id', name: 'My Board')
+      expect(Trello::Board).to receive(:all).and_return([board])
+      expect(wrapper.board_choices).to eq('id' => 'My Board')
     end
   end
 
   context '#list_choices' do
     it 'returns a hash of Trello lists' do
       board = double(Trello::Board)
-      list  = OpenStruct.new(:id => 'id', :name => 'My List')
-      Trello::Board.should_receive(:find).with('board_id').and_return(board)
-      board.should_receive(:lists).and_return([list])
-      wrapper.list_choices('board_id').should == {
-        'id'  => 'My List',
-        false => "[don't import these stories]",
-      }
+      list  = OpenStruct.new(id: 'id', name: 'My List')
+      expect(Trello::Board).to receive(:find).with('board_id').and_return(board)
+      expect(board).to receive(:lists).and_return([list])
+      expect(wrapper.list_choices('board_id')).to eq( 'id'  => 'My List',
+                                                      false => "[don't import these stories]")
     end
   end
 
   context '#cards_for_list' do
     it 'returns a hash of Trello lists' do
       list = double(Trello::List)
-      card  = OpenStruct.new(:name => 'My Card', :desc => 'My Description')
-      Trello::List.should_receive(:find).with('list_id').and_return(list)
-      list.should_receive(:cards).and_return([card])
-      expected = {'193060beddd00d64259bdc1271d6c5a330e92e7d' => card}
-      wrapper.cards_for_list('list_id').should == expected
+      card = OpenStruct.new(name: 'My Card', desc: 'My Description')
+      expect(Trello::List).to receive(:find).with('list_id').and_return(list)
+      expect(list).to receive(:cards).and_return([card])
+      expected = { '193060beddd00d64259bdc1271d6c5a330e92e7d' => card }
+      expect(wrapper.cards_for_list('list_id')).to eq(expected)
       # Test caching.
-      Trello::List.should_not_receive(:find)
-      wrapper.cards_for_list('list_id').should == expected
+      expect(Trello::List).not_to receive(:find)
+      expect(wrapper.cards_for_list('list_id')).to eq(expected)
     end
   end
 
   context '#add_label' do
+    let(:board) { double(Trello::Board) }
+
+    before do
+      expect(Trello::Board).to receive(:find).with('board_id').and_return(board)
+      allow(board).to receive(:labels).and_return([])
+    end
+
     it 'adds a label if it does not already exist' do
-      card = mock_trello_card
-      card.stub(:labels => [])
-      card.should_receive(:add_label).with('red')
-      wrapper.add_label(card, 'red')
+      label = double(Trello::Label)
+      card  = mock_trello_card(board_id: 'board_id')
+      allow(card).to receive_messages(labels: [])
+      expect(Trello::Label).to receive(:create).with(name: 'bug', board_id: 'board_id', color: 'red').and_return(label)
+      expect(card).to receive(:add_label).with(label)
+      wrapper.add_label(card, 'bug', 'red')
     end
 
     it 'does not add a label if it already exists' do
-      card = mock_trello_card
-      card.stub(:labels => [OpenStruct.new(:color => 'red')])
-      card.should_not_receive(:add_label)
-      wrapper.add_label(card, 'red')
+      label = double(Trello::Label, id: '1234', name: 'bug', color: 'red')
+      card  = mock_trello_card(board_id: 'board_id')
+      allow(board).to receive(:labels).and_return([label])
+      allow(card).to receive_messages(labels: [label])
+      expect(card).not_to receive(:add_label)
+      wrapper.add_label(card, 'bug', 'red')
     end
   end
 
-end
+end