troo 0.0.7 → 0.0.8

data/test/lib/troo/actions/create_card_test.rb

@@ -1,54 +1,71 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe CreateCard do
     let(:described_class) { CreateCard }
-    let(:list_id)  { "526d8e130a14a9d846001d97" }
-    let(:card_name) { "My New Card" }
-    let(:description) { "A description to get us started." }
+    let(:list_id)  { '526d8e130a14a9d846001d97' }
+    let(:card_name) { 'My New Card' }
+    let(:description) { 'A description to get us started.' }
 
     before do
       @list = Fabricate(:list)
       @card = Fabricate(:card, name: card_name, desc: description)
-
-      CardPersistence.stubs(:for).returns(@card)
+      Persistence::Card.stubs(:for).returns(@card)
     end
 
     after { database_cleanup }
 
-    describe ".initialize" do
+    describe '.initialize' do
       subject { described_class.new(@list, card_name, description) }
 
-      it "assigns the list to an instance variable" do
-        subject.instance_variable_get("@list").must_equal(@list)
+      it 'assigns the list to an instance variable' do
+        subject.instance_variable_get('@list').must_equal(@list)
       end
 
-      it "assigns the name to an instance variable" do
-        subject.instance_variable_get("@name").must_equal(card_name)
+      it 'assigns the name to an instance variable' do
+        subject.instance_variable_get('@name').must_equal(card_name)
       end
 
-      it "assigns the description to an instance variable" do
-        subject.instance_variable_get("@description").must_equal(description)
+      it 'assigns the description to an instance variable' do
+        subject.instance_variable_get('@description')
+          .must_equal(description)
       end
     end
 
-    describe ".for" do
-      before { VCR.insert_cassette(:create_card, decode_compressed_response: true) }
+    describe '.for' do
+      before do
+        VCR.insert_cassette(:create_card,
+                            decode_compressed_response: true)
+      end
+
       after  { VCR.eject_cassette }
 
       subject { described_class.for(@list, card_name, description) }
 
-      context "when the card was created" do
-        it "returns the new card" do
+      context 'when the card was created' do
+        it 'returns the new card' do
           subject.must_equal(@card)
         end
       end
 
-      context "when the card was not created" do
+      context 'when the card was not created' do
         before { Trello::Card.stubs(:create).raises(Trello::Error) }
 
         it { subject.must_equal false }
       end
+
+      context 'when the access token is invalid' do
+        before do
+          Trello::Card.stubs(:create)
+            .raises(Trello::InvalidAccessToken)
+        end
+
+        subject { described_class.for(@list, card_name, description) }
+
+        it 'catches the exception and re-raises' do
+          proc { subject }.must_raise(Troo::InvalidAccessToken)
+        end
+      end
     end
   end
 end

data/test/lib/troo/actions/create_comment_test.rb

@@ -1,49 +1,71 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe CreateComment do
     let(:described_class) { CreateComment }
-    let(:card_id) { "526d8f19ddb279532e005259" }
-    let(:comment) { "Some much needed feedback..." }
+    let(:card_id) { '526d8f19ddb279532e005259' }
+    let(:comment) { 'Some much needed feedback...' }
 
     before do
       @card = Fabricate(:card)
       @comment = Fabricate(:comment, text: comment)
-
-      CommentPersistence.stubs(:for).returns(@comment)
+      Persistence::Comment.stubs(:for).returns(@comment)
     end
 
     after { database_cleanup }
 
-    describe ".initialize" do
+    describe '.initialize' do
       subject { described_class.new(@card, comment) }
 
-      it "assigns the card to an instance variable" do
-        subject.instance_variable_get("@card").must_equal(@card)
+      it 'assigns the card to an instance variable' do
+        subject.instance_variable_get('@card').must_equal(@card)
       end
 
-      it "assigns the comment to an instance variable" do
-        subject.instance_variable_get("@comment").must_equal(comment)
+      it 'assigns the comment to an instance variable' do
+        subject.instance_variable_get('@comment').must_equal(comment)
       end
     end
 
-    describe ".for" do
-      before { VCR.insert_cassette(:create_comment, decode_compressed_response: true) }
+    describe '.for' do
+      before do
+        VCR.insert_cassette(:create_comment,
+                            decode_compressed_response: true)
+      end
+
       after  { VCR.eject_cassette }
 
       subject { described_class.for(@card, comment) }
 
-      context "when the comment was created" do
-        it "returns the new comment" do
+      context 'when the comment was created' do
+        it 'returns the new comment' do
           subject.must_equal(@comment)
         end
       end
 
-      context "when the comment was not created" do
-        before { Trello::Card.any_instance.stubs(:add_comment).raises(Trello::Error) }
+      context 'when the comment was not created' do
+        before do
+          Trello::Card.any_instance.stubs(:add_comment)
+            .raises(Trello::Error)
+        end
 
         it { subject.must_equal false }
       end
+
+      context 'when the access token is invalid' do
+        let(:card) { stub }
+
+        before do
+          Trello::Card.stubs(:new).returns(card)
+          card.stubs(:update_fields).returns(card)
+          card.stubs(:add_comment).raises(Trello::InvalidAccessToken)
+        end
+
+        subject { described_class.for(@card, comment) }
+
+        it 'catches the exception and re-raises' do
+          proc { subject }.must_raise(Troo::InvalidAccessToken)
+        end
+      end
     end
   end
 end

data/test/lib/troo/actions/create_list_test.rb

@@ -1,49 +1,65 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe CreateList do
     let(:described_class) { CreateList }
-    let(:board_id)  { "526d8e130a14a9d846001d96" }
-    let(:list_name) { "My New List" }
+    let(:board_id)  { '526d8e130a14a9d846001d96' }
+    let(:list_name) { 'My New List' }
 
     before do
       @board = Fabricate(:board)
       @list = Fabricate(:list, name: list_name)
-
-      ListPersistence.stubs(:for).returns(@list)
+      Persistence::List.stubs(:for).returns(@list)
     end
 
     after { database_cleanup }
 
-    describe ".initialize" do
+    describe '.initialize' do
       subject { described_class.new(@board, list_name) }
 
-      it "assigns the board to an instance variable" do
-        subject.instance_variable_get("@board").must_equal(@board)
+      it 'assigns the board to an instance variable' do
+        subject.instance_variable_get('@board').must_equal(@board)
       end
 
-      it "assigns the name to an instance variable" do
-        subject.instance_variable_get("@name").must_equal(list_name)
+      it 'assigns the name to an instance variable' do
+        subject.instance_variable_get('@name').must_equal(list_name)
       end
     end
 
-    describe ".for" do
-      before { VCR.insert_cassette(:create_list, decode_compressed_response: true) }
+    describe '.for' do
+      before do
+        VCR.insert_cassette(:create_list,
+                            decode_compressed_response: true)
+      end
+
       after  { VCR.eject_cassette }
 
       subject { described_class.for(@board, list_name) }
 
-      context "when the list was created" do
-        it "returns the new list" do
+      context 'when the list was created' do
+        it 'returns the new list' do
           subject.must_equal(@list)
         end
       end
 
-      context "when the list was not created" do
+      context 'when the list was not created' do
         before { Trello::List.stubs(:create).raises(Trello::Error) }
 
         it { subject.must_equal false }
       end
+
+      context 'when the access token is invalid' do
+        before do
+          Trello::List.stubs(:create)
+            .raises(Trello::InvalidAccessToken)
+        end
+
+        subject { described_class.for(@board, list_name) }
+
+        it 'catches the exception and re-raises' do
+          proc { subject }.must_raise(Troo::InvalidAccessToken)
+        end
+      end
     end
   end
 end

data/test/lib/troo/actions/move_card_test.rb

@@ -1,43 +1,51 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe MoveCard do
     let(:described_class) { MoveCard }
+    let(:id) { '526d8e130a14a9d846001d98' }
 
     before do
-      @list = Fabricate(:list, external_list_id: "526d8e130a14a9d846001d98")
+      @list = Fabricate(:list, external_list_id: id)
       @card = Fabricate(:card)
       Troo::External::Card.stubs(:fetch).returns(true)
-     end
+    end
 
     after { database_cleanup }
 
-    describe ".initialize" do
+    describe '.initialize' do
       subject { described_class.new(@card, @list) }
 
-      it "assigns the card to an instance variable" do
-        subject.instance_variable_get("@card").must_equal(@card)
+      it 'assigns the card to an instance variable' do
+        subject.instance_variable_get('@card').must_equal(@card)
       end
 
-      it "assigns the list to an instance variable" do
-        subject.instance_variable_get("@list").must_equal(@list)
+      it 'assigns the list to an instance variable' do
+        subject.instance_variable_get('@list').must_equal(@list)
       end
     end
 
-    describe "#perform" do
-      before { VCR.insert_cassette(:move_card, decode_compressed_response: true) }
-      after  { VCR.eject_cassette }
+    describe '#perform' do
+      before do
+        VCR.insert_cassette(:move_card,
+                            decode_compressed_response: true)
+      end
+
+      after { VCR.eject_cassette }
 
       subject { described_class.with(@card, @list) }
 
-      context "when the card was moved" do
-        it "returns a refresh of all cards for the board" do
+      context 'when the card was moved' do
+        it 'returns a refresh of all cards for the board' do
           subject.wont_equal false
         end
       end
 
-      context "when the card was not moved" do
-        before { Trello::Card.any_instance.stubs(:move_to_list).raises(Trello::Error) }
+      context 'when the card was not moved' do
+        before do
+          Trello::Card.any_instance.stubs(:move_to_list)
+            .raises(Trello::Error)
+        end
 
         it { subject.must_equal false }
       end

data/test/lib/troo/adaptors/board_test.rb

@@ -0,0 +1,38 @@
+require_relative '../../../test_helper'
+
+module Troo
+  describe Adaptors::Board do
+    let(:described_class) { Adaptors::Board }
+    let(:resource) do
+      OpenStruct.new(
+        id:          '526d8e130a14a9d846001d96',
+        name:        'My Test Board',
+        description: 'A very brief description...',
+        closed:      false)
+    end
+
+    after { database_cleanup }
+
+    describe '#initialize' do
+      subject { described_class.new(resource) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+    end
+
+    describe '#adapted' do
+      subject { described_class.adapt(resource) }
+
+      it 'returns an adapted resource for local persistence' do
+        subject.must_equal(
+          external_board_id: '526d8e130a14a9d846001d96',
+          name:              'My Test Board',
+          description:       'A very brief description...',
+          closed:            'false'
+        )
+      end
+    end
+  end
+end

data/test/lib/troo/adaptors/card_test.rb

@@ -0,0 +1,49 @@
+require_relative '../../../test_helper'
+
+module Troo
+  describe Adaptors::Card do
+    let(:described_class) { Adaptors::Card }
+    let(:resource) do
+      OpenStruct.new(
+        board_id:           '526d8e130a14a9d846001d96',
+        list_id:            '526d8e130a14a9d846001d97',
+        id:                 '526d8f19ddb279532e005259',
+        member_ids:         ['5195fdb5a8c01a2318004f5d'],
+        short_id:           67,
+        name:               'My Test Card',
+        desc:               'some description',
+        url:                'some trello url',
+        pos:                16,
+        last_activity_date: '2013-12-17 21:48:09 UTC',
+        closed:             false)
+    end
+
+    describe '#initialize' do
+      subject { described_class.new(resource) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+    end
+
+    describe '#adapted' do
+      subject { described_class.adapt(resource) }
+
+      it 'returns an adapted resource for local persistence' do
+        subject.must_equal(
+          external_board_id:   '526d8e130a14a9d846001d96',
+          external_list_id:    '526d8e130a14a9d846001d97',
+          external_card_id:    '526d8f19ddb279532e005259',
+          external_member_ids: "[\"5195fdb5a8c01a2318004f5d\"]",
+          short_id:            '67',
+          name:                'My Test Card',
+          desc:                'some description',
+          url:                 'some trello url',
+          position:            '16',
+          last_activity_date:  '2013-12-17 21:48:09 UTC',
+          closed:              'false')
+      end
+    end
+  end
+end

data/test/lib/troo/adaptors/comment_test.rb

@@ -0,0 +1,41 @@
+require_relative '../../../test_helper'
+
+module Troo
+  describe Adaptors::Comment do
+    let(:described_class) { Adaptors::Comment }
+    let(:resource) do
+      OpenStruct.new(
+        id:                '51f9277b2822b8654f0023af',
+        member_creator_id: '5195fdb5a8c01a2318004f5d',
+        date:              '2013-12-17 22:01:13 UTC',
+        data: {
+          'text'  => 'My Test Comment',
+          'board' => { 'id' => '526d8e130a14a9d846001d96' },
+          'card'  => { 'id' => '526d8f19ddb279532e005259' } })
+    end
+
+    describe '#initialize' do
+      subject { described_class.new(resource) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+    end
+
+    describe '#adapted' do
+      subject { described_class.adapt(resource) }
+
+      it 'returns an adapted resource for local persistence' do
+        subject.must_equal(
+          external_comment_id: '51f9277b2822b8654f0023af',
+          external_board_id:   '526d8e130a14a9d846001d96',
+          external_card_id:    '526d8f19ddb279532e005259',
+          external_member_id:  '5195fdb5a8c01a2318004f5d',
+          date:                '2013-12-17 22:01:13 UTC',
+          text:                'My Test Comment'
+        )
+      end
+    end
+  end
+end

data/test/lib/troo/adaptors/list_test.rb

@@ -0,0 +1,38 @@
+require_relative '../../../test_helper'
+
+module Troo
+  describe Adaptors::List do
+    let(:described_class) { Adaptors::List }
+    let(:resource) do
+      OpenStruct.new(
+        board_id: '526d8e130a14a9d846001d96',
+        id:       '526d8e130a14a9d846001d97',
+        name:     'My Test List',
+        pos:      32_768,
+        closed:   false)
+    end
+
+    describe '#initialize' do
+      subject { described_class.new(resource) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+    end
+
+    describe '#adapted' do
+      subject { described_class.adapt(resource) }
+
+      it 'returns an adapted resource for local persistence' do
+        subject.must_equal(
+          external_board_id: '526d8e130a14a9d846001d96',
+          external_list_id:  '526d8e130a14a9d846001d97',
+          name:              'My Test List',
+          position:          '32768',
+          closed:            'false'
+        )
+      end
+    end
+  end
+end

data/test/lib/troo/adaptors/member_test.rb

@@ -0,0 +1,44 @@
+require_relative '../../../test_helper'
+
+module Troo
+  describe Adaptors::Member do
+    let(:described_class) { Adaptors::Member }
+    let(:resource) do
+      OpenStruct.new(
+        id:        '5195fdb5a8c01a2318004f5d',
+        username:  'gavinlaking1',
+        email:     'gavinlaking@gmail.com',
+        full_name: 'Gavin Laking',
+        initials:  'GL',
+        avatar_id: 'some_avatar_id',
+        bio:       'some bio',
+        url:       'http://www.gavinlaking.name/')
+    end
+
+    describe '#initialize' do
+      subject { described_class.new(resource) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+    end
+
+    describe '#adapted' do
+      subject { described_class.adapt(resource) }
+
+      it 'returns an adapted resource for local persistence' do
+        subject.must_equal(
+          external_member_id: '5195fdb5a8c01a2318004f5d',
+          username:           'gavinlaking1',
+          email:              'gavinlaking@gmail.com',
+          full_name:          'Gavin Laking',
+          initials:           'GL',
+          avatar_id:          'some_avatar_id',
+          bio:                'some bio',
+          url:                'http://www.gavinlaking.name/'
+        )
+      end
+    end
+  end
+end

data/test/lib/troo/commands/add/board_test.rb

@@ -0,0 +1,43 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  module Commands
+    module Add
+      describe Board do
+        let(:described_class) { Board }
+        let(:value) { 'Add Board Test' }
+        let(:id)    {}
+
+        before do
+          CreateBoard.stubs(:with).returns(false)
+        end
+
+        describe '#add' do
+          subject { described_class.new(value, id).add }
+
+          context 'when the parent resource exists' do
+            context 'and the board was created' do
+              before { CreateBoard.stubs(:with).returns(true) }
+
+              it 'returns a polite message' do
+                subject.must_match(/\'Add Board Test\' created/)
+              end
+            end
+
+            context 'and the board was not created' do
+              it 'returns a polite message' do
+                subject.must_match(/could not/)
+              end
+            end
+          end
+
+          context 'when the parent resource does not exist' do
+            it 'returns a polite message' do
+              subject.must_match(/could not/)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/commands/add/card_test.rb

@@ -0,0 +1,44 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  module Commands
+    module Add
+      describe Card do
+        let(:described_class) { Card }
+        let(:value) { 'Add Card Test' }
+        let(:id)    {}
+
+        before do
+          Retrieval::List.stubs(:retrieve)
+          CreateCard.stubs(:for).returns(false)
+        end
+
+        describe '#add' do
+          subject { described_class.new(value, id).add }
+
+          context 'when the parent resource exists' do
+            context 'and the card was created' do
+              before { CreateCard.stubs(:for).returns(true) }
+
+              it 'returns a polite message' do
+                subject.must_match(/\'Add Card Test\' created/)
+              end
+            end
+
+            context 'and the card was not created' do
+              it 'returns a polite message' do
+                subject.must_match(/could not/)
+              end
+            end
+          end
+
+          context 'when the parent resource does not exist' do
+            it 'returns a polite message' do
+              subject.must_match(/could not/)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/commands/add/comment_test.rb

@@ -0,0 +1,44 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  module Commands
+    module Add
+      describe Comment do
+        let(:described_class) { Comment }
+        let(:value) { 'Add Comment Test' }
+        let(:id)    {}
+
+        before do
+          Retrieval::Card.stubs(:retrieve)
+          CreateComment.stubs(:for).returns(false)
+        end
+
+        describe '#add' do
+          subject { described_class.new(value, id).add }
+
+          context 'when the parent resource exists' do
+            context 'and the comment was created' do
+              before { CreateComment.stubs(:for).returns(true) }
+
+              it 'returns a polite message' do
+                subject.must_match(/New comment created/)
+              end
+            end
+
+            context 'and the comment was not created' do
+              it 'returns a polite message' do
+                subject.must_match(/could not/)
+              end
+            end
+          end
+
+          context 'when the parent resource does not exist' do
+            it 'returns a polite message' do
+              subject.must_match(/could not/)
+            end
+          end
+        end
+      end
+    end
+  end
+end