ruby-trello-czuger 2.0.0

data/spec/array_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+require 'trello/core_ext/array'
+
+describe Array, '#jsoned_into' do
+  include Helpers
+
+  it "should convert an array of parsed json into cards" do
+    expected = cards_details.map do |card_details|
+      card_details.jsoned_into(Trello::Card)
+    end
+    expect(cards_details.jsoned_into(Trello::Card)).to eq(expected)
+  end
+end

data/spec/association_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+module Trello
+  describe Association do
+    include Helpers
+
+    let(:organization) { client.find(:organization, '4ee7e59ae582acdec8000291') }
+    let(:client) { Client.new(consumer_key: 'xxx') }
+
+    before do
+      allow(client)
+        .to receive(:get)
+        .with('/organizations/4ee7e59ae582acdec8000291', {})
+        .and_return organization_payload
+
+      allow(client)
+        .to receive(:get)
+        .with('/organizations/4ee7e59ae582acdec8000291/boards/all')
+        .and_return boards_payload
+    end
+
+    it 'should set the proper client for all associated boards of the organization' do
+      expect(organization.boards.first.client.consumer_key).to eq 'xxx'
+    end
+  end
+end

data/spec/basic_auth_policy_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+include Trello
+include Trello::Authorization
+
+describe BasicAuthPolicy do
+  before do
+    BasicAuthPolicy.developer_public_key = 'xxx'
+    BasicAuthPolicy.member_token = 'xxx'
+  end
+
+  it 'adds key and token query parameters' do
+    BasicAuthPolicy.developer_public_key = 'xxx_developer_public_key_xxx'
+    BasicAuthPolicy.member_token = 'xxx_member_token_xxx'
+
+    uri = Addressable::URI.parse('https://xxx/')
+
+    request = Request.new :get, uri
+
+    authorized_request = BasicAuthPolicy.authorize request
+
+    the_query_parameters = Addressable::URI.parse(authorized_request.uri).query_values
+
+    expect(the_query_parameters['key']).to eq 'xxx_developer_public_key_xxx'
+    expect(the_query_parameters['token']).to eq 'xxx_member_token_xxx'
+  end
+
+  it 'preserves other query parameters' do
+    uri = Addressable::URI.parse('https://xxx/?name=Phil')
+
+    request = Request.new :get, uri, {example_header: 'example_value'}
+
+    authorized_request = BasicAuthPolicy.authorize request
+
+    the_query_parameters = Addressable::URI.parse(authorized_request.uri).query_values
+
+    expect(the_query_parameters['name']).to eq 'Phil'
+  end
+
+  it 'preserves headers' do
+    uri = Addressable::URI.parse('https://xxx/')
+
+    request = Request.new :get, uri, {example_header: 'example_value'}
+
+    authorized_request = BasicAuthPolicy.authorize request
+
+    expect(authorized_request.headers).to eq request.headers
+  end
+
+  it 'does what when a query parameter already exists called key or token?'
+end

data/spec/board_spec.rb

@@ -0,0 +1,442 @@
+require 'spec_helper'
+
+module Trello
+  describe Board do
+    include Helpers
+
+    let(:board) { client.find(:board, 'abcdef123456789123456789') }
+    let(:client) { Client.new }
+    let(:member) { Member.new(user_payload) }
+
+    before do
+      allow(client)
+        .to receive(:get)
+        .with("/boards/abcdef123456789123456789", {})
+        .and_return JSON.generate(boards_details.first)
+    end
+
+    context "finding" do
+      let(:client) { Trello.client }
+
+      it "delegates to client#find" do
+        expect(client)
+          .to receive(:find)
+          .with(:board, 'abcdef123456789123456789', {})
+
+        Board.find('abcdef123456789123456789')
+      end
+
+      it "is equivalent to client#find" do
+        expect(Board.find('abcdef123456789123456789')).to eq(board)
+      end
+    end
+
+    context "self.all" do
+      let(:client) { Trello.client }
+
+      before do
+        allow(Member)
+          .to receive_message_chain(:find, :username)
+          .and_return "testuser"
+
+        allow(client)
+          .to receive(:get)
+          .with("/members/testuser/boards")
+          .and_return boards_payload
+      end
+
+      it "gets all boards" do
+        expect(Board.all.first).to eq Board.new(boards_details.first)
+      end
+    end
+
+    context "fields" do
+      it "gets an id" do
+        expect(board.id).to_not be_nil
+      end
+
+      it "gets a name" do
+        expect(board.name).to_not be_nil
+      end
+
+      it "gets the description" do
+        expect(board.description).to_not be_nil
+      end
+
+      it "knows if it is closed or open" do
+        expect(board).to_not be_closed
+      end
+
+      it "knows if it is starred or not" do
+        expect(board).to_not be_starred
+      end
+
+      it "gets its url" do
+        expect(board.url).to_not be_nil
+      end
+
+      it "gets its last_activity_date" do
+        expect(board.last_activity_date).to_not be_nil
+      end
+
+      it "gets a Time object for last_activity_date" do
+        expect(board.last_activity_date).to be_a(Time)
+      end
+    end
+
+    context "actions" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/actions", {filter: :all})
+          .and_return actions_payload
+      end
+
+      it "has a list of actions" do
+        expect(board.actions.count).to be > 0
+      end
+    end
+
+    context "cards" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/cards", {filter: :open})
+          .and_return cards_payload
+      end
+
+      it "gets its list of cards" do
+        expect(board.cards.count).to be > 0
+      end
+    end
+
+    context "labels" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/labels")
+          .and_return label_payload
+
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/labelnames")
+          .and_return label_name_payload
+      end
+
+      it "gets the specific labels for the board" do
+        allow(client).to receive(:get).with("/boards/abcdef123456789123456789/labels", {:limit => 1000}).
+          and_return label_payload
+        labels = board.labels
+
+        expect(labels.count).to eq(4)
+        expect(labels[2].color).to  eq('red')
+        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('dbcdef123456789123456789')
+        expect(labels[3].board_id).to  eq('abcdef123456789123456789')
+        expect(labels[3].name).to  eq('on hold')
+        expect(labels[3].uses).to  eq(6)
+      end
+
+      it "passes the label limit" do
+        allow(client).to receive(:get).with("/boards/abcdef123456789123456789/labels", {:limit => 50}).
+          and_return label_payload
+        labels = board.labels(:limit => 50)
+      end
+    end
+
+    context "find_card" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/cards/1")
+          .and_return card_payload
+      end
+
+      it "gets a card" do
+        expect(board.find_card(1)).to be_a(Card)
+      end
+    end
+
+    context "add_member" do
+      before do
+        allow(client)
+          .to receive(:put)
+      end
+
+      it "adds a member to the board as a normal user (default)" do
+        expect(client)
+          .to receive(:put)
+          .with("/boards/abcdef123456789123456789/members/id", type: :normal)
+
+        board.add_member(member)
+      end
+
+      it "adds a member to the board as an admin" do
+        expect(client)
+          .to receive(:put)
+          .with("/boards/abcdef123456789123456789/members/id", type: :admin)
+
+        board.add_member(member, :admin)
+      end
+    end
+
+    context "remove_member" do
+      before do
+        allow(client)
+          .to receive(:delete)
+      end
+
+      it "removes a member from the board" do
+        expect(client)
+          .to receive(:delete)
+          .with("/boards/abcdef123456789123456789/members/id")
+
+        board.remove_member(member)
+      end
+    end
+
+    context "lists" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/lists", hash_including(filter: :open))
+          .and_return lists_payload
+      end
+
+      it "has a list of lists" do
+        expect(board).to be_has_lists
+      end
+    end
+
+    context "members" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/members", hash_including(filter: :all))
+          .and_return JSON.generate([user_details])
+      end
+
+      it "has a list of members" do
+        expect(board.members.count).to be > 0
+      end
+    end
+
+    context "checklists" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/checklists", {filter: :all})
+          .and_return checklists_payload
+      end
+
+      it "has a list of checklists" do
+        expect(board.checklists.count).to be > 0
+      end
+    end
+
+    context "organization" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/organizations/abcdef123456789123456789", {})
+          .and_return JSON.generate(orgs_details.first)
+      end
+
+      it "belongs to an organization" do
+        expect(board.organization).to_not be_nil
+      end
+    end
+
+    it "is not closed" do
+      expect(board).not_to be_closed
+    end
+
+    it "is not starred" do
+      expect(board).not_to be_starred
+    end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+         'id' => "id",
+         'name' => "name",
+         'desc' => "desc",
+         'closed' => false,
+         'starred' => false,
+         'url' => "url",
+         'idOrganization' => "org_id"
+        }
+
+        board = Board.new(expected)
+        board.client = client
+
+        board.update_fields({})
+
+        expected.each_pair do |key, value|
+          if board.respond_to?(key)
+            expect(board.send(key)).to eq value
+          end
+        end
+
+        expect(board.description).to eq expected['desc']
+        expect(board.organization_id).to eq expected['idOrganization']
+      end
+
+      it "initializes all fields from response-like formatted hash" do
+        board_details = boards_details.first
+        board = Board.new(board_details)
+        expect(board.id).to                 eq board_details['id']
+        expect(board.name).to               eq board_details['name']
+        expect(board.description).to        eq board_details['desc']
+        expect(board.closed).to             eq board_details['closed']
+        expect(board.starred).to            eq board_details['starred']
+        expect(board.organization_id).to    eq board_details['idOrganization']
+        expect(board.url).to                eq board_details['url']
+        expect(board.last_activity_date).to eq board_details['dateLastActivity']
+      end
+
+      it "initializes all fields from options-like formatted hash" do
+        board_details = boards_details[1]
+        board = Board.new(board_details)
+        expect(board.name).to            eq board_details[:name]
+        expect(board.description).to     eq board_details[:desc]
+        expect(board.closed).to          eq board_details[:closed]
+        expect(board.starred).to         eq board_details[:starred]
+        expect(board.organization_id).to eq board_details[:organization_id]
+      end
+    end
+
+    describe "#save" do
+      let(:client) { Trello.client }
+
+      let(:any_board_json) do
+        JSON.generate(boards_details.first)
+      end
+
+      before do
+        allow(client)
+          .to receive(:put)
+      end
+
+      it "cannot currently save a new instance" do
+        expect(client).to_not receive(:put)
+        expect {
+          Board.new.save
+        }.to raise_error(Trello::ConfigurationError)
+      end
+
+      it "puts all fields except id" do
+        expected_fields = %w{ name description closed starred idOrganization}.map { |s| s.to_sym }
+
+        expect(client).to receive(:put) do |anything, body|
+          expect(body.keys).to match expected_fields
+          any_board_json
+        end
+
+        Board.new('id' => "xxx").save
+      end
+
+      it "mutates the current instance" do
+        allow(client)
+          .to receive(:put)
+          .and_return any_board_json
+
+        board = Board.new 'id' => "xxx"
+        expect(board.save).to eq board
+      end
+
+      it "uses the correct resource" do
+        expected_resource_id = "xxx_board_id_xxx"
+
+        expect(client).to receive(:put) do |path, anything|
+          expect(path).to match(/#{expected_resource_id}\/\z/)
+          any_board_json
+        end
+
+        Board.new('id' => expected_resource_id).save
+      end
+
+      it "saves OR updates depending on whether or not it has an id set"
+    end
+
+    describe '#update!' do
+      let(:client) { Trello.client }
+
+      let(:any_board_json) do
+        JSON.generate(boards_details.first)
+      end
+
+      it "puts basic attributes" do
+        board = Board.new 'id' => "board_id"
+
+        board.name        = "new name"
+        board.description = "new description"
+        board.closed      = true
+        board.starred     = true
+
+        expect(client)
+          .to receive(:put)
+          .with("/boards/#{board.id}/", {
+            name: "new name",
+            description: "new description",
+            closed: true,
+            starred: true,
+            idOrganization: nil })
+          .and_return any_board_json
+
+        board.update!
+      end
+    end
+
+    describe "Repository" do
+      include Helpers
+
+      let(:client) { Trello.client }
+
+      let(:any_board_json) do
+        JSON.generate(boards_details.first)
+      end
+
+      before do
+        allow(client)
+          .to receive(:post)
+      end
+
+      it "creates a new board with whatever attributes are supplied " do
+        expected_attributes = { name: "Any new board name", description: "Any new board desription" }
+        sent_attributes = { name: expected_attributes[:name], desc: expected_attributes[:description] }
+
+        expect(client)
+          .to receive(:post)
+          .with("/boards", sent_attributes)
+          .and_return any_board_json
+
+        Board.create expected_attributes
+      end
+
+      it "posts to the boards collection" do
+        expect(client)
+          .to receive(:post)
+          .with("/boards", anything)
+          .and_return any_board_json
+
+        Board.create xxx: ""
+      end
+
+      it "returns a board" do
+        allow(client)
+          .to receive(:post)
+          .with("/boards", anything)
+          .and_return any_board_json
+
+        expect(Board.create(xxx: "")).to be_a Board
+      end
+
+      it "at least name is required"
+    end
+  end
+end