troo 0.0.4 → 0.0.5

data/test/lib/troo/cli/main_cli_test.rb

@@ -11,53 +11,55 @@ module Troo
       after  { database_cleanup }
 
       describe "#status" do
+        let(:default) { false }
+
         before do
-          @board = Fabricate(:board, default: true)
-          @list  = Fabricate(:list, default: true)
-          @card  = Fabricate(:card, default: true)
+          @board = Fabricate(:board, default: default)
+          @list  = Fabricate(:list, default: default)
+          @card  = Fabricate(:card, default: default)
         end
 
         subject { capture_io { described_instance.status }.join }
 
         context "when a default board is set" do
+          let(:default) { true }
+
           it "returns a polite message" do
             subject.must_match /My Test Board/
           end
         end
 
         context "when a default board is not set" do
-          before { Troo::BoardRetrieval.stubs(:default).returns(nil) }
-
           it "returns a polite message" do
-            subject.must_match /Board: No default/
+            subject.must_match /No default board set/
           end
         end
 
         context "when a default list is set" do
+          let(:default) { true }
+
           it "returns a polite message" do
             subject.must_match /My Test List/
           end
         end
 
         context "when a default list is not set" do
-          before { Troo::ListRetrieval.stubs(:default).returns(nil) }
-
           it "returns a polite message" do
-            subject.must_match /List: No default/
+            subject.must_match /No default list set/
           end
         end
 
         context "when a default card is set" do
+          let(:default) { true }
+
           it "returns a polite message" do
             subject.must_match /My Test Card/
           end
         end
 
         context "when a default card is not set" do
-          before { Troo::CardRetrieval.stubs(:default).returns(nil) }
-
           it "returns a polite message" do
-            subject.must_match /Card: No default/
+            subject.must_match /No default card set/
           end
         end
 
@@ -65,7 +67,7 @@ module Troo
           before { Troo::Board.stubs(:count).returns(0) }
 
           it "returns a polite message" do
-            subject.must_match /No local board data/
+            subject.must_match /No boards found/
           end
         end
 
@@ -73,7 +75,7 @@ module Troo
           before { Troo::List.stubs(:count).returns(0) }
 
           it "returns a polite message" do
-            subject.must_match /No local list data/
+            subject.must_match /No lists found/
           end
         end
 
@@ -81,7 +83,7 @@ module Troo
           before { Troo::Card.stubs(:count).returns(0) }
 
           it "returns a polite message" do
-            subject.must_match /No local card data/
+            subject.must_match /No cards found/
           end
         end
       end
@@ -94,7 +96,7 @@ module Troo
         before do
           RefreshAll.stubs(:all)
           RefreshAll.stubs(:default)
-          Troo::BoardRetrieval.stubs(:default).returns(default)
+          BoardRetrieval.stubs(:default).returns(default)
           RefreshAll.stubs(:lists)
           RefreshAll.stubs(:cards)
         end
@@ -187,8 +189,8 @@ module Troo
           @card = Fabricate(:card)
           @list = Fabricate(:list)
           Troo::MoveCard.stubs(:with).returns(true)
-          Troo::CardRetrieval.stubs(:retrieve).returns(@card)
-          Troo::ListRetrieval.stubs(:retrieve).returns(@list)
+          CardRetrieval.stubs(:retrieve).returns(@card)
+          ListRetrieval.stubs(:retrieve).returns(@list)
         end
 
         subject { capture_io { described_class.new.move(card_id, list_id) }.join }
@@ -211,7 +213,7 @@ module Troo
           end
 
           context "when the list was not found" do
-            before { Troo::ListRetrieval.stubs(:retrieve).returns(nil) }
+            before { ListRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
               subject.must_match /list was not found/
@@ -220,7 +222,7 @@ module Troo
         end
 
         context "when the card was not found" do
-          before { Troo::CardRetrieval.stubs(:retrieve).returns(nil) }
+          before { CardRetrieval.stubs(:retrieve) }
 
           it "returns a polite message" do
             subject.must_match /card was not found/

data/test/lib/troo/cli/show_cli_test.rb

@@ -31,7 +31,7 @@ module Troo
         end
 
         context "when no boards exist" do
-          before { Troo::BoardRetrieval.stubs(:all).returns([]) }
+          before { BoardRetrieval.stubs(:all).returns([]) }
 
           it "returns a polite message" do
             subject.must_match /Boards not found./
@@ -49,13 +49,21 @@ module Troo
               subject.must_match /Test List/
               subject.must_match /Test Card/
             end
+
+            context "but there are no lists" do
+              before { @list.delete }
+
+              it "returns a polite message" do
+                subject.must_match /No lists were found/
+              end
+            end
           end
 
           context "and the board does not exist" do
-            before { Troo::BoardRetrieval.stubs(:retrieve).returns() }
+            before { BoardRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
-              subject.must_match /Board not found./
+              subject.must_match /Board cannot be found./
             end
           end
         end
@@ -64,7 +72,7 @@ module Troo
           let(:board_id) { }
 
           context "and the default board is set" do
-            before { Troo::BoardRetrieval.stubs(:retrieve).returns(@board) }
+            before { BoardRetrieval.stubs(:retrieve).returns(@board) }
 
             it "returns the board with all lists and all cards" do
               subject.must_match /Test Board/
@@ -74,7 +82,7 @@ module Troo
           end
 
           context "and the default board is not set" do
-            before { Troo::BoardRetrieval.stubs(:retrieve).returns() }
+            before { BoardRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
               subject.must_match /set a default board first/
@@ -87,19 +95,27 @@ module Troo
         subject { capture_io { described_class.new.list(list_id) }.join }
 
         context "when a list_id was provided" do
-          context "when the list exists" do
+          context "and the list exists" do
             it "returns the list's board, the list and all cards" do
               subject.must_match /Test Board/
               subject.must_match /Test List/
               subject.must_match /Test Card/
             end
+
+            context "but there are no cards" do
+              before { @card.delete }
+
+              it "returns a polite message" do
+                subject.must_match /No cards were found/
+              end
+            end
           end
 
-          context "when the list does not exist" do
-            before { Troo::ListRetrieval.stubs(:retrieve).returns() }
+          context "and the list does not exist" do
+            before { ListRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
-              subject.must_match /List not found./
+              subject.must_match /List cannot be found/
             end
           end
         end
@@ -108,7 +124,7 @@ module Troo
           let(:list_id) { }
 
           context "and the default list is set" do
-            before { Troo::ListRetrieval.stubs(:retrieve).returns(@list) }
+            before { ListRetrieval.stubs(:retrieve).returns(@list) }
 
             it "returns the list's board, the list and all cards" do
               subject.must_match /Test Board/
@@ -118,7 +134,7 @@ module Troo
           end
 
           context "and the default list is not set" do
-            before { Troo::ListRetrieval.stubs(:retrieve).returns() }
+            before { ListRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
               subject.must_match /set a default list first/
@@ -131,22 +147,28 @@ module Troo
         subject { capture_io { described_class.new.card(card_id) }.join }
 
         context "when a card_id was provided" do
-          context "when the card exists" do
-            before { @comment.delete }
-
+          context "and the card exists" do
             it "returns the card details" do
               subject.must_match /\(67\) My Test Card/
               subject.must_match /some description/
               subject.must_match /Metadata/
               subject.must_match /Tue, Dec 17 at 21:48/
             end
+
+            context "but there are no comments" do
+              before { @comment.delete }
+
+              it "returns a polite message" do
+                subject.must_match /No comments have been left/
+              end
+            end
           end
 
-          context "when the card does not exist" do
-            before { Troo::CardRetrieval.stubs(:retrieve).returns() }
+          context "and the card does not exist" do
+            before { CardRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
-              subject.must_match /Card not found./
+              subject.must_match /Card cannot be found/
             end
           end
         end
@@ -155,7 +177,7 @@ module Troo
           let(:card_id) { }
 
           context "and the default card is set" do
-            before { Troo::CardRetrieval.stubs(:retrieve).returns(@card) }
+            before { CardRetrieval.stubs(:retrieve).returns(@card) }
 
             it "returns the card details" do
               subject.must_match /\(67\) My Test Card/
@@ -166,7 +188,7 @@ module Troo
           end
 
           context "and the default card is not set" do
-            before { Troo::CardRetrieval.stubs(:retrieve).returns() }
+            before { CardRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
               subject.must_match /set a default card first/
@@ -179,18 +201,26 @@ module Troo
         subject { capture_io { described_class.new.comments(card_id) }.join }
 
         context "when a card_id was provided" do
-          context "when the card exists" do
+          context "and the card exists" do
             it "returns the card and all comments" do
-              subject.must_match /\(67\) My Test Card/
+              subject.must_match /My Test Card/
               subject.must_match /My Test Comment/
             end
+
+            context "but there are no comments" do
+              before { @comment.delete }
+
+              it "returns a polite message" do
+                subject.must_match /No comments were found/
+              end
+            end
           end
 
-          context "when the card does not exist" do
-            before { Troo::CardRetrieval.stubs(:retrieve).returns() }
+          context "and the card does not exist" do
+            before { CardRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
-              subject.must_match /Card not found./
+              subject.must_match /Card cannot be found/
             end
           end
         end
@@ -199,16 +229,16 @@ module Troo
           let(:card_id) { }
 
           context "and the default card is set" do
-            before { Troo::CardRetrieval.stubs(:retrieve).returns(@card) }
+            before { CardRetrieval.stubs(:retrieve).returns(@card) }
 
             it "returns the card and all comments" do
-              subject.must_match /\(67\) My Test Card/
+              subject.must_match /My Test Card/
               subject.must_match /My Test Comment/
             end
           end
 
           context "and the default card is not set" do
-            before { Troo::CardRetrieval.stubs(:retrieve).returns() }
+            before { CardRetrieval.stubs(:retrieve) }
 
             it "returns a polite message" do
               subject.must_match /set a default card first/

data/test/lib/troo/display/board_decorator_test.rb

@@ -71,7 +71,7 @@ module Troo
       end
 
       context "when the board has no description" do
-        before { @board.stubs(:description).returns(nil) }
+        before { @board.stubs(:description) }
 
         it "returns N/A" do
           subject.must_equal("N/A")
@@ -89,7 +89,7 @@ module Troo
       end
 
       context "when the board has no name" do
-        before { @board.stubs(:name).returns(nil) }
+        before { @board.stubs(:name) }
 
         it "returns N/A" do
           subject.must_equal("N/A")

data/test/lib/troo/display/board_presenter_test.rb

@@ -44,8 +44,8 @@ module Troo
       end
     end
 
-    describe "#render_show" do
-      subject { capture_io { described_class.new(@board, options).render_show }.join }
+    describe "#show" do
+      subject { capture_io { described_class.show(@board, options) }.join }
 
       context "when the board has lists" do
         context "and the list has cards" do

data/test/lib/troo/display/card_decorator_test.rb

@@ -38,31 +38,23 @@ module Troo
       subject { described_instance.short }
 
       it "returns a one line overview of the card" do
-        subject.must_equal(" *  (67) My Test Card\n")
+        subject.must_equal(" * \e[35m\e[4m(67) \e[0m\e[35m\e[4mMy Test Card\e[0m\n")
       end
 
       context "when the ansicolor option is false" do
         let(:options) { { ansicolor: false } }
 
-        it "returns a one line overview of the board" do
+        it "returns a one line overview of the card" do
           subject.must_equal(" *  (67) My Test Card\n")
         end
       end
     end
 
-    describe "#title" do
-      subject { described_instance.title }
-
-      it "returns a formatted string representing the card title" do
-        subject.must_match /\* \(67\) My Test Card/
-      end
-    end
-
     describe "#name_str" do
       subject { described_instance.name_str }
 
       it "returns the formatted card name" do
-        subject.must_equal("My Test Card")
+        subject.must_equal("\e[35m\e[4mMy Test Card\e[0m")
       end
     end
 
@@ -78,7 +70,7 @@ module Troo
       subject { described_instance.id_str }
 
       it "returns the formatted card id" do
-        subject.must_equal(" (67) ")
+        subject.must_equal("\e[35m\e[4m(67) \e[0m")
       end
     end
 
@@ -206,7 +198,7 @@ module Troo
       subject { described_instance.board }
 
       it "returns the board details" do
-        subject.must_match "(1) My Test Board"
+        subject.must_equal "   \e[35m\e[4m(1) \e[0m\e[35m\e[4mMy Test Board\e[0m\n"
       end
     end
 
@@ -214,7 +206,7 @@ module Troo
       subject { described_class.new(@card).list }
 
       it "returns the list details" do
-        subject.must_match "(1) My Test List"
+        subject.must_equal "   \e[35m\e[4m(1) \e[0m\e[35m\e[4mMy Test List\e[0m\n"
       end
     end
   end

data/test/lib/troo/display/card_presenter_test.rb

@@ -25,8 +25,8 @@ module Troo
       end
     end
 
-    describe "#render_show" do
-      subject { capture_io { described_class.render_show(@card, options) }.join }
+    describe "#show" do
+      subject { capture_io { described_class.show(@card, options) }.join }
 
       it "renders the view" do
         subject.must_match /My Test Card/

data/test/lib/troo/display/comment_decorator_test.rb

@@ -4,6 +4,7 @@ module Troo
   describe CommentDecorator do
     let(:described_class) { CommentDecorator }
     let(:default) { true }
+    let(:options) { {} }
 
     before do
       @comment = Fabricate(:comment)
@@ -13,11 +14,15 @@ module Troo
     after { database_cleanup }
 
     describe "#initialize" do
-      subject { described_class.new(@comment) }
+      subject { described_class.new(@comment, options) }
 
       it "assigns the comment to an instance variable" do
         subject.instance_variable_get("@comment").must_equal(@comment)
       end
+
+      it "assigns the options to an instance variable" do
+        subject.instance_variable_get("@options").must_equal(options)
+      end
     end
 
     describe "#as_view" do

data/test/lib/troo/display/comment_presenter_test.rb

@@ -25,8 +25,8 @@ module Troo
       end
     end
 
-    describe "#render_show" do
-      subject { capture_io { described_class.render_show(@card, options) }.join }
+    describe "#show" do
+      subject { capture_io { described_class.show(@card, options) }.join }
 
       context "when the card has comments" do
         it "renders the view" do

data/test/lib/troo/display/decorator_helpers_test.rb

@@ -1,7 +1,27 @@
 require_relative "../../../test_helper"
 
+class DecoratorHelpersDummy
+  include Troo::DecoratorHelpers
+end
+
 module Troo
   describe DecoratorHelpers do
-    let(:described_class) { DecoratorHelpers }
+    let(:described_class) { DecoratorHelpersDummy }
+    let(:described_instance) { described_class.new }
+
+    describe "#title" do
+      before do
+        described_instance.stubs(:default).returns("*")
+        described_instance.stubs(:id).returns(67)
+        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/
+      end
+    end
   end
 end

data/test/lib/troo/display/list_presenter_test.rb

@@ -25,8 +25,8 @@ module Troo
       end
     end
 
-    describe "#render_show" do
-      subject { capture_io { described_class.new(@list, options).render_show }.join }
+    describe "#show" do
+      subject { capture_io { described_class.show(@list, options) }.join }
 
       context "when the list has cards" do
         it "renders the view" do

data/test/lib/troo/display/member_decorator_test.rb

@@ -3,16 +3,21 @@ require_relative "../../../test_helper"
 module Troo
   describe MemberDecorator do
     let(:described_class) { MemberDecorator }
+    let(:options) { {} }
 
     before { @member = Fabricate(:member) }
     after  { database_cleanup }
 
     describe "#initialize" do
-      subject { described_class.new(@member) }
+      subject { described_class.new(@member, options) }
 
       it "assigns the member to an instance variable" do
         subject.instance_variable_get("@member").must_equal(@member)
       end
+
+      it "assigns the options to an instance variable" do
+        subject.instance_variable_get("@options").must_equal(options)
+      end
     end
 
     describe "#username" do

data/test/lib/troo/display/member_presenter_test.rb

@@ -25,8 +25,8 @@ module Troo
       end
     end
 
-    describe "#render_show" do
-      subject { described_class.render_show(@card, options) }
+    describe "#show" do
+      subject { described_class.show(@card, options) }
 
       context "when there is one member" do
         it "returns the member" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: troo
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Gavin Laking
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-18 00:00:00.000000000 Z
+date: 2014-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aruba
@@ -320,10 +320,10 @@ files:
 - lib/troo/actions/refresh_all.rb
 - lib/troo/actions/set_default.rb
 - lib/troo/cli/add_cli.rb
+- lib/troo/cli/cli_helpers.rb
 - lib/troo/cli/default_cli.rb
 - lib/troo/cli/main_cli.rb
 - lib/troo/cli/show_cli.rb
-- lib/troo/cli/thor_fixes.rb
 - lib/troo/display/board_decorator.rb
 - lib/troo/display/board_presenter.rb
 - lib/troo/display/card_decorator.rb
@@ -395,10 +395,10 @@ files:
 - test/lib/troo/actions/refresh_all_test.rb
 - test/lib/troo/actions/set_default_test.rb
 - test/lib/troo/cli/add_cli_test.rb
+- test/lib/troo/cli/cli_helpers_test.rb
 - test/lib/troo/cli/default_cli_test.rb
 - test/lib/troo/cli/main_cli_test.rb
 - test/lib/troo/cli/show_cli_test.rb
-- test/lib/troo/cli/thor_fixes_test.rb
 - test/lib/troo/display/board_decorator_test.rb
 - test/lib/troo/display/board_presenter_test.rb
 - test/lib/troo/display/card_decorator_test.rb
@@ -465,7 +465,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.0
+rubygems_version: 2.2.1
 signing_key: 
 specification_version: 4
 summary: CLI interface for Trello.
@@ -513,10 +513,10 @@ test_files:
 - test/lib/troo/actions/refresh_all_test.rb
 - test/lib/troo/actions/set_default_test.rb
 - test/lib/troo/cli/add_cli_test.rb
+- test/lib/troo/cli/cli_helpers_test.rb
 - test/lib/troo/cli/default_cli_test.rb
 - test/lib/troo/cli/main_cli_test.rb
 - test/lib/troo/cli/show_cli_test.rb
-- test/lib/troo/cli/thor_fixes_test.rb
 - test/lib/troo/display/board_decorator_test.rb
 - test/lib/troo/display/board_presenter_test.rb
 - test/lib/troo/display/card_decorator_test.rb

data/lib/troo/cli/thor_fixes.rb

@@ -1,9 +0,0 @@
-module Troo
-  module CLI
-    class ThorFixes < Thor
-      def self.banner(command, namespace = nil, subcommand = false)
-        "#{basename} #{@package_name} #{command.usage}"
-      end
-    end
-  end
-end