troo 0.0.8 → 0.0.9

data/features/show/boards.feature

@@ -0,0 +1,19 @@
+Feature: Showing all boards
+
+  @show
+  Scenario: When boards exist
+    Given a board exists
+    When I run `troo show boards`
+    Then the output should contain:
+      """
+      (1) My Test Board
+          No lists were found.
+      """
+
+  @show
+  Scenario: When no boards exist
+    When I run `troo show boards`
+    Then the output should contain:
+      """
+      Boards cannot be found.
+      """

data/features/show/card.feature

@@ -1,16 +1,12 @@
 Feature: Showing a card
-  Background:
-    Given a board exists
-    And a list exists
-    And a member exists
 
   @show
   Scenario: Showing a card with ID
     Given a card exists
-    When I run `troo show card 1`
+    When I run `troo show card 200`
     Then the output should contain:
       """
-       (67) My Test Card
+      (1) My Test Card
 
       Description:
       some description
@@ -22,16 +18,19 @@ Feature: Showing a card
       @gavinlaking1
 
       Metadata:
-         Board:    (1) My Test Board
-          List:    (1) My Test List
+         Board: (1) My Test Board
+          List: (1) My Test List
        Updated: Tue, Dec 17 at 21:48
       """
 
   @show
   Scenario: Cannot show card; not found
-  Given the Trello API is stubbed with "fetch_card_by_id"
-    When I run `troo show card 69`
-    Then the output should contain "Card cannot be found."
+    Given the Trello API is stubbed with "400_card_by_id"
+    When I run `troo show card 400`
+    Then the output should contain:
+      """
+      Card cannot be found.
+      """
 
   @show
   Scenario: Showing the default card
@@ -39,7 +38,7 @@ Feature: Showing a card
     When I run `troo show card`
     Then the output should contain:
       """
-      * (67) My Default Card
+      (1) My Default Card *
 
       Description:
       some description
@@ -51,12 +50,84 @@ Feature: Showing a card
       @gavinlaking1
 
       Metadata:
-         Board:    (1) My Test Board
-          List:    (1) My Test List
+         Board: (1) My Test Board
+          List: (1) My Test List
        Updated: Tue, Dec 17 at 21:48
       """
 
   @show
   Scenario: Cannot show; no default card
     When I run `troo show card`
-    Then the output should contain "set a default card first"
+    Then the output should contain:
+    """
+    set a default card first
+    """
+
+  @show
+  Scenario: Showing a card with comments
+    Given a card exists
+    Given 2 comments exist
+    When I run `troo show card 200`
+    Then the output should contain:
+      """
+      (1) My Test Card
+
+      Description:
+      some description
+
+      Comments:
+      @gavinlaking1:
+        My Test Comment
+          (Tue, Dec 17 at 22:01)
+
+      @gavinlaking1:
+        My Test Comment
+          (Tue, Dec 17 at 22:01)
+
+
+
+      Members:
+      @gavinlaking1
+
+      Metadata:
+         Board: (1) My Test Board
+          List: (1) My Test List
+       Updated: Tue, Dec 17 at 21:48
+      """
+
+  @show
+  Scenario: Showing a card with many comments
+    Given a card exists
+    Given 5 comments exist
+    When I run `troo show card 200`
+    Then the output should contain:
+      """
+      (1) My Test Card
+
+      Description:
+      some description
+
+      Comments:
+      @gavinlaking1:
+        My Test Comment
+          (Tue, Dec 17 at 22:01)
+
+      @gavinlaking1:
+        My Test Comment
+          (Tue, Dec 17 at 22:01)
+
+      @gavinlaking1:
+        My Test Comment
+          (Tue, Dec 17 at 22:01)
+
+
+      (There are more comments, use: 'troo show comments 67' to view  all comments.)
+
+      Members:
+      @gavinlaking1
+
+      Metadata:
+         Board: (1) My Test Board
+          List: (1) My Test List
+       Updated: Tue, Dec 17 at 21:48
+      """

data/features/show/comment.feature

@@ -1,23 +1,24 @@
 Feature: Showing comments
-  Background:
-    Given a card exists
-    And a member exists
 
   @show
   Scenario: Showing comments for a card with ID
     Given a comment exists
-    When I run `troo show comments 1`
-    Then the output should contain "My Test Comment"
+    When I run `troo show comments 200`
+    Then the output should contain:
+      """
+      My Test Comment
+      """
 
   @show
   Scenario: Showing comments for a card with ID; no comments
-    When I run `troo show comments 1`
+    Given a card exists
+    When I run `troo show comments 200`
     Then the output should contain "No comments"
 
   @show
   Scenario: Cannot show comments; card not found
-    Given the Trello API is stubbed with "fetch_card_by_id"
-    When I run `troo show comments 69`
+    Given the Trello API is stubbed with "400_card_by_id"
+    When I run `troo show comments 400`
     Then the output should contain "Card cannot be found."
 
   @show
@@ -27,10 +28,11 @@ Feature: Showing comments
     When I run `troo show comments`
     Then the output should contain:
       """
-       * (67) My Default Card
-          gavinlaking1:
-          My Test Comment
-              (Tue, Dec 17 at 22:01)
+      (1) My Default Card *
+
+      @gavinlaking1:
+        My Test Comment
+          (Tue, Dec 17 at 22:01)
       """
 
   @show

data/features/show/list.feature

@@ -1,24 +1,25 @@
 Feature: Showing a list
-  Background:
-    Given a board exists
-    And a card exists
 
   @show
   Scenario: Showing a list with ID
     Given a list exists
-    When I run `troo show list 1`
+    When I run `troo show list 200`
     Then the output should contain:
       """
-       (1) My Test Board
-           (1) My Test List
-           (67) My Test Card
+      (1) My Test Board
+
+          (1) My Test List
+              No cards were found.
       """
 
   @show
   Scenario: Cannot show list; not found
-    Given the Trello API is stubbed with "fetch_list_by_id"
-    When I run `troo show list 69`
-    Then the output should contain "List cannot be found."
+    Given the Trello API is stubbed with "400_list_by_id"
+    When I run `troo show list 400`
+    Then the output should contain:
+      """
+      List cannot be found.
+      """
 
   @show
   Scenario: Showing the default list
@@ -26,12 +27,16 @@ Feature: Showing a list
     When I run `troo show list`
     Then the output should contain:
       """
-       (1) My Test Board
-          * (1) My Default List
-           (67) My Test Card
+      (1) My Test Board
+
+          (1) My Default List *
+              No cards were found.
       """
 
   @show
   Scenario: Cannot show; no default list
     When I run `troo show list`
-    Then the output should contain "set a default list first"
+    Then the output should contain:
+    """
+    set a default list first
+    """

data/features/status.feature

@@ -13,34 +13,48 @@ Feature: Showing the current status
 
   @status
   Scenario: When there is local data; all defaults set
-    Given a default board exists
-    And a default list exists
-    And a default card exists
+    Given local data exists, all defaults set
     When I run `troo status`
     Then the output should contain:
       """
       Status:
         Boards: 1 board found.
-                * (1) My Default Board
+                (1) My Default Board *
         Lists:  1 list found.
-                * (1) My Default List
+                (1) My Default List *
         Cards:  1 card found.
-                * (67) My Default Card
+                (1) My Default Card *
       """
 
   @status
   Scenario: When there is local data; some defaults set
-    Given a default board exists
-    And a board exists
-    And a default list exists
-    And a card exists
+    Given local data exists
     When I run `troo status`
     Then the output should contain:
       """
       Status:
         Boards: 2 boards found.
-                * (1) My Default Board
+                (1) My Default Board *
         Lists:  1 list found.
-                * (1) My Default List
-        Cards:  No default card set.
+                (1) My Default List *
+        Cards:  No default card set. (1)
       """
+
+  @status
+  Scenario: Reports when local data was last updated
+    When I run `troo status`
+    Then the output should contain:
+      """
+      Last refreshed:
+      Unknown. Run `troo refresh all`.
+      """
+
+  @status
+  Scenario: Reports when local data was last updated
+    Given local data exists
+    When I run `troo status`
+    Then the output should not contain:
+      """
+      Last refreshed:
+      Unknown. Run `troo refresh all`.
+      """

data/features/step_definitions/fabrication_steps.rb

@@ -2,33 +2,68 @@ require 'fabrication'
 require_relative '../../test/support/fabrication.rb'
 
 Given(/^a board exists$/) do
-  Fabricate(:board)
+  Fabricate(:board, external_id: 200)
 end
 
 Given(/^a list exists$/) do
-  Fabricate(:list)
+  Fabricate(:board)
+  Fabricate(:list, external_id: 200)
 end
 
 Given(/^a card exists$/) do
-  Fabricate(:card)
+  Fabricate(:board)
+  Fabricate(:list)
+  Fabricate(:member)
+  Fabricate(:card, external_id: 200)
 end
 
 Given(/^a comment exists$/) do
-  Fabricate(:comment)
+  Fabricate(:card, external_id: 200)
+  Fabricate(:member)
+  Fabricate(:comment, external_card_id: 200)
 end
 
-Given(/^a member exists$/) do
+Given(/^(\d+) comments exist$/) do |count|
+  Fabricate(:card, external_id: 200)
   Fabricate(:member)
+  count.to_i.times do
+    Fabricate(:comment, external_card_id: 200)
+  end
 end
 
 Given(/^a default board exists$/) do
-  Fabricate(:board, default: true, name: 'My Default Board')
+  Fabricate(:board, default: true,
+                    name: 'My Default Board',
+                    external_id: 200)
 end
 
 Given(/^a default list exists$/) do
-  Fabricate(:list, default: true, name: 'My Default List')
+  Fabricate(:board)
+  Fabricate(:list, default: true,
+                   name: 'My Default List',
+                   external_id: 200)
 end
 
 Given(/^a default card exists$/) do
-  Fabricate(:card, default: true, name: 'My Default Card')
+  Fabricate(:board)
+  Fabricate(:list)
+  Fabricate(:member)
+  Fabricate(:card, default: true,
+                   name:    'My Default Card',
+                   external_id: 200)
+end
+
+Given(/^local data exists, all defaults set$/) do
+  Fabricate(:board, default: true, name: 'My Default Board')
+  Fabricate(:list,  default: true, name: 'My Default List')
+  Fabricate(:card,  default: true, name: 'My Default Card')
+  Troo::Refresh.completed!
+end
+
+Given(/^local data exists$/) do
+  Fabricate(:board, default: true, name: 'My Default Board')
+  Fabricate(:board)
+  Fabricate(:list, default: true, name: 'My Default List')
+  Fabricate(:card, external_id: 200)
+  Troo::Refresh.completed!
 end

data/features/step_definitions/troo_steps.rb

@@ -1,5 +1,5 @@
 Given(/^the Trello API is stubbed with "(.*?)"$/) do |stub|
-  VCR.insert_cassette(stub)
+  VCR.insert_cassette(stub, allow_playback_repeats: true)
 end
 
 Then(/^the output should be the version number of troo$/) do
@@ -7,7 +7,8 @@ Then(/^the output should be the version number of troo$/) do
 end
 
 Before do
-  Ohm.connect(db: Troo.config.test_db)
+  config = Troo::Configuration.load('test/support/.trooconf', :test)
+  Ohm.connect(db: config.database)
 end
 
 After do

data/features/support/env.rb

@@ -5,6 +5,7 @@ require 'aruba/in_process'
 require 'ohm'
 require 'vcr'
 require 'webmock/cucumber'
+require 'mocha/api'
 
 SimpleCov.start do
   add_filter   '/test/'

data/features/troo.feature

@@ -17,15 +17,3 @@ Feature: Troo Basics
       """
       Could not find command "unknown".
       """
-
-  @pending @authentication
-  Scenario: Not authenticated with Trello
-    Given the Trello API is stubbed with "unauthenticated"
-
-  @pending @connectivity
-  Scenario: Trello not responding
-    Given the Trello API is stubbed with "trello_failure"
-
-  @pending @connectivity
-  Scenario: No network connection
-

data/lib/troo.rb

@@ -1,32 +1,36 @@
+$LOAD_PATH << File.expand_path(File.dirname(__FILE__) + '/../lib')
+
 require_relative 'troo/troo'
 require_relative 'troo/version'
 
 module Troo
-  class InvalidAccessToken < StandardError; end
-
-  def self.config
-    @config ||= OpenStruct.new(YAML.load_file(Dir.home + '/.trooconf'))
-  rescue Errno::ENOENT
-    warn 'No configuration file found...'
-    src = File.dirname(__FILE__) + '/../configuration.yml.example'
-    dst = Dir.home + '/.trooconf'
-    FileUtils.cp(src, dst)
-    warn "New configuration file created at '#{Dir.home}/.trooconf'"
-    exit!
+  InvalidAccessToken = Class.new(StandardError)
+  EndpointNotFound   = Class.new(StandardError)
+
+  def self.configuration(file = Dir.home + '/.trooconf', env = :default)
+    @configuration ||= Troo::Configuration.load(file, env)
   end
+  configuration
 
-  def self.logger
-    @logger ||= Logger.new('logs/troo.log')
+  def self.endpoints(version = :version_1)
+    @endpoints ||= Troo::API::Endpoints
+      .load(File.dirname(__FILE__) + '/../config/trello_api.yml', version)
   end
+  endpoints
 
-  # Trello.logger = Logger.new("logs/trello.log")
-  Trello.configure do |trello|
-    trello.consumer_key       = config.key
-    trello.consumer_secret    = config.secret
-    trello.oauth_token        = config.oauth_token
-    trello.oauth_token_secret = config.oauth_token_secret
+  def self.logger
+    @logger ||= Logger
+      .new(File.dirname(__FILE__) + '/../logs/troo.log').tap do |log|
+      log.formatter = proc do |mode, time, prog, msg|
+        "#{time.iso8601} #{mode}:\n#{msg}\n\n"
+      end
+    end
   end
 
+  # RestClient.log = File.dirname(__FILE__) + '/../logs/restclient.log'
+
+  Ohm.connect(db: Troo.configuration.database)
+
   class Launcher
     def initialize(argv, stdin = STDIN,
                          stdout = STDOUT,
@@ -41,19 +45,18 @@ module Troo
 
     def execute!
       $stdin, $stdout, $stderr = @stdin, @stdout, @stderr
+      pad { Troo::CLI::Main.start(@argv) }
+      @kernel.exit(0)
+    ensure
+      $stdin, $stdout, $stderr = STDIN, STDOUT, STDERR
+    end
 
+    private
+
+    def pad
       puts
-      Troo::CLI::Main.start(@argv)
+      yield
       puts
-
-      @kernel.exit(0)
-    rescue Troo::InvalidAccessToken
-      @stderr.puts 'Your Trello access credentials have expired, ' \
-                   ' please renew and try again.'
-      @kernel.exit(1)
-    rescue SocketError
-      @stderr.puts 'Cannot continue, no network connection.'
-      @kernel.exit(1)
     end
   end
 end