ruby-trello 2.2.0 → 3.1.0

data/spec/unit/trello/board/update_spec.rb

@@ -0,0 +1,197 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Board#update!' do
+
+  let(:client) { Trello.client }
+  let(:board) { Trello::Board.new(id: 'abc123') }
+
+  before { allow(board).to receive(:from_response) }
+
+  it 'can update name through client' do
+    board.name = 'Board 1'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'name' => 'Board 1'
+      })
+
+    board.update!
+  end
+
+  it 'can update description through client' do
+    board.description = 'description ...'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'desc' => 'description ...'
+      })
+
+    board.update!
+  end
+
+  it 'can update closed through client' do
+    board.closed = true
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'closed' => true
+      })
+
+    board.update!
+  end
+
+  # it 'can update subscribed through client' do
+  # end
+
+  it 'can update organization_id through client' do
+    board.organization_id = 123
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'idOrganization' => 123
+      })
+
+    board.update!
+  end
+
+  it 'can update visibility_level through client' do
+    board.visibility_level = 'org'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/permissionLevel' => 'org'
+      })
+
+    board.update!
+  end
+
+  it 'can update enable_self_join through client' do
+    board.enable_self_join = true
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/selfJoin' => true
+      })
+
+    board.update!
+  end
+
+  it 'can update enable_card_covers through client' do
+    board.enable_card_covers = true
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/cardCovers' => true
+      })
+
+    board.update!
+  end
+
+  # it 'can update hide_votes through client' do
+  # end
+
+  it 'can update invitation_permission_level through client' do
+    board.invitation_permission_level = 'org'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/invitations' => 'org'
+      })
+
+    board.update!
+  end
+
+  it 'can update voting_permission_level through client' do
+    board.voting_permission_level = 'org'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/voting' => 'org'
+      })
+
+    board.update!
+  end
+
+  it 'can update comment_permission_level through client' do
+    board.comment_permission_level = 'org'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/comments' => 'org'
+      })
+
+    board.update!
+  end
+
+  it 'can update background_color through client' do
+    board.background_color = 'blue'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/background' => 'blue'
+      })
+
+    board.update!
+  end
+
+  it 'can update card_aging_type through client' do
+    board.card_aging_type = 'pirate'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'prefs/cardAging' => 'pirate'
+      })
+
+    board.update!
+  end
+
+  # it 'can update enable_calendar_feed through client' do
+  # end
+
+  it 'can update all through client' do
+    board.name = 'Board 1'
+    board.description = 'description ...'
+    board.closed = true
+    board.organization_id = 123
+    board.visibility_level = 'org'
+    board.enable_self_join = true
+    board.enable_card_covers = true
+    board.invitation_permission_level = 'org'
+    board.voting_permission_level = 'org'
+    board.comment_permission_level = 'org'
+    board.background_color = 'blue'
+    board.card_aging_type = 'pirate'
+
+    expect(client)
+      .to receive(:put)
+      .with('/boards/abc123', {
+        'name' => 'Board 1',
+        'desc' => 'description ...',
+        'closed' => true,
+        'idOrganization' => 123,
+        'prefs/permissionLevel' => 'org',
+        'prefs/selfJoin' => true,
+        'prefs/cardCovers' => true,
+        'prefs/invitations' => 'org',
+        'prefs/voting' => 'org',
+        'prefs/comments' => 'org',
+        'prefs/background' => 'blue',
+        'prefs/cardAging' => 'pirate'
+      })
+
+    board.update!
+  end
+
+end

data/spec/unit/trello/card_spec.rb

@@ -0,0 +1,103 @@
+require 'spec_helper'
+
+RSpec.describe Trello::Card do
+
+  describe '.path_name' do
+    specify { expect(Trello::Card.path_name).to eq('card') }
+  end
+
+  describe '#closed?' do
+    specify { expect(Trello::Card.new('closed' => true).closed?).to eq(true) }
+    specify { expect(Trello::Card.new('closed' => false).closed?).to eq(false) }
+  end
+
+  describe '#close' do
+    let(:card) { Trello::Card.new('closed' => false) }
+
+    it 'can mark card as closed' do
+      card.close
+      expect(card.closed).to eq(true)
+    end
+  end
+
+  describe 'close!' do
+    let(:card) { Trello::Card.new('closed' => false) }
+
+    it 'will call save' do
+      expect(card).to receive(:save)
+      card.close!
+    end
+  end
+
+  describe 'valid?' do
+    let(:card) { Trello::Card.new('name' => name, 'idList' => list_id) }
+
+    context 'when name is nil' do
+      let(:name) { nil }
+      let(:list_id) { 1 }
+
+      specify { expect(card.valid?).to eq(false) }
+    end
+
+    context 'when list_id is nil' do
+      let(:name) { 'Card Name' }
+      let(:list_id) { nil }
+
+      specify { expect(card.valid?).to eq(false) }
+    end
+
+    context "when name and list_id both aren't nil" do
+      let(:name) { 'Card' }
+      let(:list_id) { 1 }
+
+      specify { expect(card.valid?).to eq(true) }
+    end
+  end
+
+  describe '#move_to_list_on_any_board' do
+    let(:card) { Trello::Card.new(board_id: 'board-aaa') }
+    let(:move_to_list_on_any_board) { card.move_to_list_on_any_board('listid') }
+
+    let(:board) { Trello::Board.new(id: 'board-aaa') }
+    let(:target_list) { Trello::List.new(board_id: target_board_id) }
+
+    before do
+      allow(Trello::List).to receive(:find).with('listid').and_return(target_list)
+      allow(card).to receive(:board).and_return(board)
+    end
+
+    context 'when target list is in the same board' do
+      let(:target_board_id) { 'board-aaa' }
+
+      it 'call #move_to_list on card' do
+        expect(card).to receive(:move_to_list).with('listid')
+
+        move_to_list_on_any_board
+      end
+    end
+
+    context 'when target list is in a different board' do
+      let(:target_board_id) { 'board-bbb' }
+      let(:target_board) { Trello::Board.new(id: 'board-bbb') }
+
+      before { allow(Trello::Board).to receive(:find).with('board-bbb').and_return(target_board) }
+
+      it 'call #move_to_board on card' do
+        expect(card).to receive(:move_to_board).with(target_board, target_list)
+
+        move_to_list_on_any_board
+      end
+    end
+  end
+
+  describe '#created_at' do
+    it 'can get created_at from card id' do
+      card = Trello::Card.new('id' => '5e95d1b4f43f9a06497f17f7')
+      expect(card.created_at).to eq(Time.new(2020, 4, 14, 15, 7, 32, '+00:00'))
+
+      card = Trello::Card.new('id' => '5e94ee7df8784d61ee48319e')
+      expect(card.created_at).to eq(Time.new(2020, 4, 13, 22, 58, 05, '+00:00'))
+    end
+  end
+
+end

data/spec/unit/trello/member/new_spec.rb

@@ -0,0 +1,62 @@
+
+require 'spec_helper'
+
+RSpec.describe 'Trello::Member#new' do
+
+  let(:member) { Trello::Member.new(data) }
+
+  describe 'parse #id (readonly primary_key)' do
+    context 'with Trello API respone data' do
+      let(:data) { { 'id' => '5e93ba98614ac22d22f085c4' } }
+
+      it "parse from data['id']" do
+        expect(member.id).to eq('5e93ba98614ac22d22f085c4')
+      end
+    end
+
+    context 'with Ruby-like data' do
+      let(:data) { { id: '5e93ba98614ac22d22f085c4' } }
+
+      it 'parse from data[:id]' do
+        expect(member.id).to eq('5e93ba98614ac22d22f085c4')
+      end
+    end
+  end
+
+  describe 'parse #username (writable)' do
+    context 'with Trello API respone data' do
+      let(:data) { { 'username' => 'hoppertest' } }
+
+      it "parse from data['username']" do
+        expect(member.username).to eq('hoppertest')
+      end
+    end
+
+    context 'with Ruby-like data' do
+      let(:data) { { username: 'hoppertest' } }
+
+      it 'parse from data[:id]' do
+        expect(member.username).to eq('hoppertest')
+      end
+    end
+  end
+
+  describe 'parse #full_name (writable)' do
+    context 'with Trello API respone data' do
+      let(:data) { { 'fullName' => 'hoppertest' } }
+
+      it "parse from data['fullName']" do
+        expect(member.full_name).to eq('hoppertest')
+      end
+    end
+
+    context 'with Ruby-like data' do
+      let(:data) { { full_name: 'hoppertest' } }
+
+      it 'parse from data[:full_name]' do
+        expect(member.full_name).to eq('hoppertest')
+      end
+    end
+  end
+
+end

data/spec/unit/trello/member/update_fields_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Member#update_fields' do
+
+  let(:member_params) { {
+    'id' => '5e93ba403f1ab3603ba81a09',
+    'bio' => 'test',
+    'username' => 'hoppertest',
+    'email' => 'test@test.com'
+   } }
+
+  let(:member) { Trello::Member.new(member_params) }
+
+  context 'when the fields argument is empty' do
+    let(:fields) { {} }
+
+    it 'member does not set any fields' do
+      member.update_fields(fields)
+
+      expect(member.changed?).to be_falsy
+      expect(member.id).to eq(member_params['id'])
+      expect(member.username).to eq(member_params['username'])
+      expect(member.email).to eq(member_params['email'])
+    end
+  end
+
+  context 'when the fields argument has at least one field' do
+
+    context 'and the field does changed' do
+      let(:fields) { { username: 'hopperchange' } }
+
+      it 'member does set the changed fields' do
+        member.update_fields(fields)
+
+        expect(member.username).to eq('hopperchange')
+      end
+
+      it 'member has been mark as changed' do
+        member.update_fields(fields)
+
+        expect(member.changed?).to be_truthy
+      end
+    end
+
+    context "and the field doesn't changed" do
+      let(:fields) { { username: member_params['username'] } }
+
+      it "member attributes doesn't changed" do
+        member.update_fields(fields)
+
+        expect(member.username).to eq(member_params['username'])
+      end
+
+      it "member hasn't been mark as changed" do
+        member.update_fields(fields)
+
+        expect(member.changed?).to be_falsy
+      end
+    end
+
+    context "and the field isn't a correct attributes of the card" do
+      let(:fields) { { abc: 'abc' } }
+
+      it "card attributes doesn't changed" do
+        member.update_fields(fields)
+
+        expect(member.id).to eq(member_params['id'])
+        expect(member.username).to eq(member_params['username'])
+        expect(member.email).to eq(member_params['email'])
+      end
+
+      it "card hasn't been mark as changed" do
+        member.update_fields(fields)
+
+        expect(member.changed?).to be_falsy
+      end
+    end
+  end
+
+end