ticketmaster-trac 0.1.2 → 0.1.3

data/README.md

@@ -5,24 +5,24 @@ This is a provider for [ticketmaster](http://ticketrb.com). It provides interope
 # Usage and Examples
 
 First we have to instantiate a new ticketmaster instance, your trac installation should have api access enable:
-    trac = TicketMaster.new(:trac, {:username=> 'foo', :password => "bar", :url => "http://tracserver/project"})
+    trac = TicketMaster.new(:trac, {:username=> 'foo', :password => "bar", :url => "http://tracserver/username/project/trac"}) 
 
 If you do not pass in the url, username and password, you won't get any information.
 
-== Finding Projects(Repositories)
+## Finding Projects(Repositories)
 
 Right now Trac doesn't support project handling, that's why you will use a project holder which consist of your username and url for the Trac instance.
 	
-== Finding Tickets(Issues)
+## Finding Tickets(Issues)
 
     tickets = project.tickets # All tickets
     ticket = project.tickets(:status => '!closed') # All non closed tickets, you can specified any type of attributes field based on Trac scheme. ex. :component => 'Web' 
 
-== Open Tickets
+## Open Tickets
     
 	ticket = project.ticket!({:summary => "New ticket", :description=> "Body for the very new ticket"}, attributes) # summary and description are mandatory fields the attributes hash will contain all the optional info that you need for your ticket, have to use the fields provided by Trac api.
 
-== Finding comments
+## Finding comments
       
         comments = project.tickets.first.comments # All the tickets facility for searchs are available for comments as well
 

data/VERSION

@@ -1 +1 @@
-0.1.2
+0.1.3

data/lib/provider/comment.rb

@@ -25,6 +25,22 @@ module TicketMaster::Provider
           super(hash)
         end
       end
+
+      def created_at
+        begin
+          Time.parse(self[:created_at])
+        rescue
+          self[:created_at]
+        end
+      end
+
+      def updated_at
+        begin
+          Time.parse(self[:updated_at])
+        rescue
+          self[:updated_at]
+        end
+      end
     end
   end
 end

data/lib/provider/project.rb

@@ -13,7 +13,8 @@ module TicketMaster::Provider
           @system_data = {:client => object}
           unless object.is_a? Hash
             hash = {:repository => object.url,
-                    :user => object.username}
+                    :user => object.username,
+                    :name => object.name}
             
           else
             hash = object
@@ -36,7 +37,7 @@ module TicketMaster::Provider
       def self.find(*options)
         mode = options.first
         if mode.is_a? String
-          self.new({:url => API.url, :username => "#{API.username}-project"})
+          self.new({:url => API.url, :username => API.username, :name=> "#{API.username}-project"})
         end
       end
 
@@ -44,7 +45,7 @@ module TicketMaster::Provider
         unless options.empty?
           options = options.first
           if options.is_a? Hash
-            TicketMaster::Provider::Trac::Ticket.find_by_id API.api.tickets.query(options).first
+            TicketMaster::Provider::Trac::Ticket.find_by_id(API.api.tickets.query(options).first, self[:name])
           end
         else
           TicketMaster::Provider::Trac::Ticket
@@ -69,7 +70,9 @@ module TicketMaster::Provider
 
       private
       def collect_tickets(tickets)
-        tickets.collect { |ticket_id| TicketMaster::Provider::Trac::Ticket.find_by_id ticket_id}
+        tickets.collect do |ticket_id| 
+          TicketMaster::Provider::Trac::Ticket.find_by_id(ticket_id, self[:name]) 
+        end
       end
 
     end

data/lib/provider/ticket.rb

@@ -9,7 +9,9 @@ module TicketMaster::Provider
       API = TracAPI
       def initialize(*object)
         if object.first
-          object = object.first
+          args = object
+          object = args.shift
+          project_id = args.shift
           unless object.is_a? Hash
             @system_data = {:client => object}
             hash = {:id => object.id,
@@ -23,6 +25,10 @@ module TicketMaster::Provider
               :owner => object.owner,
               :cc => object.cc,
               :summary => object.summary,
+              :assignee => object.owner,
+              :requestor => object.reporter,
+              :title => object.summary,
+              :project_id => project_id,
               :description => object.description,
               :keywords => object.keywords,
               :created_at => object.created_at,
@@ -34,8 +40,26 @@ module TicketMaster::Provider
         end
       end
 
-      def self.find_by_id(id)
-        self.new API.api.tickets.get id
+      def created_at
+        begin
+          normalize_datetime(self[:created_at]) 
+        rescue
+          self[:created_at]
+        end
+      end
+
+      def updated_at
+        begin
+          normalize_datetime(self[:updated_at])
+        rescue
+          self[:updated_at]
+        end
+      end
+
+      def self.find_by_id(id, project_id)
+        retryable(:tries => 5) do 
+          self.new API.api.tickets.get(id), project_id
+        end
       end
 
       def self.create(*options)
@@ -79,9 +103,28 @@ module TicketMaster::Provider
       end
 
       def comment!
-        warn "Trac doesn't support comments"
+        warn "Trac doesn't support creation of comments"
         [] 
       end
+
+      private
+      def normalize_datetime(datetime)
+        Time.mktime(datetime.year, datetime.month, datetime.day, datetime.hour, datetime.min, datetime.sec)
+      end
+
+      # Extracted this code from facet http://api.mackframework.com/mack-facets/classes/Kernel.html
+      def self.retryable(options = {}, &block)
+        opts = { :tries => 1, :on => Exception }.merge(options)
+
+        retry_exception, retries = opts[:on], opts[:tries]
+
+        begin
+          return yield
+        rescue retry_exception
+          retry if (retries -= 1) > 0
+        end
+        yield
+      end
     end
   end
 end

data/lib/provider/trac.rb

@@ -17,7 +17,7 @@ module TicketMaster::Provider
 
     # declare needed overloaded methods here
     def projects(*options)
-       [Project.new({:url => @authentication.url, :username => @authentication.username})]  
+       [Project.new({:url => @authentication.url, :username => @authentication.username, :name => "#{@authentication.username}-project"})]  
     end
 
     def project(*options)

data/lib/trac/trac.rb

@@ -29,6 +29,7 @@ class CommentUtil
 
   def initialize(ticket_id)
     @doc = Nokogiri::HTML(open("#{TracAPI.url}/ticket/#{ticket_id}", :http_basic_authentication=>[TracAPI.username, TracAPI.password]))
+    @ticket_id = ticket_id
   end
 
   def comments

data/spec/comments_spec.rb

@@ -4,11 +4,18 @@ describe "Ticketmaster::Provider::Trac::Comment" do
   before(:each) do 
     @ticketmaster = TicketMaster.new(:trac, {:username => 'george.rafael@gmail.com', :password => '123456', :url => 'http://pl3.projectlocker.com/cored/testrepo/trac'})
     @project = @ticketmaster.projects.first
+    @project.stub!(:tickets).and_return([TicketMaster::Provider::Trac::Ticket.new])
     @ticket = @project.tickets.first
     @klass = TicketMaster::Provider::Trac::Comment
+    @comment_1 = @klass.new(:id => 1)
+    @comment_2 = @klass.new(:id => 2)
+    @comments = [@comment_1, @comment_2]
+    @ticket.stub!(:comments).and_return(@comments)
+    @ticket.stub!(:comment).and_return(@comment_2)
   end
 
   it "should load all comments from a ticket" do 
+    
     @ticket.comments.should be_an_instance_of(Array)
     @ticket.comments.first.should be_an_instance_of(@klass)
   end
@@ -40,9 +47,9 @@ describe "Ticketmaster::Provider::Trac::Comment" do
   end
 
   it "should be able to load a comment using attributes" do
-    @comment = @ticket.comment(:ticket_id => 1)
+    @comment = @ticket.comment(:ticket_id => 2)
     @comment.should be_an_instance_of(@klass)
-    @comment.id.should == 1
+    @comment.id.should == 2
   end
 
 end

data/spec/tickets_spec.rb

@@ -9,6 +9,11 @@ describe "Ticketmaster::Provider::Trac::Ticket" do
     @ticketmaster = TicketMaster.new(:trac, {:url => 'http://pl3.projectlocker.com/cored/testrepo/trac', :username => 'george.rafael@gmail.com', :password => '123456'})
     @project = @ticketmaster.project(@project_id)
     @klass = TicketMaster::Provider::Trac::Ticket
+    @ticket_1 = @klass.new(:summary => 'test', :status => 'open')
+    @tickets = [@ticket_1]
+    @project.stub!(:tickets).and_return(@tickets)
+    @project.stub!(:ticket).and_return(@ticket_1)
+    @project.stub!(:ticket!).and_return(@klass.new)
   end
 
   it "should be able to load all tickets" do 
@@ -31,6 +36,7 @@ describe "Ticketmaster::Provider::Trac::Ticket" do
   end
 
   it "should return the ticket class" do
+    @project.stub!(:ticket).and_return(@klass)
     @project.ticket.should == @klass
   end
 

data/ticketmaster-trac.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{ticketmaster-trac}
-  s.version = "0.1.2"
+  s.version = "0.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Rafael George"]
-  s.date = %q{2011-01-06}
+  s.date = %q{2011-01-07}
   s.description = %q{Allows ticketmaster to interact with Your System.}
   s.email = %q{george.rafael@gmail.com}
   s.extra_rdoc_files = [

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ticketmaster-trac
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: false
   segments: 
   - 0
   - 1
-  - 2
-  version: 0.1.2
+  - 3
+  version: 0.1.3
 platform: ruby
 authors: 
 - Rafael George
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-06 00:00:00 -04:00
+date: 2011-01-07 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency