troo 0.0.8 → 0.0.9

data/test/lib/troo/remote/persistence/move_card_test.rb

@@ -0,0 +1,78 @@
+require_relative '../../../../test_helper'
+
+module Troo
+  module Remote
+    module Persistence
+      describe MoveCard do
+        let(:described_class)   { MoveCard }
+        let(:external_board_id) { '526d8e130a14a9d846001d96' }
+        let(:external_card_id)  { '526d8f19ddb279532e005259' }
+        let(:external_list_id)  { '526d8e130a14a9d846001d97' }
+        let(:card)              { [Fabricate.build(:card)] }
+
+        before do
+          Troo::Persistence::Local.stubs(:with_collection)
+            .returns(card)
+        end
+
+        after { database_cleanup }
+
+        describe '.with' do
+          let(:board) {}
+
+          subject do
+            described_class.with(external_card_id,
+                                 external_list_id,
+                                 external_board_id)
+          end
+
+          context 'when a board was specified' do
+            let(:board) { @board }
+
+            before do
+              VCR.insert_cassette(:move_card_board,
+                                  decode_compressed_response: true)
+            end
+
+            after { VCR.eject_cassette }
+
+            context 'and the card was moved' do
+              it 'returns the updated card' do
+                subject.must_be_instance_of Troo::Card
+              end
+            end
+
+            context 'and the card was not moved' do
+              before { API::Client.stubs(:perform).returns([]) }
+
+              it { subject.must_equal false }
+            end
+          end
+
+          context 'when a board was not specified' do
+            let(:external_board_id) {}
+
+            before do
+              VCR.insert_cassette(:move_card_list,
+                                  decode_compressed_response: true)
+            end
+
+            after { VCR.eject_cassette }
+
+            context 'and the card was moved' do
+              it 'returns the updated card' do
+                subject.must_be_instance_of Troo::Card
+              end
+            end
+
+            context 'and the card was not moved' do
+              before { API::Client.stubs(:perform).returns([]) }
+
+              it { subject.must_equal false }
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/retrieval/local_test.rb

@@ -0,0 +1,192 @@
+require_relative '../../../test_helper'
+
+module Troo
+  module Retrieval
+    describe Local do
+      let(:described_class)  { Local }
+      let(:klass)            { stub(type: type) }
+      let(:id)               {}
+      let(:options)          { {} }
+      let(:type)             { :resource_type }
+      let(:default_board)    {}
+      let(:local_resource)   {}
+      let(:remote_resource)  { [] }
+      let(:default_resource) {}
+
+      before do
+        Troo::Board.stubs(:default).returns(default_board)
+        klass.stubs(:remote).returns(stub)
+        klass.stubs(:default).returns(default_resource)
+        klass.stubs(:first).returns(local_resource)
+        klass.stubs(:[]).returns(local_resource)
+        klass.stubs(:by_external_id).returns(local_resource)
+        Retrieval::Remote.stubs(:fetch).returns(remote_resource)
+      end
+
+      describe '.all' do
+        before { klass.stubs(:all).returns(collection) }
+
+        subject { described_class.all(klass) }
+
+        context 'when local resources exist' do
+          let(:collection) { [:resource, :resource] }
+
+          it 'returns all the local resources' do
+            subject.must_equal(collection)
+          end
+        end
+
+        context 'when no local resources exist' do
+          let(:collection) { [] }
+
+          it 'returns an empty collection' do
+            subject.must_equal []
+          end
+        end
+      end
+
+      describe '.default' do
+        subject { described_class.default(klass, options) }
+
+        context 'when the default is set' do
+          let(:default_resource) { stub }
+
+          it 'returns the resource marked as default' do
+            subject.wont_equal nil
+          end
+        end
+
+        context 'when the default is not set' do
+          it { subject.must_equal nil }
+        end
+      end
+
+      describe '.retrieve' do
+        subject { described_class.retrieve(klass, id, options) }
+
+        context 'without an ID' do
+          context 'when the default is set' do
+            let(:default_resource) { stub }
+
+            it 'returns the resource marked as default' do
+              subject.wont_equal nil
+            end
+          end
+
+          context 'when the default is not set' do
+            it { subject.must_equal nil }
+          end
+        end
+
+        context 'with an ID' do
+          let(:id) { 67 }
+
+          context 'attempts local retrieval by short_id' do
+            context 'when the resource type is not a card' do
+              context 'when the resource exists' do
+                let(:local_resource) { stub }
+
+                it 'returns the resource' do
+                  subject.must_equal(local_resource)
+                end
+              end
+
+              context 'when the resource does not exist' do
+                it { subject.must_equal nil }
+              end
+            end
+
+            context 'when the resource type is a card' do
+              let(:type) { :card }
+
+              context 'and a default board exists' do
+                let(:default_board) { stub(external_id: 27) }
+
+                context 'and the resource exists' do
+                  let(:local_resource) { stub }
+
+                  it 'returns the resource' do
+                    subject.must_equal(local_resource)
+                  end
+                end
+
+                context 'but the resource does not exist' do
+                  it { subject.must_equal nil }
+                end
+              end
+
+              context 'and no default board exists' do
+                context 'and the resource exists' do
+                  let(:local_resource) { stub }
+
+                  it 'returns the resource' do
+                    subject.must_equal(local_resource)
+                  end
+                end
+
+                context 'and the resource does not exist' do
+                  it { subject.must_equal nil }
+                end
+              end
+            end
+          end
+
+          context 'attempts local retrieval by database ID' do
+            context 'when the resource exists' do
+              let(:local_resource) { stub }
+
+              it 'returns the resource' do
+                subject.must_equal(local_resource)
+              end
+            end
+
+            context 'when the resource does not exist' do
+              it { subject.must_equal nil }
+            end
+          end
+
+          context 'attempts local retrieval by external ID' do
+            context 'when the resource exists' do
+              let(:local_resource) { stub }
+
+              it 'returns the resource' do
+                subject.must_equal(local_resource)
+              end
+            end
+
+            context 'when the resource does not exist' do
+              it { subject.must_equal nil }
+            end
+          end
+
+          context 'when remote access is allowed' do
+            context 'attempts remote retrieval by ID' do
+              context 'when the resource exists' do
+                let(:resource)        { stub }
+                let(:remote_resource) { [resource] }
+
+                it 'returns the resource' do
+                  subject.must_equal(resource)
+                end
+              end
+
+              context 'when the resource does not exist' do
+                it { subject.must_equal nil }
+              end
+            end
+          end
+
+          context 'when remote access is not allowed' do
+            let(:options) { { allow_remote: false } }
+
+            it { subject.must_equal nil }
+          end
+
+          context 'finally when the resource cannot be found' do
+            it { subject.must_equal nil }
+          end
+        end
+      end
+    end
+  end
+end

data/test/lib/troo/retrieval/remote_test.rb

@@ -0,0 +1,49 @@
+require_relative '../../../test_helper'
+
+module Troo
+  module Retrieval
+    describe Remote do
+      let(:described_class) { Remote }
+      let(:klass)           do
+        mock('klass', all:            {},
+                      by_board_id:    {},
+                      by_list_id:     {},
+                      by_card_id:     {},
+                      by_member_id:   {})
+      end
+      let(:external_id)     { '526d8e130a14a9d846001d96' }
+      let(:options)         { { mode: :board } }
+      let(:resources)       { [:not_persisted] }
+      let(:persisted)       { [:persisted] }
+
+      before do
+        klass.stubs(:remote_options).returns({})
+        API::Client.stubs(:perform).returns(resources)
+        Troo::Persistence::Local.stubs(:with_collection)
+          .returns(persisted)
+      end
+
+      describe '#fetch' do
+        subject { described_class.fetch(klass, external_id, options) }
+
+        context 'when no resources were retrieved' do
+          let(:resources) { [] }
+
+          it { subject.must_equal [] }
+        end
+
+        context 'when resources were retrieved' do
+          context 'and persistence is allowed' do
+            it { subject.must_equal [:persisted] }
+          end
+
+          context 'and persistence is not allowed' do
+            let(:options) { { persist: false } }
+
+            it { subject.must_equal [:not_persisted] }
+          end
+        end
+      end
+    end
+  end
+end

data/test/support/fabrication.rb

@@ -1,5 +1,5 @@
 Fabricator(:board, from: Troo::Board) do
-  external_board_id '526d8e130a14a9d846001d96'
+  external_id '526d8e130a14a9d846001d96'
   name              'My Test Board'
   description       'A very brief description...'
   default           false
@@ -8,7 +8,7 @@ end
 
 Fabricator(:list, from: Troo::List) do
   external_board_id '526d8e130a14a9d846001d96'
-  external_list_id  '526d8e130a14a9d846001d97'
+  external_id       '526d8e130a14a9d846001d97'
   name              'My Test List'
   position          32_768
   default           false
@@ -18,7 +18,7 @@ end
 Fabricator(:card, from: Troo::Card) do
   external_board_id   '526d8e130a14a9d846001d96'
   external_list_id    '526d8e130a14a9d846001d97'
-  external_card_id    '526d8f19ddb279532e005259'
+  external_id         '526d8f19ddb279532e005259'
   external_member_ids ['5195fdb5a8c01a2318004f5d']
   short_id            '67'
   name                'My Test Card'
@@ -31,23 +31,23 @@ Fabricator(:card, from: Troo::Card) do
 end
 
 Fabricator(:comment, from: Troo::Comment) do
-  external_board_id   '526d8e130a14a9d846001d96'
-  external_card_id    '526d8f19ddb279532e005259'
-  external_comment_id '51f9277b2822b8654f0023af'
-  external_member_id  '5195fdb5a8c01a2318004f5d'
-  text                'My Test Comment'
-  date                '2013-12-17 22:01:13 UTC'
+  external_board_id  '526d8e130a14a9d846001d96'
+  external_card_id   '526d8f19ddb279532e005259'
+  external_id        '51f9277b2822b8654f0023af'
+  external_member_id '5195fdb5a8c01a2318004f5d'
+  text               'My Test Comment'
+  date               '2013-12-17 22:01:13 UTC'
 end
 
 Fabricator(:member, from: Troo::Member) do
-  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/'
+  external_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
 
 Fabricator(:refresh, from: Troo::Refresh) do

data/test/support/fake_trello/fake_response.rb

@@ -0,0 +1,38 @@
+require 'json'
+require 'yajl'
+
+class FakeResponse
+  def self.render(resource, id = nil, collection = false)
+    new(resource, id, collection).render
+  end
+
+  def initialize(resource, id = nil, collection = false)
+    @resource, @id, @collection = resource, id, collection
+  end
+
+  def render
+    collection ? '[' + output + ']' : output
+  end
+
+  private
+
+  attr_reader :resource, :id, :collection
+
+  def output
+    return converted if id
+    raw
+  end
+
+  def converted
+    JSON(parsed)
+  end
+
+  def parsed
+    @parsed ||= Yajl::Parser.parse(raw).merge!('id' => id)
+  end
+
+  def raw
+    @raw ||= File
+      .read(File.dirname(__FILE__) + "/../remotes/#{resource}.json")
+  end
+end

data/test/support/fake_trello/server.rb

@@ -0,0 +1,124 @@
+require 'sinatra/base'
+require 'webrick'
+require 'webrick/https'
+require 'openssl'
+
+require_relative 'fake_response'
+
+trap('INT') { exit! }
+
+my_server_crt   = File.open(File.join('./', 'my-server.crt')).read
+my_server_key   = File.open(File.join('./', 'my-server.key')).read
+webrick_options = {
+  Port:            8443,
+  Logger:          WEBrick::Log.new($stderr, WEBrick::Log::DEBUG),
+  DocumentRoot:    '/',
+  SSLEnable:       true,
+  SSLVerifyClient: OpenSSL::SSL::VERIFY_NONE,
+  SSLCertificate:  OpenSSL::X509::Certificate.new(my_server_crt),
+  SSLPrivateKey:   OpenSSL::PKey::RSA.new(my_server_key),
+  SSLCertName:     [['CN', WEBrick::Utils.getservername]]
+}
+
+class MyFakeTrello < Sinatra::Base
+  helpers do
+    def fake(resource, id = nil, collection = false)
+      halt(400, 'Bad Request')  if id == '400'
+      halt(401, 'Unauthorized') if id == '401'
+
+      FakeResponse.render(resource, id, collection)
+    end
+  end
+
+  # boards_all
+  get '/1/members/me/boards' do
+    fake('all_boards', nil, true)
+  end
+
+  # board_by_id
+  get '/1/boards/:id' do
+    fake('board', params['id'])
+  end
+
+  # card_by_id
+  get '/1/cards/:id' do
+    fake('card_by_card_id', params['id'])
+  end
+
+  # list_by_id
+  get '/1/lists/:id' do
+    fake('list', params['id'])
+  end
+
+  # member_by_id
+  get '/1/members/:id' do
+    fake('member', params['id'])
+  end
+
+  # cards_by_board_id
+  get '/1/boards/:id/cards' do
+    fake('card', params['id'], true)
+  end
+
+  # cards_by_list_id
+  get '/1/lists/:id/cards' do
+    fake('card', params['id'], true)
+  end
+
+  # comments_by_board_id
+  get '/1/boards/:id/actions' do
+    fake('comment', params['id'])
+  end
+
+  # comments_by_card_id
+  get '/1/cards/:id/actions' do
+    fake('comment', params['id'])
+  end
+
+  # comments_by_list_id
+  get '/1/lists/:id/actions' do
+    fake('comment', params['id'])
+  end
+
+  # lists_by_board_id
+  get '/1/boards/:id/lists' do
+    fake('list', params['id'], true)
+  end
+
+  # members_by_board_id
+  get '/1/boards/:id/members' do
+    fake('member', params['id'], true)
+  end
+
+  # create_board
+  post '/1/boards' do
+    fake('board')
+  end
+
+  # create_card
+  post '/1/cards' do
+    fake('card')
+  end
+
+  # create_comment
+  post '/1/cards/:id/actions/comments' do
+    fake('comment')
+  end
+
+  # create_list
+  post '/1/lists' do
+    fake('list')
+  end
+
+  # move_card_list
+  put '/1/cards/:id/idList' do
+    fake('card')
+  end
+
+  # move_card_board
+  put '/1/cards/:id/idBoard' do
+    fake('card')
+  end
+end
+
+Rack::Handler::WEBrick.run MyFakeTrello, webrick_options