morale 0.1.1 → 0.1.2

data/features/tickets.feature

@@ -4,7 +4,7 @@ Feature: Running the tickets command
   I should be able to run tickets to view, create, update, and delete tickets
 
   @interactive
-  Scenario: Running projects should not require authorization if the account and api key are stored
+  Scenario: Posting a ticket should return the ticket results
     Given a file named "credentials" with:
       """
       spartan
@@ -23,4 +23,52 @@ Feature: Running the tickets command
     And the output should contain:
       """
       Jimmy P.
+      """
+
+  @interactive
+  Scenario: Posting a ticket without a project should ask for a project
+    Given a file named "credentials" with:
+      """
+      spartan
+      12345
+      """
+    When I run `morale ticket "task: This is a new task as: Jimmy"` interactively
+    Then the output should contain:
+      """
+      No project specified.
+      """
+
+  @interactive
+  Scenario: Running tickets should return a list of all active tickets
+    Given a file named "credentials" with:
+      """
+      spartan
+      12345
+      1
+      """
+    When I run `morale tickets` interactively
+    Then the output should contain:
+      """
+      Task
+      """
+    And the output should contain:
+      """
+      Refactor user signup code
+      """
+    And the output should contain:
+      """
+      Create icon for list view
+      """
+
+  @interactive
+  Scenario: Running tickets without a project should ask for a project
+    Given a file named "credentials" with:
+      """
+      spartan
+      12345
+      """
+    When I run `morale tickets` interactively
+    Then the output should contain:
+      """
+      No project specified.
       """

data/lib/morale/account.rb

@@ -30,6 +30,7 @@ module Morale
       end
       
       def project
+        get_credentials
         @credentials[2] if !@credentials.nil? && @credentials.length > 2
       end
       

data/lib/morale/client.rb

@@ -57,6 +57,11 @@ module Morale
     end
     
     def tickets(options={})
+      authorize
+      response = self.class.get("/projects/#{options[:project_id]}/tickets")
+      raise Unauthorized if response.code == 401
+      raise NotFound if response.code == 404
+      response
     end
     
     def ticket(project_id, command)

data/lib/morale/command.rb

@@ -45,13 +45,9 @@ module Morale
       Morale::Commands::Ticket.command command
     end
     
-    desc "test", "Testing the ticket output."
-    def test
-      say "+-----+------+-----------------------------------------------------------------------------------------+----------------+----------+----------------+----------+"
-      say "| id  | type | title                                                                                   | created by     | due date | assigned to    | priority |"
-      say "+-----+------+-----------------------------------------------------------------------------------------+----------------+----------+----------------+----------+"
-      say "| 123 | task | hbsdbjhsdjhfdsjfhsdjfhsdjfhsdjfhsdf                                                     | Jimmy P.       | 08/08/11 | Robert P.      | 1        |"
-      say "+-----+------+-----------------------------------------------------------------------------------------+----------------+----------+----------------+----------+"
+    desc "tickets", "Lists all the projects tickets."
+    def tickets
+      Morale::Commands::Ticket.list
     end
     
     class << self

data/lib/morale/commands/ticket.rb

@@ -1,36 +1,69 @@
 require 'morale/client'
 require 'morale/command'
 require 'morale/authorization'
+require 'morale/flow'
 require 'hirb'
 
 module Morale::Commands
   class Ticket
     class << self
       include Morale::Platform
+      include Morale::Flow
       
       def command(command)
-        ticket = Morale::Command.client.ticket Morale::Account.project, command
-        print ticket
+        ask_for_project
+        print Morale::Command.client.ticket(Morale::Account.project, command) unless Morale::Account.project.nil?
+      end
+      
+      def list
+        ask_for_project
+        print Morale::Command.client.tickets({ :project_id => Morale::Account.project }) unless Morale::Account.project.nil?
       end
       
       private
       
-      def print(ticket)
-        due_date = Date.parse(ticket['due_date']).strftime("%b. %d") unless ticket['due_date'].nil?
-        assigned_to = "#{ticket['assigned_to']['user']['first_name']} #{(ticket['assigned_to']['user']['last_name']).to_s[0,1]}." unless ticket['assigned_to'].nil?
-        
-        say Hirb::Helpers::Table.render [{
-          :id => ticket['identifier'], 
-          :type => ticket['type'],
-          :title => ticket['title'],
-          :created_by => "#{ticket['created_by']['user']['first_name']} #{(ticket['created_by']['user']['last_name']).to_s[0,1]}.",
-          :due_date => due_date,
-          :assigned_to => assigned_to,
-          :priority => ticket['priority']
-          }], 
+      def ask_for_project
+        if Morale::Account.project.nil?
+          say "No project specified."
+          retryable(:retries => 1) do
+            Morale::Commands::Project.list true
+          end
+        end
+      end
+      
+      def print(tickets)
+        say Hirb::Helpers::Table.render fetch(tickets), 
           :fields => [:id, :type, :title, :created_by, :due_date, :assigned_to, :priority],
           :headers => { :created_by => "created by", :due_date => "due date", :assigned_to => "assigned to" }
       end
+      
+      def fetch(tickets)
+        tickets = tickets.kind_of?(Array) ? tickets : Array.new(1, tickets)
+        data = []
+        tickets.each { |t| 
+          ticket = t['task'] unless t['task'].nil?
+          ticket = t['bug'] unless t['bug'].nil?
+          ticket = t if ticket.nil?
+          
+          due_date = ticket['due_date']
+          unless due_date.nil?
+            due_date = Date.parse(due_date) unless due_date.kind_of?(Time)
+            due_date = due_date.strftime("%b. %d")
+          end
+          assigned_to = "#{ticket['assigned_to']['user']['first_name']} #{(ticket['assigned_to']['user']['last_name']).to_s[0,1]}." unless ticket['assigned_to'].nil?
+          
+          data << {
+            :id => ticket['identifier'], 
+            :type => ticket['type'],
+            :title => ticket['title'],
+            :created_by => "#{ticket['created_by']['user']['first_name']} #{(ticket['created_by']['user']['last_name']).to_s[0,1]}.",
+            :due_date => due_date,
+            :assigned_to => assigned_to,
+            :priority => ticket['priority']
+          } 
+        }
+        data
+      end
     end
   end
 end

data/lib/morale.rb

@@ -1,5 +1,5 @@
 module Morale
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end
 
 require 'morale/command'

data/morale.gemspec

@@ -5,8 +5,8 @@ Gem::Specification.new do |s|
   s.rubygems_version = '1.3.5'
 
   s.name              = 'morale'
-  s.version           = '0.1.1'
-  s.date              = '2011-09-30'
+  s.version           = '0.1.2'
+  s.date              = '2011-10-01'
   s.rubyforge_project = 'morale'
 
   s.summary     = "Command line interface to create & manage tickets on Morale."

data/spec/morale/client_spec.rb

@@ -89,4 +89,31 @@ describe Morale::Client do
       lambda { client.ticket('1', "This is a test that should create a new task as: Lester") }.should raise_error(Morale::Client::NotFound)
     end
   end
+  
+  describe "#tickets" do
+    it "should return all active tickets when no options are specified" do
+      stub_request(:get, "http://blah:123456@blah.lvh.me:3000/api/v1/projects/1/tickets").to_return(:body => 
+        [{"ticket" => {"title" => "This is test #1","created_at" => "2011-07-31T21:40:24Z","updated_at" => "2011-07-31T21:40:24Z","project_id" => 1,"id" => 1,"identifier" => "1","type" => "Task","due_date" => "2011-10-15 16:00:00.000000","priority" => nil,"archived" => "f"}},
+         {"ticket" => {"title" => "This is test #2","created_at" => "2011-07-31T21:28:53Z","updated_at" => "2011-07-31T21:28:53Z","project_id" => 1,"id" => 2, "identifier" => "2","type" => "Bug","due_date" => nil,"priority" => "2","archived" => "f"}}].to_json)
+      client = Morale::Client.new('blah', '123456')
+      response = client.tickets({:project_id => 1})
+      response.count.should == 2
+      response[0]["ticket"]["title"].should == "This is test #1"
+      response[1]["ticket"]["title"].should == "This is test #2"
+    end
+    
+    it "should raise unauthorized if a 401 is received" do
+      stub_request(:get, "http://blah:123456@blah.lvh.me:3000/api/v1/projects/1/tickets").
+        to_return(:status => 401)
+      client = Morale::Client.new('blah', '123456')
+      lambda { client.tickets({:project_id => 1}) }.should raise_error(Morale::Client::Unauthorized)
+    end
+    
+    it "should raise notfound if a 404 is received" do
+      stub_request(:get, "http://blah:123456@blah.lvh.me:3000/api/v1/projects/1/tickets").
+        to_return(:status => 404)
+      client = Morale::Client.new('blah', '123456')
+      lambda { client.tickets({ :project_id => 1 }) }.should raise_error(Morale::Client::NotFound)
+    end
+  end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - Brilliant Fantastic
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-09-30 00:00:00 -04:00
+date: 2011-10-01 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency