troo 0.0.10 → 0.0.11

data/lib/troo/remote/persistence/board.rb

@@ -3,16 +3,23 @@ module Troo
     module Persistence
       class Board
         class << self
+          # @param  [String]
+          # @param  [String, NilClass]
+          # @return []
           def with(name, description = nil)
             new(name, description).perform
           end
         end
 
+        # @param  [String]
+        # @param  [String, NilClass]
+        # @return []
         def initialize(name, description = nil)
           @name        = name
           @description = description
         end
 
+        # @return []
         def perform
           create_local
         end

data/lib/troo/remote/persistence/card.rb

@@ -3,17 +3,26 @@ module Troo
     module Persistence
       class Card
         class << self
+          # @param  [String]
+          # @param  [String, NilClass]
+          # @param  [String, NilClass]
+          # @return []
           def with(external_list_id, name = nil, description = nil)
             new(external_list_id, name, description).perform
           end
         end
 
+        # @param  [String]
+        # @param  [String, NilClass]
+        # @param  [String, NilClass]
+        # @return []
         def initialize(external_list_id, name = nil, description = nil)
           @external_list_id = external_list_id
           @name             = name
           @description      = description
         end
 
+        # @return []
         def perform
           create_local
         end

data/lib/troo/remote/persistence/comment.rb

@@ -3,16 +3,23 @@ module Troo
     module Persistence
       class Comment
         class << self
+          # @param  [String]
+          # @param  []
+          # @return []
           def with(external_card_id, comment)
             new(external_card_id, comment).perform
           end
         end
 
+        # @param  [String]
+        # @param  []
+        # @return []
         def initialize(external_card_id, comment)
           @external_card_id = external_card_id
           @comment          = comment
         end
 
+        # @return []
         def perform
           create_local
         end

data/lib/troo/remote/persistence/list.rb

@@ -3,16 +3,23 @@ module Troo
     module Persistence
       class List
         class << self
+          # @param  [String]
+          # @param  [String]
+          # @return []
           def with(external_board_id, name)
             new(external_board_id, name).perform
           end
         end
 
+        # @param  [String]
+        # @param  [String]
+        # @return []
         def initialize(external_board_id, name)
           @external_board_id = external_board_id
           @name              = name
         end
 
+        # @return []
         def perform
           create_local
         end

data/lib/troo/remote/persistence/move_card.rb

@@ -3,6 +3,10 @@ module Troo
     module Persistence
       class MoveCard
         class << self
+          # @param  [String]
+          # @param  [String]
+          # @param  [String, NilClass]
+          # @return []
           def with(external_card_id,
                    external_list_id,
                    external_board_id = nil)
@@ -11,6 +15,10 @@ module Troo
           end
         end
 
+        # @param  [String]
+        # @param  [String]
+        # @param  [String, NilClass]
+        # @return []
         def initialize(external_card_id,
                        external_list_id,
                        external_board_id = nil)
@@ -19,6 +27,7 @@ module Troo
           @external_board_id = external_board_id
         end
 
+        # @return []
         def perform
           update_cards
         end

data/lib/troo/retrieval/local.rb

@@ -4,31 +4,47 @@ module Troo
       attr_reader :id
 
       class << self
+        # @param  []
+        # @return []
         def all(klass)
           new(klass).all
         end
 
+        # @param  []
+        # @param  [Hash]
+        # @return []
         def default(klass, options = {})
           new(klass, nil, options).default
         end
 
+        # @param  []
+        # @param  []
+        # @param  [Hash]
+        # @return []
         def retrieve(klass, id = nil, options = {})
           new(klass, id, options).retrieve
         end
       end
 
+      # @param  []
+      # @param  []
+      # @param  [Hash]
+      # @return []
       def initialize(klass, id = nil, options = {})
         @klass, @id, @options = klass, id, options
       end
 
+      # @return []
       def all
         klass.all
       end
 
+      # @return []
       def default
         klass.default
       end
 
+      # @return []
       def retrieve
         return default unless id
         by_short_id || by_id || by_external_id || optional_remote

data/lib/troo/retrieval/remote.rb

@@ -4,17 +4,26 @@ module Troo
       attr_reader :external_id
 
       class << self
+        # @param  []
+        # @param  []
+        # @param  [Hash]
+        # @return []
         def fetch(klass, external_id, options = {})
           new(klass, external_id, options).fetch
         end
       end
 
+      # @param  []
+      # @param  []
+      # @param  [Hash]
+      # @return []
       def initialize(klass, external_id, options = {})
         @klass       = klass
         @external_id = external_id
         @options     = options
       end
 
+      # @return []
       def fetch
         return []        if none?
         return persist   if persist?

data/lib/troo/troo.rb

@@ -20,9 +20,11 @@ require 'virtus'
 require 'yajl'
 require 'yaml'
 
+require_relative 'preference'
 require_relative 'configuration'
 require_relative 'database'
 require_relative 'debug'
+require_relative 'options'
 
 require_relative 'api/endpoints'
 require_relative 'api/oauth_settings'
@@ -86,3 +88,5 @@ require_relative 'cli/default'
 require_relative 'cli/refresh'
 require_relative 'cli/show'
 require_relative 'cli/main'
+
+require_relative 'launcher'

data/lib/troo/version.rb

@@ -1,3 +1,3 @@
 module Troo
-  VERSION = '0.0.10'
+  VERSION = '0.0.11'
 end

data/test/lib/troo/api/client_test.rb

@@ -4,7 +4,7 @@ module Troo
   module API
     describe Client do
       let(:described_class) { Client }
-      let(:parameters) do
+      let(:parameters)      {
         {
           verb:          :get,
           endpoint:      endpoint,
@@ -12,7 +12,7 @@ module Troo
           model:         Remote::Board,
           query:         {}
         }
-      end
+      }
       let(:endpoint) { :board_by_id }
       let(:response) { Response.new }
       let(:parsed_response) { '' }
@@ -28,9 +28,9 @@ module Troo
 
         context 'when all required parameters are provided' do
           context 'and the API request returns a collection' do
-            let(:parsed_response) do
+            let(:parsed_response) {
               [{ name: 'Board 1' }, { name: 'Board 2' }]
-            end
+            }
 
             it 'builds the remote model' do
               subject.size.must_equal(2)

data/test/lib/troo/api/response_test.rb

@@ -2,14 +2,41 @@ require_relative '../../../test_helper'
 
 module Troo
   module API
+    describe Responder do
+      let(:described_class) { Responder }
+      let(:parameters)      { { code: code } }
+
+      subject { described_class.build(parameters) }
+
+      context 'when the status code is 200' do
+        let(:code) { '200' }
+
+        it { subject.must_be_instance_of(Troo::API::Response) }
+      end
+
+      context 'when the status code is any other value' do
+        let(:code) { '410' }
+
+        it { subject.must_be_instance_of(Troo::API::ErrorResponse) }
+      end
+    end
+
     describe Response do
       let(:described_class) { Response }
       let(:parameters)      { {} }
+
+      subject { described_class.new(parameters) }
+
+      it { subject.must_be_instance_of(Troo::API::Response) }
     end
 
     describe ErrorResponse do
       let(:described_class) { ErrorResponse }
       let(:parameters)      { {} }
+
+      subject { described_class.new(parameters) }
+
+      it { subject.must_be_instance_of(Troo::API::ErrorResponse) }
     end
   end
 end

data/test/lib/troo/cli/commands/refresh/all_test.rb

@@ -4,9 +4,9 @@ module Troo
   module Commands
     describe RefreshAll do
       let(:described_class) { RefreshAll }
-      let(:resource) do
+      let(:resource)        {
         [mock_trello_response('board_200.json', Troo::Remote::Board)]
-      end
+      }
 
       before { Retrieval::Remote.stubs(:fetch).returns(resource) }
 

data/test/lib/troo/cli/commands/refresh_test.rb

@@ -7,9 +7,9 @@ module Troo
       let(:id)              {}
       let(:resource)        {}
       let(:retrieved)       {}
-      let(:klass) do
+      let(:klass)           {
         stub(type: :resource_type, remote: stub)
-      end
+      }
 
       before do
         API::Client.stubs(:perform)

data/test/lib/troo/configuration_test.rb

@@ -4,7 +4,7 @@ module Troo
   describe Configuration do
     let(:described_class) { Configuration }
 
-    subject { described_class.load('config/trooconf.yml', :test) }
+    subject { described_class.load('config/trooconf.yml', 'test') }
 
     context 'configures' do
       it 'the name of the configuration' do

data/test/lib/troo/database_test.rb

@@ -0,0 +1,7 @@
+require_relative '../../test_helper'
+
+module Troo
+  describe Database do
+    let(:described_class) { Database }
+  end
+end

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

@@ -4,7 +4,7 @@ module Troo
   module Decorators
     describe Resource do
       let(:described_class)    { Resource }
-      let(:klass) do
+      let(:klass)              {
         stub(id:          67,
              name:        resource_name,
              description: description,
@@ -12,7 +12,7 @@ module Troo
              text:        'Some text...',
              date:        'Wed, Dec 17 at 22:01',
              type:        :resource_type)
-      end
+      }
       let(:options)            { {} }
       let(:described_instance) { described_class.new(klass, options) }
       let(:resource_name)      { 'My Resource' }
@@ -61,7 +61,9 @@ module Troo
         context 'when the resource is not the default' do
           let(:default) { false }
 
-          it { subject.must_equal nil }
+          it 'returns nil so that join compacts the value away' do
+            subject.must_equal nil
+          end
         end
       end
 

data/test/lib/troo/launcher_test.rb

@@ -0,0 +1,7 @@
+require_relative '../../test_helper'
+
+module Troo
+  describe Launcher do
+    let(:described_class) { Launcher }
+  end
+end

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

@@ -34,11 +34,11 @@ module Troo
           end
 
           context 'and the attribute is a string' do
-            let(:member_ids) { "[\"20050\"]" }
-            let(:described_instance) do
+            let(:member_ids)         { "[\"20050\"]" }
+            let(:described_instance) {
               Fabricate.build(:card,
                               external_member_ids: member_ids)
-            end
+            }
 
             it 'converts to an array and returns the attribute' do
               subject.external_member_ids
@@ -48,9 +48,9 @@ module Troo
         end
 
         context 'when the attribute is not set' do
-          let(:described_instance) do
+          let(:described_instance) {
             Fabricate.build(:card, external_member_ids: nil)
-          end
+          }
 
           it 'returns an empty collection' do
             subject.external_member_ids.must_equal([])

data/test/lib/troo/options_test.rb

@@ -0,0 +1,7 @@
+require_relative '../../test_helper'
+
+module Troo
+  describe Options do
+    let(:described_class) { Options }
+  end
+end

data/test/lib/troo/preference_test.rb

@@ -0,0 +1,21 @@
+require_relative '../../test_helper'
+
+module Troo
+  describe Preference do
+    let(:described_class) { Preference }
+    let(:parameters)      {
+      {
+        label: 'my_preference',
+        value: 'Some value...'
+      }
+    }
+
+    describe '#view' do
+      subject { described_class.view(parameters) }
+
+      it 'presents the preference' do
+        subject.must_equal('           my_preference: Some value...')
+      end
+    end
+  end
+end

data/test/lib/troo/presentation/formatter_test.rb

@@ -21,7 +21,7 @@ end
 module Troo
   describe Wordwrap do
     let(:described_class) { Wordwrap }
-    let(:value) do
+    let(:value)           {
       'Lorem ipsum dolor sit amet, consectetur adipiscing elit. '    \
       "Curabitur aliquet turpis id dui condimentum elementum.\n"     \
       'Pellentesque blandit vulputate imperdiet. Quisque ut arcu '   \
@@ -35,11 +35,11 @@ module Troo
       "quis diam.\n\nDonec mollis, nisi sit amet congue sagittis, "  \
       'sapien magna rhoncus justo, vel molestie metus sapien eget '  \
       "libero.\n\n\n"
-    end
-    let(:options) { {} }
+    }
+    let(:options)         { {} }
 
     describe '#wordwrap' do
-      let(:formatted_value) do
+      let(:formatted_value) {
         "Lorem ipsum dolor sit amet, consectetur adipiscing elit. "  \
         "Curabitur\naliquet turpis id dui condimentum elementum.\n"  \
         "Pellentesque blandit vulputate imperdiet. Quisque ut arcu " \
@@ -53,7 +53,7 @@ module Troo
         "lectus placerat\ngravida sit amet quis diam.\n\nDonec "     \
         "mollis, nisi sit amet congue sagittis, sapien magna "       \
         "rhoncus\njusto, vel molestie metus sapien eget libero."
-      end
+      }
 
       subject { described_class.this(value, options) }
 
@@ -62,11 +62,11 @@ module Troo
       end
 
       context 'when the content should be pruned' do
-        let(:options) { { width: 70, prune: true } }
-        let(:formatted_value) do
+        let(:options)         { { width: 70, prune: true } }
+        let(:formatted_value) {
           'Lorem ipsum dolor sit amet, consectetur adipiscing elit.' \
           " Curabitur a...\e[0m"
-        end
+        }
 
         it 'returns formatted text' do
           subject.must_equal(formatted_value)