troo 0.0.7 → 0.0.8

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

@@ -1,84 +1,91 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   module External
     describe List do
       let(:described_class) { List }
 
-      describe ".initialize" do
-        subject { described_class.new("some_id", {}) }
-
-        it "assigns the external_id" do
-          subject.instance_variable_get("@external_id").must_equal "some_id"
-        end
-
-        it "assigns the options" do
-          subject.instance_variable_get("@options").must_equal({})
+      describe 'when the mode is board' do
+        before do
+          VCR.insert_cassette(:lists_by_board_id,
+                              decode_compressed_response: true)
         end
-      end
-
-      describe "when the mode is board" do
-        before { VCR.insert_cassette(:lists_by_board_id, decode_compressed_response: true) }
         after  { VCR.eject_cassette }
 
-        let(:board_id) { "526d8e130a14a9d846001d96" }
+        let(:board_id) { '526d8e130a14a9d846001d96' }
         let(:options)  { { mode: :board } }
 
         subject { described_class.fetch(board_id, options) }
 
-        it "returns multiple lists" do
+        it 'returns multiple lists' do
           subject.size.must_equal(4)
         end
 
-        context "when the board cannot be found" do
+        context 'when the board cannot be found' do
           before { Trello::Board.stubs(:find).raises(Trello::Error) }
 
-          it "returns an empty collection" do
+          it 'returns an empty collection' do
             subject.must_equal([])
           end
         end
+
+        context 'when the access token is invalid' do
+          before do
+            Trello::Board.stubs(:find)
+              .raises(Trello::InvalidAccessToken)
+          end
+
+          subject { described_class.new(board_id, options).fetch }
+
+          it 'catches the exception and re-raises' do
+            proc { subject }.must_raise(Troo::InvalidAccessToken)
+          end
+        end
       end
 
-      describe "when the mode is list" do
-        before { VCR.insert_cassette(:list_by_list_id, decode_compressed_response: true) }
+      describe 'when the mode is list' do
+        before do
+          VCR.insert_cassette(:list_by_list_id,
+                              decode_compressed_response: true)
+        end
         after  { VCR.eject_cassette }
 
-        let(:list_id) { "526d8e130a14a9d846001d97" }
+        let(:list_id) { '526d8e130a14a9d846001d97' }
         let(:options) { { mode: :list } }
 
         subject { described_class.fetch(list_id, options) }
 
-        it "returns a list with the list_id" do
+        it 'returns a list with the list_id' do
           subject.size.must_equal(1)
         end
 
-        context "when the list cannot be found" do
+        context 'when the list cannot be found' do
           before { Trello::List.stubs(:find).raises(Trello::Error) }
 
-          it "returns an empty collection" do
+          it 'returns an empty collection' do
             subject.must_equal([])
           end
         end
       end
 
-      describe "when the mode is card" do
-        let(:card_id) { "526d8f19ddb279532e005259" }
+      describe 'when the mode is card' do
+        let(:card_id) { '526d8f19ddb279532e005259' }
         let(:options) { { mode: :card } }
 
         subject { described_class.fetch(card_id, options) }
 
-        it "returns an empty collection" do
+        it 'returns an empty collection' do
           subject.must_equal([])
         end
       end
 
-      describe "when the mode is member" do
-        let(:member_id) { "5195fdb5a8c01a2318004f5d" }
+      describe 'when the mode is member' do
+        let(:member_id) { '5195fdb5a8c01a2318004f5d' }
         let(:options) { { mode: :member } }
 
         subject { described_class.fetch(member_id, options) }
 
-        it "returns an empty collection" do
+        it 'returns an empty collection' do
           subject.must_equal([])
         end
       end

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

@@ -1,83 +1,90 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   module External
     describe Member do
       let(:described_class) { Member }
 
-      describe ".initialize" do
-        subject { described_class.new("some_id", {}) }
-
-        it "assigns the external_id" do
-          subject.instance_variable_get("@external_id").must_equal("some_id")
-        end
-
-        it "assigns the options" do
-          subject.instance_variable_get("@options").must_equal({})
+      describe 'when the mode is board' do
+        before do
+          VCR.insert_cassette(:members_by_board_id,
+                              decode_compressed_response: true)
         end
-      end
-
-      describe "when the mode is board" do
-        before { VCR.insert_cassette(:members_by_board_id, decode_compressed_response: true) }
         after  { VCR.eject_cassette }
 
-        let(:board_id) { "526d8e130a14a9d846001d96" }
+        let(:board_id) { '526d8e130a14a9d846001d96' }
         let(:options)  { { mode: :board } }
 
         subject { described_class.fetch(board_id, options) }
 
-        it "returns multiple members" do
+        it 'returns multiple members' do
           subject.size.must_equal(1)
         end
 
-        context "when the board cannot be found" do
+        context 'when the board cannot be found' do
           before { Trello::Board.stubs(:find).raises(Trello::Error) }
 
-          it "returns an empty collection" do
+          it 'returns an empty collection' do
             subject.must_equal([])
           end
         end
+
+        context 'when the access token is invalid' do
+          before do
+            Trello::Board.stubs(:find)
+              .raises(Trello::InvalidAccessToken)
+          end
+
+          subject { described_class.new(board_id, options).fetch }
+
+          it 'catches the exception and re-raises' do
+            proc { subject }.must_raise(Troo::InvalidAccessToken)
+          end
+        end
       end
 
-      describe "when the mode is list" do
-        let(:list_id) { "526d8e130a14a9d846001d97" }
+      describe 'when the mode is list' do
+        let(:list_id) { '526d8e130a14a9d846001d97' }
         let(:options) { { mode: :list } }
 
         subject { described_class.fetch(list_id, options) }
 
-        it "returns an empty collection" do
+        it 'returns an empty collection' do
           subject.must_equal([])
         end
       end
 
-      describe "when the mode is card" do
-        let(:card_id) { "526d8f19ddb279532e005259" }
+      describe 'when the mode is card' do
+        let(:card_id) { '526d8f19ddb279532e005259' }
         let(:options) { { mode: :card } }
 
         subject { described_class.fetch(card_id, options) }
 
-        it "returns an empty collection" do
+        it 'returns an empty collection' do
           subject.must_equal([])
         end
       end
 
-      describe "when the mode is member" do
-        before { VCR.insert_cassette(:member_by_member_id, decode_compressed_response: true) }
+      describe 'when the mode is member' do
+        before do
+          VCR.insert_cassette(:member_by_member_id,
+                              decode_compressed_response: true)
+        end
         after  { VCR.eject_cassette }
 
-        let(:member_id) { "5195fdb5a8c01a2318004f5d" }
+        let(:member_id) { '5195fdb5a8c01a2318004f5d' }
         let(:options)   { { mode: :member } }
 
         subject { described_class.fetch(member_id, options) }
 
-        it "returns a member with the member_id" do
+        it 'returns a member with the member_id' do
           subject.size.must_equal(1)
         end
 
-        context "when the member cannot be found" do
+        context 'when the member cannot be found' do
           before { Trello::Member.stubs(:find).raises(Trello::Error) }
 
-          it "returns an empty collection" do
+          it 'returns an empty collection' do
             subject.must_equal([])
           end
         end

data/test/lib/troo/external/resource_test.rb

@@ -1,8 +1,25 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   module External
     describe Resource do
+      let(:described_class) { Resource }
+      let(:external_id) { '526d8e130a14a9d846001d96' }
+      let(:options) { {} }
+
+      describe '.initialize' do
+        subject { described_class.new(external_id, options) }
+
+        it 'assigns the external_id to an instance variable' do
+          subject.instance_variable_get('@external_id')
+            .must_equal(external_id)
+        end
+
+        it 'assigns the options to an instance variable' do
+          subject.instance_variable_get('@options')
+            .must_equal(options)
+        end
+      end
     end
   end
 end

data/test/lib/troo/helpers/command_helpers_test.rb

@@ -0,0 +1,82 @@
+require_relative '../../../test_helper'
+
+class CommandHelpersDummy
+  include Troo::CommandHelpers
+
+  def initialize(type, id)
+    @type, @id = type, id
+  end
+
+  private
+
+  attr_accessor :type
+end
+
+module Troo
+  describe CommandHelpers do
+    let(:described_class)    { CommandHelpersDummy }
+    let(:described_instance) { described_class.new(type, id) }
+    let(:type)               { :board }
+    let(:id)                 { nil }
+
+    describe '#error_no_default' do
+      subject { described_instance.error_no_default }
+
+      context 'when the type is comments' do
+        let(:type) { :comments }
+
+        it 'returns the no default set error message' do
+          subject.must_match(/to set a default card first/)
+        end
+      end
+
+      context 'when the type is not comments' do
+        it 'returns the no default set error message' do
+          subject.must_match(/to set a default board first/)
+        end
+      end
+    end
+
+    describe '#error_not_found' do
+      subject { described_instance.error_not_found }
+
+      context 'when the type is comments' do
+        let(:type) { :comments }
+
+        it 'returns the not found error message' do
+          subject.must_match(/Card cannot be found/)
+        end
+      end
+
+      context 'when the type is not comments' do
+        it 'returns the no default set error message' do
+          subject.must_match(/Board cannot be found/)
+        end
+      end
+    end
+
+    describe '#type_str' do
+      subject { described_instance.type_str }
+
+      it 'returns the type as a string' do
+        subject.must_equal('board')
+      end
+    end
+
+    describe '#type_capitalize' do
+      subject { described_instance.type_capitalize }
+
+      it 'returns the type as a capitalized string' do
+        subject.must_equal('Board')
+      end
+    end
+
+    describe '#type_pluralize' do
+      subject { described_instance.type_pluralize }
+
+      it 'returns the type as a pluralized string' do
+        subject.must_equal('boards')
+      end
+    end
+  end
+end

data/test/lib/troo/{display → helpers}/decorator_helpers_test.rb

@@ -1,4 +1,4 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 class DecoratorHelpersDummy
   include Troo::DecoratorHelpers
@@ -9,18 +9,18 @@ module Troo
     let(:described_class) { DecoratorHelpersDummy }
     let(:described_instance) { described_class.new }
 
-    describe "#title" do
+    describe '#title' do
       before do
-        described_instance.stubs(:default).returns("*")
+        described_instance.stubs(:default).returns('*')
         described_instance.stubs(:id).returns(67)
-        described_instance.stubs(:name).returns("My Test Dummy")
+        described_instance.stubs(:name).returns('My Test Dummy')
         described_instance.stubs(:options).returns({})
       end
 
       subject { described_instance.title }
 
-      it "returns a formatted string representing the card title" do
-        subject.must_match /\* \(67\) My Test Dummy/
+      it 'returns a formatted string representing the card title' do
+        subject.must_match(/\* \(67\) My Test Dummy/)
       end
     end
   end

data/test/lib/troo/helpers/model_helpers_test.rb

@@ -0,0 +1,103 @@
+require_relative '../../../test_helper'
+
+class ModelHelpersDummy < Ohm::Model
+  include Troo::ModelHelpers
+
+  attribute :name
+  attribute :default
+
+  index     :name
+  index     :default
+end
+
+module Troo
+  describe ModelHelpersDummy do
+    let(:described_class) { ModelHelpersDummy }
+
+    before do
+      @dumb   = ModelHelpersDummy.create(
+                 name: 'My Dumb Model',
+                 default: false)
+      @dumber = ModelHelpersDummy.create(
+                 name: 'My Dumber Model',
+                 default: true)
+    end
+
+    after { database_cleanup }
+
+    describe '.first' do
+      subject { described_class.first(criteria) }
+
+      context 'when no criteria are provided' do
+        let(:criteria) {}
+
+        it 'returns the first model stored of that type' do
+          subject.name.must_equal('My Dumb Model')
+        end
+      end
+
+      context 'when criteria are provided' do
+        let(:criteria) { { name: 'My Dumber Model' } }
+
+        it 'returns the first model matching the criteria' do
+          subject.name.must_equal('My Dumber Model')
+        end
+      end
+    end
+
+    describe '.update' do
+      subject { described_class.update(criteria) }
+
+      context 'when no criteria are provided' do
+        let(:criteria) {}
+
+        it 'returns false; no update was performed' do
+          subject.must_equal(false)
+        end
+      end
+
+      context 'when criteria are provided' do
+        let(:criteria) { { name: 'My Updated Model' } }
+
+        it 'updates all records with the criteria' do
+          subject.must_equal(true)
+          ModelHelpersDummy.all.each do |dummy|
+            dummy.name.must_equal('My Updated Model')
+          end
+        end
+      end
+    end
+
+    describe '.default' do
+      subject { described_class.default }
+
+      context 'when there is no default model' do
+        it 'returns the model' do
+          subject.must_equal(@dumber)
+        end
+      end
+
+      context 'when there is a default model' do
+        before { @dumber.delete }
+
+        it { subject.must_equal(nil) }
+      end
+    end
+
+    describe '.count' do
+      subject { described_class.count }
+
+      it 'returns the number of this model persisted' do
+        subject.must_equal(2)
+      end
+    end
+
+    describe '#external_attributes' do
+      subject { @dumb.external_attributes }
+
+      it 'returns the attributes of the model which Trello affects' do
+        subject.must_equal(name: 'My Dumb Model')
+      end
+    end
+  end
+end

data/test/lib/troo/models/behaviours/set_default_test.rb

@@ -0,0 +1,54 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  describe Behaviours::NullEntity do
+    let(:described_class) { Behaviours::NullEntity }
+
+    describe '#default?' do
+      subject { described_class.new.default? }
+
+      it { subject.must_equal false }
+    end
+  end
+
+  describe Behaviours::SetDefault do
+    let(:described_class) { Behaviours::SetDefault }
+
+    describe '.initialize' do
+      let(:entity) { :some_model }
+
+      subject { described_class.new(entity) }
+
+      it 'assigns the entity to an instance variable' do
+        subject.instance_variable_get('@entity').must_equal(entity)
+      end
+    end
+
+    describe '#set_default!' do
+      before do
+        @board_1 = Fabricate(:board, default: true)
+        @board_2 = Fabricate(:board, default: false)
+      end
+
+      after { database_cleanup }
+
+      subject { described_class.for(entity) }
+
+      context 'when the entity is already the default' do
+        let(:entity) { @board_1 }
+
+        it { subject.must_equal true }
+      end
+
+      context 'when the entity is not already the default' do
+        let(:entity) { @board_2 }
+
+        it 'sets the specified entity to be the default' do
+          subject.default.must_equal(true)
+
+          subject.must_equal(@board_2)
+        end
+      end
+    end
+  end
+end

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

@@ -1,4 +1,4 @@
-require_relative "../../../test_helper"
+require_relative '../../../test_helper'
 
 module Troo
   describe Board do
@@ -7,29 +7,29 @@ module Troo
 
     subject { described_instance }
 
-    context "attributes" do
-      it "should have a name attribute" do
-        subject.name.must_equal "My Test Board"
+    context 'attributes' do
+      it 'should have a name attribute' do
+        subject.name.must_equal 'My Test Board'
       end
 
-      it "should have a description attribute" do
-        subject.description.must_equal "A very brief description..."
+      it 'should have a description attribute' do
+        subject.description.must_equal 'A very brief description...'
       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
     end
 
-    context "associations" do
+    context 'associations' do
       before do
         @list  = Fabricate(:list)
         @card  = Fabricate(:card)
@@ -37,14 +37,13 @@ module Troo
 
       after { database_cleanup }
 
-      it "can have many lists" do
+      it 'can have many lists' do
         subject.lists.size.must_equal 1
       end
 
-      it "can have many cards" do
+      it 'can have many cards' do
         subject.cards.size.must_equal 1
       end
     end
   end
 end
-