ruby-trello 2.0.1 → 3.0.0

data/spec/integration/organization/associations/actions_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Organization#actions' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can get actions' do
+    VCR.use_cassette('can_get_actions_of_organization') do
+      organization = Trello::Organization.find('5e93ba154634282b6df23bcc')
+      actions = organization.actions
+
+      expect(actions).to be_a(Array)
+      expect(actions[0]).to be_a(Trello::Action)
+    end
+  end
+
+end

data/spec/integration/organization/associations/boards_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Organization#boards' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can get boards' do
+    VCR.use_cassette('can_get_boards_of_organization') do
+      organization = Trello::Organization.find('5e93ba154634282b6df23bcc')
+      boards = organization.boards
+
+      expect(boards).to be_a(Array)
+      expect(boards[0]).to be_a(Trello::Board)
+    end
+  end
+
+end

data/spec/integration/organization/associations/members_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Organization#members' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can get members' do
+    VCR.use_cassette('can_get_members_of_organization') do
+      organization = Trello::Organization.find('5e93ba154634282b6df23bcc')
+      members = organization.members
+
+      expect(members).to be_a(Array)
+      expect(members[0]).to be_a(Trello::Member)
+    end
+  end
+
+end

data/spec/integration/organization/create_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Organization.create' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can success create the organization' do
+    VCR.use_cassette('can_success_create_a_organization') do
+      organization = Trello::Organization.create(
+        name: 'integrationtest98',
+        display_name: 'Integration Test 98',
+        description: 'This is a team for doing integration tests.'
+      )
+      expect(organization).to be_a(Trello::Organization)
+
+      expect(organization.id).not_to be_nil
+      expect(organization.name).to eq('integrationtest98')
+      expect(organization.display_name).to eq('Integration Test 98')
+      expect(organization.description).to eq('This is a team for doing integration tests.')
+    end
+  end
+end

data/spec/integration/organization/find_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Organization#find' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'find with id and get all fields' do
+    VCR.use_cassette('organization_find_with_id_and_get_all_fields') do
+      organization = Trello::Organization.find('5e93ba154634282b6df23bcc')
+      expect(organization).to be_a(Trello::Organization)
+
+      expect(organization.id).not_to be_nil
+      expect(organization.name).not_to be_nil
+      expect(organization.display_name).not_to be_nil
+      expect(organization.team_type).not_to be_nil
+      expect(organization.description).not_to be_nil
+      expect(organization.url).not_to be_nil
+    end
+  end
+
+  it 'find with id and get specific fields' do
+    VCR.use_cassette('organization_find_with_id_and_get_specific_fields') do
+      organization = Trello::Organization.find('5e93ba154634282b6df23bcc', fields: 'name,url')
+      expect(organization).to be_a(Trello::Organization)
+
+      expect(organization.id).not_to be_nil
+      expect(organization.name).not_to be_nil
+      expect(organization.url).not_to be_nil
+
+      expect(organization.display_name).to be_nil
+      expect(organization.team_type).to be_nil
+      expect(organization.description).to be_nil
+    end
+  end
+
+end

data/spec/integration/organization/update!_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+RSpec.describe 'Trell::Organization.update!' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can update! a organization' do
+    VCR.use_cassette('can_success_update_bong_a_organization') do
+      organization = Trello::Organization.find('5f60d9f72c17b23dbb2d9d79')
+
+      expect(organization.name).to eq('integrationtest98')
+      expect(organization.display_name).to eq('Integration Test 98')
+      expect(organization.description).to eq('This is a team for doing integration tests.')
+
+      organization.name = 'integrationtest99'
+      organization.display_name = 'Integration Test 99'
+      organization.description = 'This is a team for doing integration tests for 99'
+
+      organization.update!
+
+      expect(organization.name).to eq('integrationtest99')
+      expect(organization.display_name).to eq('Integration Test 99')
+      expect(organization.description).to eq('This is a team for doing integration tests for 99')
+    end
+  end
+
+end

data/spec/integration/token/find_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Token#find' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'find with token string' do
+    VCR.use_cassette('token_find_with_token_string') do
+      token = Trello::Token.find('MEMBER_TOKEN')
+      expect(token).to be_a(Trello::Token)
+
+      expect(token.id).not_to be_nil
+      expect(token.identifier).not_to be_nil
+      expect(token.member_id).not_to be_nil
+      expect(token.created_at).not_to be_nil
+      # expect(token.expires_at).not_to be_nil
+      expect(token.permissions).not_to be_nil
+      expect(token.webhooks).not_to be_nil
+      expect(token.webhooks[0]).to be_a(Trello::Webhook)
+    end
+  end
+
+end

data/spec/integration/webhook/create_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Webhook.create' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can success create the custom_field' do
+    VCR.use_cassette('can_success_create_a_webhook') do
+      webhook = Trello::Webhook.create(
+        description: 'test',
+        model_id: '5fa759f1564c6e5bb803c61e',
+        callback_url: 'https://example.com/webhook/trello_callbacks',
+        active: true
+      )
+      expect(webhook).to be_a(Trello::Webhook)
+
+      expect(webhook.id).not_to be_nil
+      expect(webhook.description).to eq('test')
+      expect(webhook.model_id).to eq('5fa759f1564c6e5bb803c61e')
+      expect(webhook.callback_url).to eq('https://example.com/webhook/trello_callbacks')
+      expect(webhook.active).to eq(true)
+      expect(webhook.consecutive_failures).to eq(0)
+      expect(webhook.first_consecutive_fail_date).to be_nil
+    end
+  end
+end

data/spec/integration/webhook/delete_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Webhook#delete' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'find with id' do
+    VCR.use_cassette('webhook_delete') do
+      webhook = Trello::Webhook.find('5fa8a1e0009b2a6a669e6efa')
+      expect(webhook).to be_a(Trello::Webhook)
+
+      webhook.delete
+
+      expect {
+        Trello::Webhook.find('5fa8a1e0009b2a6a669e6efa')
+      }.to raise_error Trello::Error, 'The requested resource was not found.'
+    end
+  end
+
+end

data/spec/integration/webhook/find_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+RSpec.describe 'Trello::Webhook#find' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'find with id' do
+    VCR.use_cassette('webhook_find_with_id') do
+      webhook = Trello::Webhook.find('5fa8a1e0009b2a6a669e6efa')
+      expect(webhook).to be_a(Trello::Webhook)
+
+      expect(webhook.id).not_to be_nil
+      expect(webhook.description).not_to be_nil
+      expect(webhook.model_id).not_to be_nil
+      expect(webhook.callback_url).not_to be_nil
+      expect(webhook.active).not_to be_nil
+      expect(webhook.consecutive_failures).not_to be_nil
+    end
+  end
+
+end

data/spec/integration/webhook/update!_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+RSpec.describe 'Trell::Webhook.update!' do
+  include IntegrationHelpers
+
+  before { setup_trello }
+
+  it 'can update! a webhook' do
+    VCR.use_cassette('can_success_update_bong_a_webhook') do
+      webhook = Trello::Webhook.find('5fa8a1e0009b2a6a669e6efa')
+
+      expect(webhook.description).to eq('test')
+
+      webhook.description = 'Test - Changed'
+
+      webhook.update!
+
+      expect(webhook.description).to eq('Test - Changed')
+      webhook = Trello::Webhook.find('5fa8a1e0009b2a6a669e6efa')
+      expect(webhook.description).to eq('Test - Changed')
+    end
+  end
+
+end

data/spec/item_spec.rb

@@ -31,7 +31,7 @@ module Trello
     end
 
     it 'knows its pos' do
-      expect(item.pos).to eq details['pos']
+      expect(item.position).to eq details['pos']
     end
 
     describe '#complete?' do
@@ -59,7 +59,7 @@ module Trello
           'name' => 'name',
           'type' => 'type',
           'state' => 'state',
-          'pos' => 'pos'
+          'pos' => 'position'
         }
 
         item = Item.new(expected)

data/spec/label_spec.rb

@@ -20,7 +20,7 @@ module Trello
       it "delegates to Trello.client#find" do
         expect(client)
           .to receive(:find)
-          .with(:label, 'abcdef123456789123456789', {})
+          .with('label', 'abcdef123456789123456789', {})
 
         Label.find('abcdef123456789123456789')
       end
@@ -43,7 +43,6 @@ module Trello
         expect(label.color).to    eq details['color']
         expect(label.name).to     eq details['name']
         expect(label.id).to       eq details['id']
-        expect(label.uses).to     eq details['uses']
         expect(label.board_id).to eq details['idBoard']
       end
 
@@ -71,14 +70,16 @@ module Trello
 
         result = JSON.generate(cards_details.first.merge(payload.merge(idBoard: boards_details.first['id'])))
 
-        expected_payload = {name: "Test Label", color: nil, idBoard: "abcdef123456789123456789" }
+        expected_payload = {'name' => 'Test Label', 'color' => 'yellow', 'idBoard' => 'abcdef123456789123456789' }
 
-        expect(client)
+        expect(Label.client)
           .to receive(:post)
-          .with("/labels", expected_payload)
+          .with('/labels', expected_payload)
           .and_return result
 
-        label = Label.create(label_details.first.merge(payload.merge(board_id: boards_details.first['id'])))
+        params = label_details.first.merge(payload.merge(board_id: boards_details.first['id']))
+        params.delete('id')
+        label = Label.create params
 
         expect(label).to be_a Label
       end
@@ -89,13 +90,19 @@ module Trello
         expected_new_name = "xxx"
 
         payload = {
-          name: expected_new_name,
+          'name' => expected_new_name,
         }
 
         expect(client)
           .to receive(:put)
           .once
           .with("/labels/abcdef123456789123456789", payload)
+          .and_return({
+            "id" => "abcdef123456789123456789",
+            "idBoard" => "5e70f5bed3f34a49e2f11409",
+            "name" => "xxx",
+            "color" => "purple"
+          }.to_json)
 
         label.name = expected_new_name
         label.save
@@ -105,13 +112,19 @@ module Trello
         expected_new_color = "purple"
 
         payload = {
-          color: expected_new_color,
+          'color' => expected_new_color
         }
 
         expect(client)
           .to receive(:put)
           .once
           .with("/labels/abcdef123456789123456789", payload)
+          .and_return({
+            "id" => "abcdef123456789123456789",
+            "idBoard" => "5e70f5bed3f34a49e2f11409",
+            "name" => "",
+            "color" => "purple"
+          }.to_json)
 
         label.color = expected_new_color
         label.save
@@ -121,26 +134,19 @@ module Trello
         %w(green yellow orange red purple blue sky lime pink black).each do |color|
           allow(client)
             .to receive(:put)
-            .with("/labels/abcdef123456789123456789", {color: color})
-            .and_return "not important"
+            .with("/labels/abcdef123456789123456789", {'color' => color})
+            .and_return({
+              "id" => "abcdef123456789123456789",
+              "idBoard" => "5e70f5bed3f34a49e2f11409",
+              "name" => "",
+              "color" => color
+            }.to_json)
 
           label.color = color
           label.save
           expect(label.errors).to be_empty
         end
       end
-
-      it "throws an error when trying to update a label with an unknown colour" do
-        allow(client)
-          .to receive(:put)
-          .with("/labels/abcdef123456789123456789", {})
-          .and_return "not important"
-
-        label.color = 'mauve'
-        label.save
-
-        expect(label.errors.full_messages.to_sentence).to eq("Label color 'mauve' does not exist")
-      end
     end
 
     context "deleting" do
@@ -162,10 +168,6 @@ module Trello
         expect(label.name).to_not be_nil
       end
 
-      it "gets its usage" do
-        expect(label.uses).to_not be_nil
-      end
-
       it "gets its color" do
         expect(label.color).to_not be_nil
       end
@@ -188,8 +190,7 @@ module Trello
           'id' => 'id',
           'name' => 'name',
           'color' => 'color',
-          'idBoard' => 'board_id',
-          'uses' => 'uses'
+          'idBoard' => 'board_id'
         }
 
         label = Label.new(expected)
@@ -201,5 +202,81 @@ module Trello
         end
       end
     end
+
+    describe '#update_fields' do
+      let(:label_detail) {{
+        'id' => 'id',
+        'name' => 'name',
+        'color' => 'color',
+        'idBoard' => 'board_id'
+      }}
+      let(:label) { Label.new(label_detail) }
+
+      context 'when the fields argument is empty' do
+        let(:fields) { {} }
+
+        it 'card does not set any fields' do
+          label.update_fields(fields)
+
+          expect(label.id).to eq label_detail['id']
+          expect(label.name).to eq label_detail['name']
+          expect(label.color).to eq label_detail['color']
+          expect(label.board_id).to eq label_detail['idBoard']
+        end
+      end
+
+      context 'when the fields argument has at least one field' do
+        context 'and the field does changed' do
+          let(:fields) { { name: 'Awesome Name' } }
+
+          it 'label does set the changed fields' do
+            label.update_fields(fields)
+
+            expect(label.name).to eq('Awesome Name')
+          end
+
+          it 'label has been mark as changed' do
+            label.update_fields(fields)
+
+            expect(label.changed?).to be_truthy
+          end
+        end
+
+        context "and the field doesn't changed" do
+          let(:fields) { { name: label_detail['name'] } }
+
+          it "label attributes doesn't changed" do
+            label.update_fields(fields)
+
+            expect(label.name).to eq(label_detail['name'])
+          end
+
+          it "label hasn't been mark as changed" do
+            label.update_fields(fields)
+
+            expect(label.changed?).to be_falsy
+          end
+        end
+
+        context "and the field isn't a correct attributes of the label" do
+          let(:fields) { { abc: 'abc' } }
+
+          it "label attributes doesn't changed" do
+            label.update_fields(fields)
+
+            expect(label.id).to eq label_detail['id']
+            expect(label.name).to eq label_detail['name']
+            expect(label.color).to eq label_detail['color']
+            expect(label.board_id).to eq label_detail['idBoard']
+          end
+
+          it "label hasn't been mark as changed" do
+            label.update_fields(fields)
+
+            expect(label.changed?).to be_falsy
+          end
+        end
+      end
+    end
   end
 end