ruby-trello 1.4.2 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: eff2182d04d8c17e2f7978adac83a17b053e77c2
-  data.tar.gz: 98a0c4853b9033260ee8b7095dddcf2b65899d42
+  metadata.gz: 3131b86cac06b4df9efe8f417ab1bd32de3cbdb1
+  data.tar.gz: 862affe69703265224606c58b6a72cb5e8c1f668
 SHA512:
-  metadata.gz: 8e50b332a4d61974e2c27dd2d2e9d110bb54971d668ea659cca4f06762d0d42b155da98c00e391eb151451983e95405e44545425841692647e9ca17b3bdbe7c0
-  data.tar.gz: 2b3526b6a11edee0d6ba2357f4fda861bc2c2c0c376f8df945fe4dd412ed2e86589a4be66892765e8de581314fb265172fd0c009e38636f65bde74abceb36fa4
+  metadata.gz: 3f6f8f608417641e88c05ab8df2ebdb5e50bff8f522070be335d3be733d9eca5f6cd5470e34de6542875a9d1dfbb4b2dd78147403d035b2c4e931ebfb15c42e1
+  data.tar.gz: 1e53a8176105929d6c972888e3d6b218bbba627b92abcef67b6e660119ac5371dbca524c4fd2990751750a267ad318fc67803a95dc083dfc17b743053534c562

data/README.md

@@ -20,7 +20,10 @@ Seriously, [check it out](http://www.trello.com/).
 Full Disclosure: This library is mostly complete, if you do find anything missing or not functioning as you expect it
 to, please [let us know](https://trello.com/card/spot-a-bug-report-it/4f092b2ee23cb6fe6d1aaabd/17).
 
-Supports Ruby 2.0 or newer. Version 1.3.0 is the last version that supports Ruby that is older than 1.9.3. 
+Supports Ruby 2.1.0 or newer.
+
+Use version 1.3.0 or earlier for ruby 1.9.3 support.
+Use version 1.4.x or earlier for ruby 2.0.0 support.
 
 ## Configuration
 

data/lib/trello/card.rb

@@ -29,6 +29,8 @@ module Trello
   #   @return [Dateime]
   # @!attribute [rw] card_labels
   #   @return [Array<Hash>]
+  # @!attribute [rw] labels
+  #   @return [Array<Trello::Labels>]
   # @!attribute [rw] cover_image_id
   #   @return [String] A 24-character hex string
   # @!attribute [r] badges
@@ -42,7 +44,7 @@ module Trello
 
   class Card < BasicData
     register_attributes :id, :short_id, :name, :desc, :due, :closed, :url, :short_url,
-      :board_id, :member_ids, :list_id, :pos, :last_activity_date, :card_labels,
+      :board_id, :member_ids, :list_id, :pos, :last_activity_date, :labels, :card_labels,
       :cover_image_id, :badges, :card_members, :source_card_id, :source_card_properties,
       readonly: [ :id, :short_id, :url, :short_url, :last_activity_date, :badges, :card_members ]
     validates_presence_of :id, :name, :list_id
@@ -67,6 +69,7 @@ module Trello
       pos: 'pos',
       last_activity_date: 'dateLastActivity',
       card_labels: 'idLabels',
+      labels: 'labels',
       badges: 'badges',
       card_members: 'members',
       source_card_id: "idCardSource",
@@ -151,6 +154,8 @@ module Trello
     #     (24-character hex strings).
     # @option fields [String] :pos A position. `"top"`, `"bottom"`, or a
     #     positive number. Defaults to `"bottom"`.
+    # @option fields [Array]  :labels An Array of Trello::Label objects
+    #     derived from the JSON response
     # @option fields [String] :card_labels A comma-separated list of
     #     objectIds (24-character hex strings).
     # @option fields [Object] :cover_image_id
@@ -173,6 +178,7 @@ module Trello
       attributes[:member_ids]             = fields[SYMBOL_TO_STRING[:member_ids]]
       attributes[:list_id]                = fields[SYMBOL_TO_STRING[:list_id]]
       attributes[:pos]                    = fields[SYMBOL_TO_STRING[:pos]]
+      attributes[:labels]                 = (fields[SYMBOL_TO_STRING[:labels]] || []).map { |lbl| Trello::Label.new(lbl) }
       attributes[:card_labels]            = fields[SYMBOL_TO_STRING[:card_labels]]
       attributes[:last_activity_date]     = Time.iso8601(fields[SYMBOL_TO_STRING[:last_activity_date]]) rescue nil
       attributes[:cover_image_id]         = fields[SYMBOL_TO_STRING[:cover_image_id]]
@@ -200,8 +206,6 @@ module Trello
       MultiAssociation.new(self, states).proxy
     end
 
-    many :labels
-
     # Returns a reference to the list this card is currently in.
     one :list, path: :lists, using: :list_id
 
@@ -215,6 +219,16 @@ module Trello
       MultiAssociation.new(self, members).proxy
     end
 
+    # Returns a list of members who have upvoted this card
+    # NOTE: this fetches a list each time it's called to avoid case where
+    # card is voted (or vote is removed) after card is fetched. Optimizing
+    # accuracy over network performance
+    #
+    # @return [Array<Trello::Member>]
+    def voters
+      Member.from_response client.get("/cards/#{id}/membersVoted")
+    end
+
     # Saves a record.
     #
     # @raise [Trello::Error] if the card could not be saved
@@ -340,6 +354,30 @@ module Trello
       client.delete("/cards/#{id}/members/#{member.id}")
     end
 
+    # Current authenticated user upvotes a card
+    def upvote
+      begin
+        client.post("/cards/#{id}/membersVoted", {
+          value: me.id
+        })
+      rescue Trello::Error => e
+        fail e unless e.message =~ /has already voted/i
+      end
+
+      self
+    end
+
+    # Recind upvote. Noop if authenticated user hasn't previously voted
+    def remove_upvote
+      begin
+        client.delete("/cards/#{id}/membersVoted/#{me.id}")
+      rescue Trello::Error => e
+        fail e unless e.message =~ /has not voted/i
+      end
+
+      self
+    end
+
     # Add a label
     def add_label(label)
       unless label.valid?
@@ -360,7 +398,8 @@ module Trello
 
     # Add an attachment to this card
     def add_attachment(attachment, name='')
-      if attachment.is_a? File
+      # Is it a file object or a string (url)?
+      if attachment.respond_to?(:path) && attachment.respond_to?(:read)
         client.post("/cards/#{id}/attachments", {
             file: attachment,
             name: name
@@ -393,5 +432,16 @@ module Trello
     def comments
       comments = Comment.from_response client.get("/cards/#{id}/actions", filter: "commentCard")
     end
+
+    # Find the creation date
+    def created_at
+      @created_at ||= Time.at(id[0..7].to_i(16)) rescue nil
+    end
+
+    private
+
+    def me
+      @me ||= Member.find(:me)
+    end
   end
 end

data/lib/trello/checklist.rb

@@ -107,6 +107,14 @@ module Trello
       client.post("/checklists/#{id}/checkItems", {name: name, checked: checked, pos: position})
     end
 
+    # Update a checklist item's state, e.g.: "complete" or "incomplete"
+    def update_item_state(item_id, state)
+      client.put(
+        "/cards/#{card_id}/checklist/#{id}/checkItem/#{item_id}/state",
+        value: state,
+      )
+    end
+
     # Delete a checklist item
     def delete_checklist_item(item_id)
       client.delete("/checklists/#{id}/checkItems/#{item_id}")
@@ -119,7 +127,7 @@ module Trello
 
     # Copy a checklist (i.e., same attributes, items, etc.)
     def copy
-      checklist_copy = self.class.create(name: self.name, board_id: self.board_id)
+      checklist_copy = self.class.create(name: self.name, board_id: self.board_id, card_id: self.card_id)
       copy_items_to(checklist_copy)
       return checklist_copy
     end

data/lib/trello/item.rb

@@ -20,11 +20,13 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # an item.
     def update_fields(fields)
-      attributes[:id]    = fields['id']
-      attributes[:name]  = fields['name']
-      attributes[:type]  = fields['type']
-      attributes[:state] = fields['state']
-      attributes[:pos]   = fields['pos']
+      attributes[:id]           = fields['id']
+      attributes[:card_id]      = fields['idCard']
+      attributes[:checklist_id] = fields['idChecklist']
+      attributes[:name]         = fields['name']
+      attributes[:type]         = fields['type']
+      attributes[:state]        = fields['state']
+      attributes[:pos]          = fields['pos']
       self
     end
 

data/lib/trello/list.rb

@@ -12,7 +12,7 @@ module Trello
   # @!attribute [rw] pos
   #   @return [Object]
   class List < BasicData
-    register_attributes :id, :name, :closed, :board_id, :pos, readonly: [ :id, :board_id ]
+    register_attributes :id, :name, :closed, :board_id, :pos, :source_list_id, readonly: [ :id, :board_id ]
     validates_presence_of :id, :name, :board_id
     validates_length_of   :name, in: 1..16384
 
@@ -29,9 +29,10 @@ module Trello
 
       def create(options)
         client.create(:list,
-            'name'    => options[:name],
-            'idBoard' => options[:board_id],
-            'pos'     => options[:pos])
+            'name'         => options[:name],
+            'idBoard'      => options[:board_id],
+            'pos'          => options[:pos],
+            'idListSource' => options[:source_list_id])
       end
     end
 
@@ -40,11 +41,12 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # a List.
     def update_fields(fields)
-      attributes[:id]       = fields['id']
-      attributes[:name]     = fields['name']
-      attributes[:closed]   = fields['closed']
-      attributes[:board_id] = fields['idBoard']
-      attributes[:pos]      = fields['pos']
+      attributes[:id]             = fields['id']
+      attributes[:name]           = fields['name']
+      attributes[:closed]         = fields['closed']
+      attributes[:board_id]       = fields['idBoard']
+      attributes[:pos]            = fields['pos']
+      attributes[:source_list_id] = fields['idListSource']
       self
     end
 
@@ -55,7 +57,8 @@ module Trello
         name: name,
         closed: closed || false,
         idBoard: board_id,
-        pos: pos
+        pos: pos,
+        idListSource: source_list_id
       })
     end
 
@@ -98,6 +101,11 @@ module Trello
        })
     end
 
+    # Archives all the cards of the list
+    def archive_all_cards
+      client.post("/lists/#{id}/archiveAllCards")
+    end
+
     # :nodoc:
     def request_prefix
       "/lists/#{id}"

data/lib/trello/net.rb

@@ -19,6 +19,7 @@ module Trello
             Response.new(200, {}, result)
           end
         rescue RestClient::Exception => e
+          raise if !e.respond_to?(:http_code) || e.http_code.nil?
           Response.new(e.http_code, {}, e.http_body)
         end
       end

data/spec/board_spec.rb

@@ -130,12 +130,12 @@ module Trello
 
         expect(labels.count).to eq(4)
         expect(labels[2].color).to  eq('red')
-        expect(labels[2].id).to  eq('abcdef123456789123456789')
+        expect(labels[2].id).to  eq('cbcdef123456789123456789')
         expect(labels[2].board_id).to  eq('abcdef123456789123456789')
         expect(labels[2].name).to  eq('deploy')
         expect(labels[2].uses).to  eq(2)
         expect(labels[3].color).to  eq('blue')
-        expect(labels[3].id).to  eq('abcdef123456789123456789')
+        expect(labels[3].id).to  eq('dbcdef123456789123456789')
         expect(labels[3].board_id).to  eq('abcdef123456789123456789')
         expect(labels[3].name).to  eq('on hold')
         expect(labels[3].uses).to  eq(6)

data/spec/card_spec.rb

@@ -94,7 +94,7 @@ module Trello
 
         expect(card).to be_a Card
       end
-      
+
       it 'creates a duplicate card with source due date and checklists and saves it on Trello', refactor: true do
         payload = {
           source_card_id: cards_details.first['id'],
@@ -220,6 +220,16 @@ module Trello
       it "gets its pos" do
         expect(card.pos).to_not be_nil
       end
+
+
+      it 'gets its creation time' do
+        expect(card.created_at).to be_kind_of Time
+      end
+
+      it 'get its correct creation time' do
+        expect(card.created_at).to eq(Time.parse('2061-05-04 04:40:18 +0200'))
+      end
+
     end
 
     context "actions" do
@@ -402,6 +412,76 @@ module Trello
       end
     end
 
+    context "add/remove votes" do
+      let(:authenticated_member) { double(id: '4ee7df3ce582acdec80000b2') }
+
+      before do
+        allow(card)
+          .to receive(:me)
+          .and_return(authenticated_member)
+      end
+
+      it 'upvotes a card with the currently authenticated member' do
+        expect(client)
+          .to receive(:post)
+          .with("/cards/abcdef123456789123456789/membersVoted", {
+            value: authenticated_member.id
+          })
+
+        card.upvote
+      end
+
+      it 'returns the card even if the user has already upvoted' do
+        expect(client)
+          .to receive(:post)
+          .with("/cards/abcdef123456789123456789/membersVoted", {
+            value: authenticated_member.id
+          })
+          .and_raise(Trello::Error, 'member has already voted')
+        expect(card.upvote).to be_kind_of Trello::Card
+      end
+
+      it 'removes an upvote from a card' do
+        expect(client)
+          .to receive(:delete)
+          .with("/cards/abcdef123456789123456789/membersVoted/#{authenticated_member.id}")
+
+        card.remove_upvote
+      end
+
+      it 'returns card after remove_upvote even if the user has not previously upvoted it' do
+        expect(client)
+          .to receive(:delete)
+          .with("/cards/abcdef123456789123456789/membersVoted/#{authenticated_member.id}")
+          .and_raise(Trello::Error, 'member has not voted on the card')
+
+        card.remove_upvote
+      end
+
+    end
+
+    context "return all voters" do
+      it 'returns members that have voted for the card' do
+        no_voters = JSON.generate([])
+        expect(client)
+          .to receive(:get)
+          .with("/cards/#{card.id}/membersVoted")
+          .and_return(no_voters)
+
+        card.voters
+
+
+        voters = JSON.generate([user_details])
+        expect(client)
+          .to receive(:get)
+          .with("/cards/#{card.id}/membersVoted")
+          .and_return(voters)
+
+        expect(card.voters.first).to be_kind_of Trello::Member
+      end
+    end
+
+
     context "comments" do
       it "posts a comment" do
         expect(client)
@@ -415,37 +495,39 @@ module Trello
 
     context "labels" do
       before do
-        allow(client)
-          .to receive(:get)
-          .with("/cards/abcdef123456789123456789/labels")
-          .and_return label_payload
-
         allow(client)
           .to receive(:post)
-          .with("/cards/abcdef123456789123456789/labels", { value: 'green' })
+          .with("/cards/ebcdef123456789123456789/labels", { value: 'green' })
           .and_return "not important"
 
         allow(client)
           .to receive(:delete)
-          .with("/cards/abcdef123456789123456789/labels/green")
+          .with("/cards/ebcdef123456789123456789/labels/green")
           .and_return "not important"
       end
-      it "can retrieve labels" do
-        allow(client).to receive(:get).
-          with("/cards/abcdef123456789123456789/labels", {}).
-          and_return label_payload
+
+      it "includes card label objects list" do
         labels = card.labels
-        expect(labels.size).to  eq(4)
-        expect(labels[0].color).to  eq('yellow')
-        expect(labels[0].id).to  eq('abcdef123456789123456789')
-        expect(labels[0].board_id).to  eq('abcdef123456789123456789')
-        expect(labels[0].name).to  eq('iOS')
-        expect(labels[0].uses).to  eq(3)
-        expect(labels[1].color).to  eq('purple')
-        expect(labels[1].id).to  eq('abcdef123456789123456789')
-        expect(labels[1].board_id).to  eq('abcdef123456789123456789')
-        expect(labels[1].name).to  eq('Issue or bug')
-        expect(labels[1].uses).to  eq(1)
+        expect(labels.size).to        eq(4)
+        expect(labels[0].color).to    eq('yellow')
+        expect(labels[0].id).to       eq('abcdef123456789123456789')
+        expect(labels[0].board_id).to eq('abcdef123456789123456789')
+        expect(labels[0].name).to     eq('iOS')
+        expect(labels[0].uses).to     eq(3)
+        expect(labels[1].color).to    eq('purple')
+        expect(labels[1].id).to       eq('bbcdef123456789123456789')
+        expect(labels[1].board_id).to eq('abcdef123456789123456789')
+        expect(labels[1].name).to     eq('Issue or bug')
+        expect(labels[1].uses).to     eq(1)
+      end
+
+      it "includes label ids list" do
+        label_ids = card.card_labels
+        expect(label_ids.size).to eq(4)
+        expect(label_ids[0]).to   eq('abcdef123456789123456789')
+        expect(label_ids[1]).to   eq('bbcdef123456789123456789')
+        expect(label_ids[2]).to   eq('cbcdef123456789123456789')
+        expect(label_ids[3]).to   eq('dbcdef123456789123456789')
       end
 
       it "can remove a label" do

data/spec/checklist_spec.rb

@@ -139,6 +139,17 @@ module Trello
 
         checklist.add_item(expected_item_name, expected_checked, expected_pos)
       end
+
+      it "updates an item's state" do
+        expected_item_id = "1234"
+        expected_state = "incomplete"
+        expected_resource =
+          "/cards/abccardid/checklist/abcdef123456789123456789" \
+          "/checkItem/#{expected_item_id}/state"
+        payload = { value: expected_state }
+        expect(client).to receive(:put).once.with(expected_resource, payload)
+        checklist.update_item_state(expected_item_id, expected_state)
+      end
     end
 
     context "board" do

data/spec/client_spec.rb

@@ -3,202 +3,255 @@ require "spec_helper"
 include Trello
 include Trello::Authorization
 
-describe Client, "and how it handles authorization" do
-  let (:fake_ok_response) do
-    double "A fake OK response",
-      code: 200,
-      body: "A fake response body"
-  end
+describe Client do
+
   let(:client) { Client.new }
   let(:auth_policy) { double }
 
   before do
-    allow(TInternet)
-      .to receive(:execute)
-      .and_return fake_ok_response
-
     allow(Authorization::AuthPolicy)
       .to receive(:new)
-      .and_return(auth_policy)
+           .and_return(auth_policy)
 
     allow(auth_policy)
       .to receive(:authorize) { |request| request }
   end
 
-  it "authorizes before it queries the internet" do
-    expect(auth_policy)
-      .to receive(:authorize)
-      .once
-      .ordered
+  describe "and how it handles RestClient exceptions" do
+    context "with RestClient::Exception sans HTTP code" do
+      let(:rc_exception_without_http_code) { RestClient::Exception.new }
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once
-      .ordered
+      before do
+        allow(TInternet)
+          .to receive(:execute_core)
+          .and_raise(rc_exception_without_http_code)
+      end
 
-    client.get "/xxx"
-  end
+      it "bubbles up RestClient::Exception errors that don't contain an HTTP code" do
+        expect { client.get "/xxx" }.to raise_error rc_exception_without_http_code
+      end
+    end
 
-  it "queries the internet with expanded earl and query parameters" do
-    expected_uri = Addressable::URI.parse("https://api.trello.com/1/xxx?a=1&b=2")
-    expected_request = Request.new :get, expected_uri, {}
+    context "with RestClient::Exception that contains HTTP code" do
+      let(:response_with_non_200_status) { double "A fake 404 response",
+                                              code: 404,
+                                              body: "404 error response"}
+      let(:rc_exception_with_http_code) { RestClient::Exception.new(double("404 error response", {:code => 404, :body => "404 error response"}))}
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once
-      .with expected_request
+      before do
+        allow(TInternet)
+          .to receive(:execute_core)
+          .and_raise(rc_exception_with_http_code)
+      end
+
+      it "raises Error" do
+
+        expect(TInternet)
+          .to receive(:try_execute)
+          .and_return(response_with_non_200_status)
 
-    client.get "/xxx", a: "1", b: "2"
+        expect { client.get "/xxx" }.to raise_error(Error, "404 error response")
+      end
+    end
   end
 
-  it "encodes parameters" do
-    expected_uri = Addressable::URI.parse("https://api.trello.com/1/xxx?name=Jazz%20Kang")
-    expected_request = Request.new :get, expected_uri, {}
+  describe "and how it handles authorization" do
+    let (:fake_ok_response) do
+      double "A fake OK response",
+             code: 200,
+             body: "A fake response body"
+    end
+    let(:client) { Client.new }
+    let(:auth_policy) { double }
+
+    before do
+      allow(TInternet)
+        .to receive(:execute)
+             .and_return fake_ok_response
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once
-      .with expected_request
+      allow(Authorization::AuthPolicy)
+        .to receive(:new)
+             .and_return(auth_policy)
 
-    client.get "/xxx", name: "Jazz Kang"
-  end
+      allow(auth_policy)
+        .to receive(:authorize) { |request| request }
+    end
 
-  it "raises an error when response has non-200 status" do
-    expected_error_message = "An error response"
-    response_with_non_200_status = double "A fake OK response",
-      code: 404,
-      body: expected_error_message
+    it "authorizes before it queries the internet" do
+      expect(auth_policy)
+        .to receive(:authorize)
+             .once
+             .ordered
 
-    expect(TInternet)
-      .to receive(:execute)
-      .and_return response_with_non_200_status
+      expect(TInternet)
+        .to receive(:execute)
+             .once
+             .ordered
 
-    expect { client.get "/xxx" }.to raise_error expected_error_message
-  end
+      client.get "/xxx"
+    end
+
+    it "queries the internet with expanded earl and query parameters" do
+      expected_uri = Addressable::URI.parse("https://api.trello.com/1/xxx?a=1&b=2")
+      expected_request = Request.new :get, expected_uri, {}
+
+      expect(TInternet)
+        .to receive(:execute)
+             .once
+             .with expected_request
+
+      client.get "/xxx", a: "1", b: "2"
+    end
+
+    it "encodes parameters" do
+      expected_uri = Addressable::URI.parse("https://api.trello.com/1/xxx?name=Jazz%20Kang")
+      expected_request = Request.new :get, expected_uri, {}
+
+      expect(TInternet)
+        .to receive(:execute)
+             .once
+             .with expected_request
+
+      client.get "/xxx", name: "Jazz Kang"
+    end
 
-  it "uses version 1 of the API" do
-    expect(TInternet)
-      .to receive(:execute)
-      .once do |request|
+    it "raises an error when response has non-200 status" do
+      expected_error_message = "An error response"
+      response_with_non_200_status = double "A fake OK response",
+                                            code: 404,
+                                            body: expected_error_message
+
+      expect(TInternet)
+        .to receive(:execute)
+             .and_return response_with_non_200_status
+
+      expect { client.get "/xxx" }.to raise_error expected_error_message
+    end
+
+    it "uses version 1 of the API" do
+      expect(TInternet)
+        .to receive(:execute)
+             .once do |request|
         expect(request.uri.to_s).to match(/1\//)
         fake_ok_response
       end
 
-    client.get "/"
-  end
+      client.get "/"
+    end
 
-  it "omits the \"?\" when no parameters" do
-    expect(TInternet)
-      .to receive(:execute)
-      .once do |request|
+    it "omits the \"?\" when no parameters" do
+      expect(TInternet)
+        .to receive(:execute)
+             .once do |request|
         expect(request.uri.to_s).not_to match(/\?$/)
         fake_ok_response
       end
 
-    client.get "/xxx"
-  end
+      client.get "/xxx"
+    end
 
-  it "supports post" do
-    expect(TInternet)
-      .to receive(:execute)
-      .once
-      .and_return fake_ok_response
+    it "supports post" do
+      expect(TInternet)
+        .to receive(:execute)
+             .once
+             .and_return fake_ok_response
 
-    client.post "/xxx", { phil: "T' north" }
-  end
+      client.post "/xxx", { phil: "T' north" }
+    end
 
-  it "supplies the body for a post" do
-    expected_body = { name: "Phil", nickname: "The Crack Fox" }
+    it "supplies the body for a post" do
+      expected_body = { name: "Phil", nickname: "The Crack Fox" }
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once do |request|
+      expect(TInternet)
+        .to receive(:execute)
+             .once do |request|
         expect(request.body).to eq expected_body
         fake_ok_response
       end
 
-    client.post "/xxx", expected_body
-  end
+      client.post "/xxx", expected_body
+    end
 
-  it "supplies the path for a post" do
-    expected_path = "/xxx"
+    it "supplies the path for a post" do
+      expected_path = "/xxx"
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once do |request|
+      expect(TInternet)
+        .to receive(:execute)
+             .once do |request|
         expect(request.uri.path).to match(/#{expected_path}$/)
         fake_ok_response
       end
 
-    client.post "/xxx", {}
-  end
+      client.post "/xxx", {}
+    end
 
-  it "supports put" do
-    expect(TInternet)
-      .to receive(:execute)
-      .once
-      .and_return fake_ok_response
+    it "supports put" do
+      expect(TInternet)
+        .to receive(:execute)
+             .once
+             .and_return fake_ok_response
 
-    client.put "/xxx", { phil: "T' north" }
-  end
+      client.put "/xxx", { phil: "T' north" }
+    end
 
-  it "supplies the body for a put" do
-    expected_body = { name: "Phil", nickname: "The Crack Fox" }
+    it "supplies the body for a put" do
+      expected_body = { name: "Phil", nickname: "The Crack Fox" }
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once do |request|
+      expect(TInternet)
+        .to receive(:execute)
+             .once do |request|
         expect(request.body).to eq expected_body
         fake_ok_response
       end
 
-    client.put "/xxx", expected_body
-  end
+      client.put "/xxx", expected_body
+    end
 
-  it "supplies the path for a put" do
-    expected_path = "/xxx"
+    it "supplies the path for a put" do
+      expected_path = "/xxx"
 
-    expect(TInternet)
-      .to receive(:execute)
-      .once do |request|
+      expect(TInternet)
+        .to receive(:execute)
+             .once do |request|
         expect(request.uri.path).to match(/#{expected_path}$/)
         fake_ok_response
       end
 
-    client.put "/xxx", {}
-  end
-
-  context "initialize" do
-    let(:client) do
-      Client.new(
-        consumer_key: 'consumer_key',
-        consumer_secret: 'consumer_secret',
-        oauth_token: 'oauth_token',
-        oauth_token_secret: 'oauth_token_secret'
-      )
+      client.put "/xxx", {}
     end
 
-    it "is configurable" do
-      expect(client.consumer_key).to eq('consumer_key')
-      expect(client.consumer_secret).to eq('consumer_secret')
-      expect(client.oauth_token).to eq('oauth_token')
-      expect(client.oauth_token_secret).to eq('oauth_token_secret')
-    end
-  end
+    context "initialize" do
+      let(:client) do
+        Client.new(
+          consumer_key: 'consumer_key',
+          consumer_secret: 'consumer_secret',
+          oauth_token: 'oauth_token',
+          oauth_token_secret: 'oauth_token_secret'
+        )
+      end
 
-  describe "configure" do
-    it "sets key attributes through config block" do
-      client.configure do |config|
-        config.consumer_key = 'consumer_key'
-        config.consumer_secret = 'consumer_secret'
-        config.oauth_token = 'oauth_token'
-        config.oauth_token_secret = 'oauth_token_secret'
+      it "is configurable" do
+        expect(client.consumer_key).to eq('consumer_key')
+        expect(client.consumer_secret).to eq('consumer_secret')
+        expect(client.oauth_token).to eq('oauth_token')
+        expect(client.oauth_token_secret).to eq('oauth_token_secret')
       end
+    end
 
-      expect(client.consumer_key).to eq('consumer_key')
-      expect(client.consumer_secret).to eq('consumer_secret')
-      expect(client.oauth_token).to eq('oauth_token')
-      expect(client.oauth_token_secret).to eq('oauth_token_secret')
+    describe "configure" do
+      it "sets key attributes through config block" do
+        client.configure do |config|
+          config.consumer_key = 'consumer_key'
+          config.consumer_secret = 'consumer_secret'
+          config.oauth_token = 'oauth_token'
+          config.oauth_token_secret = 'oauth_token_secret'
+        end
+
+        expect(client.consumer_key).to eq('consumer_key')
+        expect(client.consumer_secret).to eq('consumer_secret')
+        expect(client.oauth_token).to eq('oauth_token')
+        expect(client.oauth_token_secret).to eq('oauth_token_secret')
+      end
     end
   end
 end

data/spec/list_spec.rb

@@ -57,12 +57,19 @@ module Trello
         payload = {
           name: 'Test List',
           board_id: 'abcdef123456789123456789',
-          pos: 42
+          pos: 42,
+          source_list_id: 'abcdef123456789'
         }
 
         result = JSON.generate(payload)
 
-        expected_payload = {name: 'Test List', closed: false, idBoard: 'abcdef123456789123456789', pos: 42}
+        expected_payload = {
+          name: 'Test List',
+          closed: false,
+          idBoard: 'abcdef123456789123456789',
+          pos: 42,
+          idListSource: 'abcdef123456789'
+        }
 
         expect(client)
           .to receive(:post)
@@ -163,6 +170,15 @@ module Trello
 
         expect(list.move_all_cards(other_list)).to eq cards_payload
       end
+
+      it 'archives all cards' do
+        allow(client)
+          .to receive(:post)
+          .with('/lists/abcdef123456789123456789/archiveAllCards')
+          .and_return cards_payload
+
+        expect(list.archive_all_cards).to eq cards_payload
+      end
     end
 
     describe '#closed?' do

data/spec/spec_helper.rb

@@ -88,6 +88,7 @@ module Helpers
       'position'   => 16384,
       'url'        => 'https://trello.com/blah/blah',
       'idBoard'    => 'abcdef123456789123456789',
+      'idCard'     => 'abccardid',
       'idList'     => 'abcdef123456789123456789',
       'idMembers'  => ['abcdef123456789123456789'],
       'checkItems' => { 'id' => 'ghijk987654321' }
@@ -142,6 +143,11 @@ module Helpers
       'idBoard'           => 'abcdef123456789123456789',
       'idAttachmentCover' => 'abcdef123456789123456789',
       'idMembers'         => ['abcdef123456789123456789'],
+      'labels'            => label_details,
+      'idLabels'          => [ 'abcdef123456789123456789',
+                               'bbcdef123456789123456789',
+                               'cbcdef123456789123456789',
+                               'dbcdef123456789123456789' ],
       'url'               => 'https://trello.com/card/board/specify-the-type-and-scope-of-the-jit-in-a-lightweight-spec/abcdef123456789123456789/abcdef123456789123456789',
       'shortUrl'          => 'https://trello.com/c/abcdef12',
       'pos'               => 12,
@@ -310,9 +316,9 @@ module Helpers
   def label_details
     [
       {'color' => 'yellow', 'name' => 'iOS', 'id' => 'abcdef123456789123456789', 'uses' => 3, 'idBoard' => 'abcdef123456789123456789'},
-      {'color' => 'purple', 'name' => 'Issue or bug', 'id' => 'abcdef123456789123456789', 'uses' => 1, 'idBoard' => 'abcdef123456789123456789'},
-      {'color' => 'red', 'name' => 'deploy', 'id' => 'abcdef123456789123456789', 'uses' => 2, 'idBoard' => 'abcdef123456789123456789'},
-      {'color' => 'blue', 'name' => 'on hold', 'id' => 'abcdef123456789123456789', 'uses' => 6, 'idBoard' => 'abcdef123456789123456789'}
+      {'color' => 'purple', 'name' => 'Issue or bug', 'id' => 'bbcdef123456789123456789', 'uses' => 1, 'idBoard' => 'abcdef123456789123456789'},
+      {'color' => 'red', 'name' => 'deploy', 'id' => 'cbcdef123456789123456789', 'uses' => 2, 'idBoard' => 'abcdef123456789123456789'},
+      {'color' => 'blue', 'name' => 'on hold', 'id' => 'dbcdef123456789123456789', 'uses' => 6, 'idBoard' => 'abcdef123456789123456789'}
     ]
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-trello
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.5.1
 platform: ruby
 authors:
 - Jeremy Tregunna
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-10 00:00:00.000000000 Z
+date: 2016-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -56,14 +56,14 @@ dependencies:
   name: oauth
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.4.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.4.5
 - !ruby/object:Gem::Dependency
@@ -158,7 +158,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="