troo 0.0.7 → 0.0.8

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

@@ -1,4 +1,4 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe Card do
@@ -7,57 +7,89 @@ module Troo
 
     subject { described_instance }
 
-    context "attributes" do
-      it "should have a short_id attribute" do
-        subject.short_id.must_equal "67"
+    context 'attributes' do
+      it 'should have a short_id attribute' do
+        subject.short_id.must_equal '67'
       end
 
-      it "should have a name attribute" do
-        subject.name.must_equal "My Test Card"
+      it 'should have a name attribute' do
+        subject.name.must_equal 'My Test Card'
       end
 
-      it "should have a desc attribute" do
-        subject.desc.must_equal "some description"
+      it 'should have a desc attribute' do
+        subject.desc.must_equal 'some description'
       end
 
-      it "should have a url attribute" do
-        subject.url.must_equal "some trello url"
+      it 'should have a url attribute' do
+        subject.url.must_equal 'some trello url'
       end
 
-      it "should have an external_member_ids attribute" do
-        subject.external_member_ids.must_equal(["5195fdb5a8c01a2318004f5d"])
+      describe '#external_member_ids' do
+        context 'when the attribute is set' do
+          context 'and the attribute is an array' do
+            it 'returns the attribute' do
+              subject.external_member_ids
+                .must_equal(['5195fdb5a8c01a2318004f5d'])
+            end
+          end
+
+          context 'and the attribute is a string' do
+            let(:member_ids) { "[\"5195fdb5a8c01a2318004f5d\"]" }
+            let(:described_instance) do
+              Fabricate.build(:card,
+                              external_member_ids: member_ids)
+            end
+
+            it 'converts to an array and returns the attribute' do
+              subject.external_member_ids
+                .must_equal(['5195fdb5a8c01a2318004f5d'])
+            end
+          end
+        end
+
+        context 'when the attribute is not set' do
+          let(:described_instance) do
+            Fabricate.build(:card, external_member_ids: nil)
+          end
+
+          it 'returns an empty collection' do
+            subject.external_member_ids.must_equal([])
+          end
+        end
       end
 
-      it "should have a position attribute" do
+      it 'should have a position attribute' do
         subject.position.must_equal 16
       end
 
-      it "should have a last_activity_date attribute" do
-        subject.last_activity_date.must_equal "2013-12-17 21:48:09 UTC"
+      it 'should have a last_activity_date attribute' do
+        subject.last_activity_date
+          .must_equal '2013-12-17 21:48:09 UTC'
       end
 
-      it "should have a default attribute" do
+      it 'should have a default attribute' do
         subject.default.must_equal false
       end
 
-      it "should have a closed attribute" do
+      it 'should have a closed attribute' do
         subject.closed.must_equal false
       end
 
-      it "should have an external_board_id attribute" do
-        subject.external_board_id.must_equal "526d8e130a14a9d846001d96"
+      it 'should have an external_board_id attribute' do
+        subject.external_board_id
+          .must_equal '526d8e130a14a9d846001d96'
       end
 
-      it "should have an external_list_id attribute" do
-        subject.external_list_id.must_equal "526d8e130a14a9d846001d97"
+      it 'should have an external_list_id attribute' do
+        subject.external_list_id.must_equal '526d8e130a14a9d846001d97'
       end
 
-      it "should have an external_card_id attribute" do
-        subject.external_card_id.must_equal "526d8f19ddb279532e005259"
+      it 'should have an external_card_id attribute' do
+        subject.external_card_id.must_equal '526d8f19ddb279532e005259'
       end
     end
 
-    context "associations" do
+    context 'associations' do
       before do
         @board   = Fabricate(:board)
         @list    = Fabricate(:list)
@@ -67,36 +99,35 @@ module Troo
 
       after { database_cleanup }
 
-      it "belongs to a board" do
+      it 'belongs to a board' do
         subject.board.must_equal @board
       end
 
-      it "belongs to a list" do
+      it 'belongs to a list' do
         subject.list.must_equal @list
       end
 
-      it "can have many comments" do
+      it 'can have many comments' do
         subject.comments.size.must_equal 1
       end
 
-      it "can have many comments" do
+      it 'can have many comments' do
         subject.recent_comments.size.must_equal 1
       end
 
-      context "when the card has members" do
-        it "returns the collection of members" do
+      context 'when the card has members' do
+        it 'returns the collection of members' do
           subject.members.size.must_equal 1
         end
       end
 
-      context "when the card has no members" do
+      context 'when the card has no members' do
         before { subject.stubs(:external_member_ids).returns([]) }
 
-        it "returns an empty collection" do
+        it 'returns an empty collection' do
           subject.members.size.must_equal 0
         end
       end
     end
   end
 end
-

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

@@ -1,4 +1,4 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe Comment do
@@ -7,33 +7,33 @@ module Troo
 
     subject { described_instance }
 
-    context "attributes" do
-      it "should have a text attribute" do
-        subject.text.must_equal "My Test Comment"
+    context 'attributes' do
+      it 'should have a text attribute' do
+        subject.text.must_equal 'My Test Comment'
       end
 
-      it "should have a date attribute" do
-        subject.date.must_equal "2013-12-17 22:01:13 UTC"
+      it 'should have a date attribute' do
+        subject.date.must_equal '2013-12-17 22:01:13 UTC'
       end
 
-      it "should have an external_board_id attribute" do
-        subject.external_board_id.must_equal "526d8e130a14a9d846001d96"
+      it 'should have an external_board_id attribute' do
+        subject.external_board_id.must_equal '526d8e130a14a9d846001d96'
       end
 
-      it "should have an external_card_id attribute" do
-        subject.external_card_id.must_equal "526d8f19ddb279532e005259"
+      it 'should have an external_card_id attribute' do
+        subject.external_card_id.must_equal '526d8f19ddb279532e005259'
       end
 
-      it "should have an external_comment_id attribute" do
-        subject.external_comment_id.must_equal "51f9277b2822b8654f0023af"
+      it 'should have an external_comment_id attribute' do
+        subject.external_comment_id.must_equal '51f9277b2822b8654f0023af'
       end
 
-      it "should have a external_member_id attribute" do
-        subject.external_member_id.must_equal "5195fdb5a8c01a2318004f5d"
+      it 'should have a external_member_id attribute' do
+        subject.external_member_id.must_equal '5195fdb5a8c01a2318004f5d'
       end
     end
 
-    context "associations" do
+    context 'associations' do
       before do
         @board  = Fabricate(:board)
         @card   = Fabricate(:card)
@@ -42,18 +42,17 @@ module Troo
 
       after { database_cleanup }
 
-      it "belongs to a board" do
+      it 'belongs to a board' do
         subject.board.must_equal @board
       end
 
-      it "belongs to a card" do
+      it 'belongs to a card' do
         subject.card.must_equal @card
       end
 
-      it "belongs to a member" do
+      it 'belongs to a member' do
         subject.member.must_equal @member
       end
     end
   end
 end
-

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

@@ -1,4 +1,4 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe List do
@@ -7,33 +7,33 @@ module Troo
 
     subject { described_instance }
 
-    context "attributes" do
-      it "should have a name attribute" do
-        subject.name.must_equal "My Test List"
+    context 'attributes' do
+      it 'should have a name attribute' do
+        subject.name.must_equal 'My Test List'
       end
 
-      it "should have a position attribute" do
-        subject.position.must_equal 32768
+      it 'should have a position attribute' do
+        subject.position.must_equal 32_768
       end
 
-      it "should have a default attribute" do
+      it 'should have a default attribute' do
         subject.default.must_equal false
       end
 
-      it "should have a closed attribute" do
+      it 'should have a closed attribute' do
         subject.closed.must_equal false
       end
 
-      it "should have a external_board_id attribute" do
-        subject.external_board_id.must_equal "526d8e130a14a9d846001d96"
+      it 'should have a external_board_id attribute' do
+        subject.external_board_id.must_equal '526d8e130a14a9d846001d96'
       end
 
-      it "should have an external_list_id attribute" do
-        subject.external_list_id.must_equal "526d8e130a14a9d846001d97"
+      it 'should have an external_list_id attribute' do
+        subject.external_list_id.must_equal '526d8e130a14a9d846001d97'
       end
     end
 
-    context "associations" do
+    context 'associations' do
       before do
         @board = Fabricate(:board)
         @card  = Fabricate(:card)
@@ -41,11 +41,11 @@ module Troo
 
       after { database_cleanup }
 
-      it "belongs to a board" do
+      it 'belongs to a board' do
         subject.board.must_equal @board
       end
 
-      it "can have many cards" do
+      it 'can have many cards' do
         subject.cards.size.must_equal 1
       end
     end

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

@@ -1,4 +1,4 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe Member do
@@ -7,37 +7,37 @@ module Troo
 
     subject { described_instance }
 
-    context "attributes" do
-      it "should have a username attribute" do
-        subject.username.must_equal "gavinlaking1"
+    context 'attributes' do
+      it 'should have a username attribute' do
+        subject.username.must_equal 'gavinlaking1'
       end
 
-      it "should have an email attribute" do
-        subject.email.must_equal "gavinlaking@gmail.com"
+      it 'should have an email attribute' do
+        subject.email.must_equal 'gavinlaking@gmail.com'
       end
 
-      it "should have a full_name attribute" do
-        subject.full_name.must_equal "Gavin Laking"
+      it 'should have a full_name attribute' do
+        subject.full_name.must_equal 'Gavin Laking'
       end
 
-      it "should have an initials attribute" do
-        subject.initials.must_equal "GL"
+      it 'should have an initials attribute' do
+        subject.initials.must_equal 'GL'
       end
 
-      it "should have an avatar_id attribute" do
-        subject.avatar_id.must_equal "some_avatar_id"
+      it 'should have an avatar_id attribute' do
+        subject.avatar_id.must_equal 'some_avatar_id'
       end
 
-      it "should have a bio attribute" do
-        subject.bio.must_equal "some bio"
+      it 'should have a bio attribute' do
+        subject.bio.must_equal 'some bio'
       end
 
-      it "should have a url attribute" do
-        subject.url.must_equal "http://www.gavinlaking.name/"
+      it 'should have a url attribute' do
+        subject.url.must_equal 'http://www.gavinlaking.name/'
       end
 
-      it "should have an external_member_id attribute" do
-        subject.external_member_id.must_equal "5195fdb5a8c01a2318004f5d"
+      it 'should have an external_member_id attribute' do
+        subject.external_member_id.must_equal '5195fdb5a8c01a2318004f5d'
       end
     end
   end

data/test/lib/troo/models/persistence/board_test.rb

@@ -0,0 +1,62 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  describe Persistence::Board do
+    let(:described_class) { Persistence::Board }
+    let(:resource) do
+      OpenStruct.new(
+        id:     '526d8e130a14a9d846001d96',
+        name:   resource_name,
+        description:   'A very brief description...',
+        closed: false)
+    end
+    let(:resource_name) { 'My Test Board' }
+    let(:options) { {} }
+
+    before { @board = Fabricate(:board) }
+    after  { database_cleanup }
+
+    describe '.initialize' do
+      subject { described_class.new(resource, options) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+
+      it 'assigns the options to an instance variable' do
+        subject.instance_variable_get('@options').must_equal(options)
+      end
+    end
+
+    describe '#persist' do
+      subject { described_class.for(resource, options) }
+
+      context 'when there is already a local copy' do
+        context 'and the local copy is identical' do
+          it 'returns the local copy' do
+            subject.must_equal(@board)
+          end
+        end
+
+        context 'and the local copy is out of date' do
+          let(:resource_name) { 'My Renamed Board' }
+
+          it 'updates and returns the new local copy' do
+            subject.name.must_equal(resource_name)
+          end
+        end
+      end
+
+      context 'when there is not already a local copy' do
+        let(:resource_name) { 'My New Test Board' }
+
+        before { database_cleanup }
+
+        it 'creates and returns the new local copy' do
+          subject.name.must_equal(resource_name)
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/models/persistence/card_test.rb

@@ -0,0 +1,61 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  describe Persistence::Card do
+    let(:described_class) { Persistence::Card }
+    let(:resource) do
+      OpenStruct.new(
+        id:     '526d8f19ddb279532e005259',
+        name:   resource_name,
+        closed: false)
+    end
+    let(:resource_name) { 'My Test Card' }
+    let(:options) { {} }
+
+    before { @card = Fabricate(:card) }
+    after  { database_cleanup }
+
+    describe '.initialize' do
+      subject { described_class.new(resource, options) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+
+      it 'assigns the options to an instance variable' do
+        subject.instance_variable_get('@options').must_equal(options)
+      end
+    end
+
+    describe '#persist' do
+      subject { described_class.for(resource, options) }
+
+      context 'when there is already a local copy' do
+        context 'and the local copy is identical' do
+          it 'returns the local copy' do
+            subject.must_equal(@card)
+          end
+        end
+
+        context 'and the local copy is out of date' do
+          let(:resource_name) { 'My Renamed Card' }
+
+          it 'updates and returns the new local copy' do
+            subject.name.must_equal(resource_name)
+          end
+        end
+      end
+
+      context 'when there is not already a local copy' do
+        let(:resource_name) { 'My New Test Card' }
+
+        before { database_cleanup }
+
+        it 'creates and returns the new local copy' do
+          subject.name.must_equal(resource_name)
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/models/persistence/comment_test.rb

@@ -0,0 +1,69 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  describe Persistence::Comment do
+    let(:described_class) { Persistence::Comment }
+    let(:resource) do
+      OpenStruct.new(
+        id:   '51f9277b2822b8654f0023af',
+        date: '2013-12-17 22:01:13 UTC',
+        data: {
+          'text' => resource_text,
+          'board' => {
+            'id' => '526d8e130a14a9d846001d96'
+          },
+          'card' => {
+            'id' => '526d8f19ddb279532e005259'
+        }
+      })
+    end
+    let(:resource_text) { 'My Test Comment' }
+    let(:options) { {} }
+
+    before { @comment = Fabricate(:comment) }
+    after  { database_cleanup }
+
+    describe '.initialize' do
+      subject { described_class.new(resource, options) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+
+      it 'assigns the options to an instance variable' do
+        subject.instance_variable_get('@options').must_equal(options)
+      end
+    end
+
+    describe '#persist' do
+      subject { described_class.for(resource, options) }
+
+      context 'when there is already a local copy' do
+        context 'and the local copy is identical' do
+          it 'returns the local copy' do
+            subject.must_equal(@comment)
+          end
+        end
+
+        context 'and the local copy is out of date' do
+          let(:resource_text) { 'My Renamed Comment' }
+
+          it 'updates and returns the new local copy' do
+            subject.text.must_equal(resource_text)
+          end
+        end
+      end
+
+      context 'when there is not already a local copy' do
+        let(:resource_text) { 'My New Test Comment' }
+
+        before { database_cleanup }
+
+        it 'creates and returns the new local copy' do
+          subject.text.must_equal(resource_text)
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/models/persistence/list_test.rb

@@ -0,0 +1,60 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  describe Persistence::List do
+    let(:described_class) { Persistence::List }
+    let(:resource) do
+      OpenStruct.new(
+        id:     '526d8e130a14a9d846001d97',
+        name:   resource_name,
+        closed: false)
+    end
+    let(:resource_name) { 'My Test List' }
+    let(:options) { {} }
+
+    before { @list = Fabricate(:list) }
+    after  { database_cleanup }
+
+    describe '.initialize' do
+      subject { described_class.new(resource, options) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource').must_equal(resource)
+      end
+
+      it 'assigns the options to an instance variable' do
+        subject.instance_variable_get('@options').must_equal(options)
+      end
+    end
+
+    describe '#persist' do
+      subject { described_class.for(resource, options) }
+
+      context 'when there is already a local copy' do
+        context 'and the local copy is identical' do
+          it 'returns the local copy' do
+            subject.must_equal(@list)
+          end
+        end
+
+        context 'and the local copy is out of date' do
+          let(:resource_name) { 'My Renamed List' }
+
+          it 'updates and returns the new local copy' do
+            subject.name.must_equal(resource_name)
+          end
+        end
+      end
+
+      context 'when there is not already a local copy' do
+        let(:resource_name) { 'My New Test List' }
+
+        before { database_cleanup }
+
+        it 'creates and returns the new local copy' do
+          subject.name.must_equal(resource_name)
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/models/persistence/member_test.rb

@@ -0,0 +1,60 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  describe Persistence::Member do
+    let(:described_class) { Persistence::Member }
+    let(:resource) do
+      OpenStruct.new(
+        id:        '5195fdb5a8c01a2318004f5d',
+        full_name: resource_full_name)
+    end
+    let(:resource_full_name) { 'My Test Member' }
+    let(:options) { {} }
+
+    before { @member = Fabricate(:member) }
+    after  { database_cleanup }
+
+    describe '.initialize' do
+      subject { described_class.new(resource, options) }
+
+      it 'assigns the resource to an instance variable' do
+        subject.instance_variable_get('@resource')
+          .must_equal(resource)
+      end
+
+      it 'assigns the options to an instance variable' do
+        subject.instance_variable_get('@options').must_equal(options)
+      end
+    end
+
+    describe '#persist' do
+      subject { described_class.for(resource, options) }
+
+      context 'when there is already a local copy' do
+        context 'and the local copy is identical' do
+          it 'returns the local copy' do
+            subject.must_equal(@member)
+          end
+        end
+
+        context 'and the local copy is out of date' do
+          let(:resource_full_name) { 'My Renamed Member' }
+
+          it 'updates and returns the new local copy' do
+            subject.full_name.must_equal(resource_full_name)
+          end
+        end
+      end
+
+      context 'when there is not already a local copy' do
+        let(:resource_full_name) { 'My New Test Member' }
+
+        before { database_cleanup }
+
+        it 'creates and returns the new local copy' do
+          subject.full_name.must_equal(resource_full_name)
+        end
+      end
+    end
+  end
+end