ruby-trello-czuger 2.0.0

data/spec/card_spec.rb

@@ -0,0 +1,822 @@
+require 'spec_helper'
+
+module Trello
+  describe Card do
+    include Helpers
+
+    let(:card)   { client.find(:card, 'abcdef123456789123456789') }
+    let(:client) { Client.new }
+
+    before do
+      allow(client)
+        .to receive(:get)
+        .with("/cards/abcdef123456789123456789", {})
+        .and_return JSON.generate(cards_details.first)
+    end
+
+    context "finding" do
+      let(:client) { Trello.client }
+
+      before do
+        allow(client)
+          .to receive(:find)
+      end
+
+      it "delegates to Trello.client#find" do
+        expect(client)
+          .to receive(:find)
+          .with(:card, 'abcdef123456789123456789', {})
+
+        Card.find('abcdef123456789123456789')
+      end
+
+      it "is equivalent to client#find" do
+        expect(Card.find('abcdef123456789123456789')).to eq(card)
+      end
+    end
+
+    context "creating" do
+      let(:client) { Trello.client }
+
+      it "creates a new record" do
+        cards_details.each do |card_details|
+          card = Card.new(card_details)
+          expect(card).to be_valid
+        end
+      end
+
+      it 'properly initializes all fields from response-like formatted hash' do
+        card_details = cards_details.first
+        card = Card.new(card_details)
+        expect(card.id).to                 eq(card_details['id'])
+        expect(card.short_id).to           eq(card_details['idShort'])
+        expect(card.name).to               eq(card_details['name'])
+        expect(card.desc).to               eq(card_details['desc'])
+        expect(card.closed).to             eq(card_details['closed'])
+        expect(card.list_id).to            eq(card_details['idList'])
+        expect(card.board_id).to           eq(card_details['idBoard'])
+        expect(card.cover_image_id).to     eq(card_details['idAttachmentCover'])
+        expect(card.member_ids).to         eq(card_details['idMembers'])
+        expect(card.labels).to             eq(card_details['labels'].map { |lbl| Trello::Label.new(lbl) })
+        expect(card.card_labels).to        eq(card_details['idLabels'])
+        expect(card.url).to                eq(card_details['url'])
+        expect(card.short_url).to          eq(card_details['shortUrl'])
+        expect(card.pos).to                eq(card_details['pos'])
+        expect(card.last_activity_date).to eq(card_details['dateLastActivity'])
+      end
+
+      it 'properly initializes all fields from options-like formatted hash' do
+        card_details = cards_details[1]
+        card = Card.new(card_details)
+        expect(card.name).to                   eq(card_details[:name])
+        expect(card.list_id).to                eq(card_details[:list_id])
+        expect(card.desc).to                   eq(card_details[:desc])
+        expect(card.member_ids).to             eq(card_details[:member_ids])
+        expect(card.card_labels).to            eq(card_details[:card_labels])
+        expect(card.due).to                    eq(card_details[:due])
+        expect(card.pos).to                    eq(card_details[:pos])
+        expect(card.source_card_id).to         eq(card_details[:source_card_id])
+        expect(card.source_card_properties).to eq(card_details[:source_card_properties])
+      end
+
+      it 'must not be valid if not given a name' do
+        card = Card.new('idList' => lists_details.first['id'])
+        expect(card).to_not be_valid
+      end
+
+      it 'must not be valid if not given a list id' do
+        card = Card.new('name' => lists_details.first['name'])
+        expect(card).to_not be_valid
+      end
+
+      it 'creates a new record and saves it on Trello', refactor: true do
+        payload = {
+          name: 'Test Card',
+          desc: nil,
+          card_labels: "abcdef123456789123456789"
+        }
+
+        result = JSON.generate(cards_details.first.merge(payload.merge(idList: lists_details.first['id'])))
+
+        expected_payload = {name: "Test Card", desc: nil, idList: "abcdef123456789123456789",
+                            idMembers: nil, idLabels: "abcdef123456789123456789", pos: nil, due: nil, dueComplete: false, idCardSource: nil, keepFromSource: 'all'}
+
+        expect(client)
+          .to receive(:post)
+          .with("/cards", expected_payload)
+          .and_return result
+
+        card = Card.create(cards_details.first.merge(payload.merge(list_id: lists_details.first['id'])))
+
+        expect(card).to be_a Card
+      end
+
+      it 'creates a duplicate card with all source properties and saves it on Trello', refactor: true do
+        payload = {
+          source_card_id: cards_details.first['id']
+        }
+
+        result = JSON.generate(cards_details.first.merge(payload.merge(idList: lists_details.first['id'])))
+
+        expected_payload = {name: nil, desc: nil, idList: "abcdef123456789123456789",
+                            idMembers: nil, idLabels: nil, pos: nil, due: nil, dueComplete: false, idCardSource: cards_details.first['id'], keepFromSource: 'all'}
+
+        expect(client)
+          .to receive(:post)
+          .with("/cards", expected_payload)
+          .and_return result
+
+        card = Card.create(cards_details.first.merge(payload.merge(list_id: lists_details.first['id'])))
+
+        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'],
+          source_card_properties: ['due', 'checklists']
+        }
+
+        result = JSON.generate(cards_details.first.merge(payload.merge(idList: lists_details.first['id'])))
+
+        expected_payload = {name: nil, desc: nil, idList: "abcdef123456789123456789",
+                            idMembers: nil, idLabels: nil, pos: nil, due: nil, dueComplete: false, idCardSource: cards_details.first['id'], keepFromSource: ['due', 'checklists']}
+
+        expect(client)
+          .to receive(:post)
+          .with("/cards", expected_payload)
+          .and_return result
+
+        card = Card.create(cards_details.first.merge(payload.merge(list_id: lists_details.first['id'])))
+
+        expect(card).to be_a Card
+      end
+
+      it 'creates a duplicate card with no source properties and saves it on Trello', refactor: true do
+        payload = {
+          source_card_id: cards_details.first['id'],
+          source_card_properties: nil
+        }
+
+        result = JSON.generate(cards_details.first.merge(payload.merge(idList: lists_details.first['id'])))
+
+        expected_payload = {name: nil, desc: nil, idList: "abcdef123456789123456789",
+                            idMembers: nil, idLabels: nil, pos: nil, due: nil, dueComplete: false, idCardSource: cards_details.first['id'], keepFromSource: nil}
+
+        expect(client)
+          .to receive(:post)
+          .with("/cards", expected_payload)
+          .and_return result
+
+        card = Card.create(cards_details.first.merge(payload.merge(list_id: lists_details.first['id'])))
+
+        expect(card).to be_a Card
+      end
+
+    end
+
+    context "updating" do
+      it "updating name does a put on the correct resource with the correct value" do
+        expected_new_name = "xxx"
+
+        payload = {
+          name: expected_new_name,
+        }
+
+        expect(client)
+          .to receive(:put)
+          .once
+          .with("/cards/abcdef123456789123456789", payload)
+
+        card.name = expected_new_name
+        card.save
+      end
+
+      it "updating desc does a put on the correct resource with the correct value" do
+        expected_new_desc = "xxx"
+
+        payload = {
+          desc: expected_new_desc,
+        }
+
+        expect(client).to receive(:put).once.with("/cards/abcdef123456789123456789", payload)
+
+        card.desc = expected_new_desc
+        card.save
+      end
+    end
+
+    context "deleting" do
+      it "deletes the card" do
+        expect(client)
+          .to receive(:delete)
+          .with("/cards/#{card.id}")
+
+        card.delete
+      end
+    end
+
+    context "fields" do
+      it "gets its id" do
+        expect(card.id).to_not be_nil
+      end
+
+      it "gets its short id" do
+        expect(card.short_id).to_not be_nil
+      end
+
+      it "gets its name" do
+        expect(card.name).to_not be_nil
+      end
+
+      it "gets its description" do
+        expect(card.desc).to_not be_nil
+      end
+
+      it "knows if it is open or closed" do
+        expect(card.closed).to_not be_nil
+      end
+
+      it "gets its url" do
+        expect(card.url).to_not be_nil
+      end
+
+      it "gets its short url" do
+        expect(card.short_url).to_not be_nil
+      end
+
+      it "gets its last active date" do
+        expect(card.last_activity_date).to_not be_nil
+      end
+
+      it "gets its cover image id" do
+        expect(card.cover_image_id).to_not be_nil
+      end
+
+      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
+      let(:filter) { :all }
+
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/cards/abcdef123456789123456789/actions", { filter: filter })
+          .and_return actions_payload
+      end
+
+      it "asks for all actions by default" do
+        expect(card.actions.count).to be > 0
+      end
+
+      context 'when overriding a filter' do
+        let(:filter) { :updateCard }
+
+        it "allows the filter" do
+          expect(card.actions(filter: filter).count).to be > 0
+        end
+      end
+    end
+
+    context "boards" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789", {})
+          .and_return JSON.generate(boards_details.first)
+      end
+
+      it "has a board" do
+        expect(card.board).to_not be_nil
+      end
+    end
+
+    context "cover image" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/attachments/abcdef123456789123456789", {})
+          .and_return JSON.generate(attachments_details.first)
+      end
+
+      it "has a cover image" do
+        expect(card.cover_image).to_not be_nil
+      end
+    end
+
+    context "checklists" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/cards/abcdef123456789123456789/checklists", { filter: :all})
+          .and_return checklists_payload
+      end
+
+      it "has a list of checklists" do
+        expect(card.checklists.count).to be > 0
+      end
+
+      it "creates a new checklist for the card" do
+        expect(client)
+          .to receive(:post)
+          .with("/cards/abcdef123456789123456789/checklists", name: "new checklist")
+
+        card.create_new_checklist("new checklist")
+      end
+    end
+
+    context "list" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/lists/abcdef123456789123456789", {})
+          .and_return JSON.generate(lists_details.first)
+      end
+      it 'has a list' do
+        expect(card.list).to_not be_nil
+      end
+
+      it 'can be moved to another list' do
+        other_list = double(id: '987654321987654321fedcba')
+        payload = {value: other_list.id}
+
+        expect(client)
+          .to receive(:put)
+          .with("/cards/abcdef123456789123456789/idList", payload)
+
+        card.move_to_list(other_list)
+      end
+
+      it 'should not be moved if new list is identical to old list' do
+        other_list = double(id: 'abcdef123456789123456789')
+        expect(client).to_not receive(:put)
+        card.move_to_list(other_list)
+      end
+
+      it "should accept a string for moving a card to list" do
+        payload = { value: "12345678"}
+
+        expect(client)
+          .to receive(:put)
+          .with("/cards/abcdef123456789123456789/idList", payload)
+
+        card.move_to_list("12345678")
+      end
+
+      it 'can be moved to another board' do
+        other_board = double(id: '987654321987654321fedcba')
+        payload = {value: other_board.id}
+
+        expect(client)
+          .to receive(:put)
+          .with("/cards/abcdef123456789123456789/idBoard", payload)
+
+        card.move_to_board(other_board)
+      end
+
+      it 'can be moved to a list on another board' do
+        other_board = double(id: '987654321987654321fedcba')
+        other_list = double(id: '987654321987654321aalist')
+        payload = {value: other_board.id, idList: other_list.id}
+
+        expect(client)
+          .to receive(:put)
+          .with("/cards/abcdef123456789123456789/idBoard", payload)
+
+        card.move_to_board(other_board, other_list)
+      end
+
+      it 'can be moved to a list on the same board' do
+        current_board = double(id: 'abcdef123456789123456789')
+        other_list = double(
+          id: '987654321987654321fedcba',
+          board_id: 'abcdef123456789123456789'
+        )
+        allow(List).to receive(:find).with('987654321987654321fedcba').
+          and_return(other_list)
+        allow(card).to receive(:board).and_return(current_board)
+        payload = {value: other_list.id}
+
+        expect(client)
+          .to receive(:put)
+          .with('/cards/abcdef123456789123456789/idList', payload)
+
+        card.move_to_list_on_any_board(other_list.id)
+      end
+
+      it 'can be moved to a list on another board' do
+        current_board = double(id: 'abcdef123456789123456789')
+        other_board = double(id: '987654321987654321fedcba')
+        other_list = double(
+          id: '987654321987654321aalist',
+          board_id: '987654321987654321fedcba'
+        )
+        allow(List).to receive(:find).with('987654321987654321aalist').
+          and_return(other_list)
+        allow(card).to receive(:board).and_return(current_board)
+        allow(Board).to receive(:find).with('987654321987654321fedcba').
+          and_return(other_board)
+        payload = { value: other_board.id, idList: other_list.id }
+
+        expect(client)
+          .to receive(:put)
+          .with('/cards/abcdef123456789123456789/idBoard', payload)
+
+        card.move_to_list_on_any_board(other_list.id)
+      end
+
+      it 'should not be moved if new board is identical with old board', focus: true do
+        other_board = double(id: 'abcdef123456789123456789')
+        expect(client).to_not receive(:put)
+        card.move_to_board(other_board)
+      end
+    end
+
+    context "members" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789", {})
+          .and_return JSON.generate(boards_details.first)
+
+        allow(client)
+          .to receive(:get)
+          .with("/members/abcdef123456789123456789")
+          .and_return user_payload
+      end
+
+      it "has a list of members" do
+        expect(card.board).to_not be_nil
+        expect(card.members).to_not be_nil
+      end
+
+      it "allows a member to be added to a card" do
+        new_member = double(id: '4ee7df3ce582acdec80000b2')
+        payload = {
+          value: new_member.id
+        }
+
+        expect(client)
+          .to receive(:post)
+          .with("/cards/abcdef123456789123456789/members", payload)
+
+        card.add_member(new_member)
+      end
+
+      it "allows a member to be removed from a card" do
+        existing_member = double(id: '4ee7df3ce582acdec80000b2')
+
+        expect(client)
+          .to receive(:delete)
+          .with("/cards/abcdef123456789123456789/members/#{existing_member.id}")
+
+        card.remove_member(existing_member)
+      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)
+          .to receive(:post)
+          .with("/cards/abcdef123456789123456789/actions/comments", { text: 'testing' })
+          .and_return JSON.generate(boards_details.first)
+
+        card.add_comment "testing"
+      end
+    end
+
+    context "labels" do
+      before do
+        allow(client)
+          .to receive(:post)
+          .with("/cards/ebcdef123456789123456789/labels", { value: 'green' })
+          .and_return "not important"
+
+        allow(client)
+          .to receive(:delete)
+          .with("/cards/ebcdef123456789123456789/labels/green")
+          .and_return "not important"
+      end
+
+      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('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
+        expect(client).to receive(:delete).once.with("/cards/abcdef123456789123456789/idLabels/abcdef123456789123456789")
+        label = Label.new(label_details.first)
+        card.remove_label(label)
+      end
+
+      it "can add a label" do
+        expect(client).to receive(:post).once.with("/cards/abcdef123456789123456789/idLabels", {:value => "abcdef123456789123456789"})
+        label = Label.new(label_details.first)
+        card.add_label label
+      end
+
+      it "throws an error when trying to add a invalid label" do
+        allow(client).to receive(:post).with("/cards/abcdef123456789123456789/idLabels", { value: 'abcdef123456789123456789' }).
+          and_return "not important"
+        label = Label.new(label_details.first)
+        label.name = nil
+        card.add_label(label)
+        expect(card.errors.full_messages.to_sentence).to eq("Label is not valid.")
+      end
+
+      it "throws an error when trying to remove a invalid label" do
+        allow(client).to receive(:delete).with("/cards/abcdef123456789123456789/idLabels/abcdef123456789123456789").
+          and_return "not important"
+        label = Label.new(label_details.first)
+        label.name = nil
+        card.remove_label(label)
+        expect(card.errors.full_messages.to_sentence).to eq("Label is not valid.")
+      end
+    end
+
+    context "plugins" do
+      it "can list the existing plugins with correct fields" do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789", {})
+          .and_return JSON.generate(boards_details.first)
+
+        allow(client)
+          .to receive(:get)
+          .with("/cards/abcdef123456789123456789/pluginData", {})
+          .and_return plugin_data_payload
+
+        expect(card.board).to_not be_nil
+        expect(card.plugin_data).to_not be_nil
+
+        first_plugin = card.plugin_data.first
+        expect(first_plugin.id).to eq plugin_data_details[0]["id"]
+        expect(first_plugin.idPlugin).to eq plugin_data_details[0]["idPlugin"]
+        expect(first_plugin.scope).to eq plugin_data_details[0]["scope"]
+        expect(first_plugin.idModel).to eq plugin_data_details[0]["idModel"]
+        expect(first_plugin.value).to eq JSON.parse plugin_data_details[0]["value"]
+        expect(first_plugin.access).to eq plugin_data_details[0]["access"]
+
+        second_plugin = card.plugin_data[1]
+        expect(second_plugin.id).to eq plugin_data_details[1]["id"]
+        expect(second_plugin.idPlugin).to eq plugin_data_details[1]["idPlugin"]
+        expect(second_plugin.scope).to eq plugin_data_details[1]["scope"]
+        expect(second_plugin.idModel).to eq plugin_data_details[1]["idModel"]
+        expect(second_plugin.value).to eq JSON.parse plugin_data_details[1]["value"]
+        expect(second_plugin.access).to eq plugin_data_details[1]["access"]
+
+      end
+    end
+
+    context "attachments" do
+      it "can add an attachment" do
+        f = File.new('spec/list_spec.rb', 'r')
+        allow(client)
+          .to receive(:get)
+          .with("/cards/abcdef123456789123456789/attachments")
+          .and_return attachments_payload
+
+        allow(client)
+          .to receive(:post)
+          .with("/cards/abcdef123456789123456789/attachments", { file: f, name: ''  })
+          .and_return "not important"
+
+        card.add_attachment(f)
+
+        expect(card.errors).to be_empty
+      end
+
+      it "can list the existing attachments with correct fields" do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789", {})
+          .and_return JSON.generate(boards_details.first)
+
+        allow(client)
+          .to receive(:get)
+          .with("/cards/abcdef123456789123456789/attachments")
+          .and_return attachments_payload
+
+        expect(card.board).to_not be_nil
+        expect(card.attachments).to_not be_nil
+
+        first_attachment = card.attachments.first
+        expect(first_attachment.id).to eq attachments_details[0]["id"]
+        expect(first_attachment.name).to eq attachments_details[0]["name"]
+        expect(first_attachment.url).to eq attachments_details[0]["url"]
+        expect(first_attachment.bytes).to eq attachments_details[0]["bytes"]
+        expect(first_attachment.member_id).to eq attachments_details[0]["idMember"]
+        expect(first_attachment.date).to eq Time.parse(attachments_details[0]["date"])
+        expect(first_attachment.is_upload).to eq attachments_details[0]["isUpload"]
+        expect(first_attachment.mime_type).to eq attachments_details[0]["mimeType"]
+        expect(first_attachment.previews).to eq attachments_details[0]["previews"]
+
+        second_attachment = card.attachments[1]
+        expect(second_attachment.previews).to eq nil
+      end
+
+      it "can remove an attachment" do
+        allow(client)
+          .to receive(:delete)
+          .with("/cards/abcdef123456789123456789/attachments/abcdef123456789123456789")
+          .and_return "not important"
+
+        allow(client)
+          .to receive(:get)
+          .with("/cards/abcdef123456789123456789/attachments")
+          .and_return attachments_payload
+
+        card.remove_attachment(card.attachments.first)
+        expect(card.errors).to be_empty
+      end
+    end
+
+    describe "#closed?" do
+      it "returns the closed attribute" do
+        expect(card).to_not be_closed
+      end
+    end
+
+    describe "#close" do
+      it "updates the close attribute to true" do
+        card.close
+        expect(card).to be_closed
+      end
+    end
+
+    describe "#close!" do
+      it "updates the close attribute to true and saves the list" do
+        payload = { closed: true }
+
+        expect(client)
+          .to receive(:put)
+          .once
+          .with("/cards/abcdef123456789123456789", payload)
+
+        card.close!
+      end
+    end
+
+    describe "can mark due_complete" do
+      it "updates the due completed attribute to true" do
+        due_date = Time.now
+
+        payload = {
+          due: due_date,
+        }
+
+        expect(client)
+          .to receive(:put)
+          .once
+          .with("/cards/abcdef123456789123456789", payload)
+
+        card.due = due_date
+        card.save
+
+        expect(card.due).to_not be_nil
+
+        payload = {
+          dueComplete: true
+        }
+
+        expect(client)
+          .to receive(:put)
+          .once
+          .with("/cards/abcdef123456789123456789", payload)
+
+        card.due_complete = true
+        card.save
+
+        expect(card.due_complete).to be true
+      end
+    end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = cards_details.first
+
+        card = Card.new(expected)
+        card.client = client
+
+        card.update_fields({})
+
+        expected.each do |key, value|
+          if card.respond_to?(key) && key != 'labels'
+            expect(card.send(key)).to eq value
+          end
+
+          expect(card.labels).to eq expected['labels'].map { |lbl| Trello::Label.new(lbl) }
+          expect(card.short_id).to eq expected['idShort']
+          expect(card.short_url).to eq expected['shortUrl']
+          expect(card.board_id).to eq expected['idBoard']
+          expect(card.member_ids).to eq expected['idMembers']
+          expect(card.cover_image_id).to eq expected['idAttachmentCover']
+          expect(card.list_id).to eq expected['idList']
+          expect(card.card_labels).to eq expected['idLabels']
+        end
+      end
+    end
+  end
+end