ticketmaster-pivotal 0.6.3 → 0.7.0

data/Gemfile

@@ -3,7 +3,7 @@ source "http://rubygems.org"
 # Example:
 #   gem "activesupport", ">= 2.3.5"
 
-gem "ticketmaster", "0.6.10"
+gem "ticketmaster", "~> 0.7"
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
 group :development do

data/Gemfile.lock

@@ -1,13 +1,13 @@
 GEM
   remote: http://rubygems.org/
   specs:
-    activemodel (3.2.2)
-      activesupport (= 3.2.2)
+    activemodel (3.2.3)
+      activesupport (= 3.2.3)
       builder (~> 3.0.0)
-    activeresource (3.2.2)
-      activemodel (= 3.2.2)
-      activesupport (= 3.2.2)
-    activesupport (3.2.2)
+    activeresource (3.2.3)
+      activemodel (= 3.2.3)
+      activesupport (= 3.2.3)
+    activesupport (3.2.3)
       i18n (~> 0.6)
       multi_json (~> 1.0)
     builder (3.0.0)
@@ -34,7 +34,7 @@ GEM
       multi_json (~> 1.0.3)
       simplecov-html (~> 0.5.3)
     simplecov-html (0.5.3)
-    ticketmaster (0.6.10)
+    ticketmaster (0.7.0)
       activeresource (~> 3.0)
       activesupport (~> 3.0)
       hashie (~> 1.2)
@@ -48,4 +48,4 @@ DEPENDENCIES
   rcov (~> 1.0)
   rspec (~> 2.8)
   simplecov (~> 0.5)
-  ticketmaster (= 0.6.10)
+  ticketmaster (~> 0.7)

data/VERSION

@@ -1 +1 @@
-0.6.3
+0.7.0

data/lib/provider/project.rb

@@ -35,15 +35,8 @@ module TicketMaster::Provider
       end
 
       def ticket!(*options)
-        if options.first.is_a?(Hash)
-          options[0].merge!(:project_id => id)
-          title = options[0].delete('title') || options[0].delete(:title) || options[0].delete(:summary) || options[0].delete('summary')
-          description = options[0].delete('body') || options[0].delete(:body)
-          options[0][:name] = title
-          options[0][:description] = description
-          warn("Pivotal Tracker requires a title or name for the story") if options[0][:name].blank? and options[0]['name'].blank?
-        end
-        provider_parent(self.class)::Ticket.create(*options)
+        options.first.merge!(:project_id => self.id)
+        Ticket.create(options.first)
       end
 
       # copy from

data/lib/provider/ticket.rb

@@ -18,16 +18,6 @@ module TicketMaster::Provider
       attr_accessor :prefix_options
       API = PivotalAPI::Story
 
-      def self.find_by_attributes(project_id, attributes = {})
-        date_to_search = attributes[:updated_at] || attributes[:created_at]
-        tickets = []
-        unless date_to_search.nil?
-          tickets = search_by_datefields(project_id, date_to_search)
-        else
-          tickets += API.find(:all, :params => {:project_id => project_id, :filter => filter(attributes)}).map { |xticket| self.new xticket }
-        end
-        tickets.flatten
-      end
 
       # The saver
       def save(*options)
@@ -58,8 +48,24 @@ module TicketMaster::Provider
         self.name=title
       end
 
+      def status
+        self.current_state
+      end
+
+      def priority
+        self.estimate
+      end
+
+      def resolution
+        self.current_state
+      end
+
+      def assignee
+        self.owned_by
+      end
+
       def comment!(*options)
-      	Comment.create(self.project_id, self.id, options.first)
+        Comment.create(self.project_id, self.id, options.first)
       end
       # The closer
       def close(resolution = 'resolved')
@@ -69,23 +75,50 @@ module TicketMaster::Provider
         ticket.save
       end
 
-      private
-      def self.search_by_datefields(project_id, date_to_search)
-        date_to_search = date_to_search.strftime("%Y/%m/%d")
-        tickets = []
-        PivotalAPI::Activity.find(:all, :params => {:project_id => project_id, :occurred_since_date => date_to_search}).each do |activity|
-          tickets = activity.stories.map { |xstory| self.new xstory }
+      class << self
+
+        def find_by_attributes(project_id, attributes = {})
+          date_to_search = attributes[:updated_at] || attributes[:created_at]
+          tickets = []
+          unless date_to_search.nil?
+            tickets = search_by_datefields(project_id, date_to_search)
+          else
+            tickets += API.find(:all, :params => {:project_id => project_id, :filter => filter(attributes)}).map { |xticket| self.new xticket }
+          end
+          tickets.flatten
         end
-        tickets
-      end
 
-      def self.filter(attributes = {})
-        filter = ""
-        attributes.each_pair do |key, value|
-          filter << "#{key}:#{value} "
+        def filter(attributes = {})
+          filter = ""
+          attributes.each_pair do |key, value|
+            filter << "#{key}:#{value} "
+          end
+          filter.strip!
+        end
+
+        def create(options)
+          super translate options, {:title => :name, 
+                                    :requestor => :requested_by,
+                                    :status => :current_state, 
+                                    :estimate => :priority,
+                                    :assignee => :owned_by}
+        end
+
+        private
+        def search_by_datefields(project_id, date_to_search)
+          date_to_search = date_to_search.strftime("%Y/%m/%d")
+          tickets = []
+          PivotalAPI::Activity.find(:all, :params => {:project_id => project_id, :occurred_since_date => date_to_search}).each do |activity|
+            tickets = activity.stories.map { |xstory| self.new xstory }
+          end
+          tickets
+        end
+
+        def translate(hash, mapping)
+          Hash[hash.map { |k, v| [mapping[k] ||= k, v]}]
         end
-        filter.strip!
       end
     end
+
   end
 end

data/spec/fixtures/stories/4056827.xml

@@ -9,6 +9,7 @@
   <description>Hi hi. Description!</description>
   <name>Hello</name>
   <requested_by>Hong Quach</requested_by>
+  <owned_by>Hong Quach</owned_by>
   <created_at type="datetime">2010/06/26 22:31:08 UTC</created_at>
   <updated_at type="datetime">2010/06/26 22:56:02 UTC</updated_at>
   <notes type="array">

data/spec/tickets_spec.rb

@@ -18,77 +18,97 @@ describe "Ticketmaster::Provider::Pivotal::Ticket" do
     @project_id = 93790
     @ticket_id = 4056827
   end
-  
+
   before(:each) do
     @ticketmaster = TicketMaster.new(:pivotal, :token => '000000')
     @project = @ticketmaster.project(@project_id)
     @klass = TicketMaster::Provider::Pivotal::Ticket
   end
-  
+
   it "should be able to load all tickets" do
     @project.tickets.should be_an_instance_of(Array)
     @project.tickets.first.should be_an_instance_of(@klass)
   end
-  
+
   it "should be able to load all tickets based on an array of ids" do
     @tickets = @project.tickets([@ticket_id])
     @tickets.should be_an_instance_of(Array)
     @tickets.first.should be_an_instance_of(@klass)
     @tickets.first.id.should == @ticket_id
   end
-  
+
   it "should be able to load all tickets based on attributes" do
     @tickets = @project.tickets(:id => @ticket_id)
     @tickets.should be_an_instance_of(Array)
     @tickets.first.should be_an_instance_of(@klass)
     @tickets.first.id.should == @ticket_id
   end
-  
+
   it "should return the ticket class" do
     @project.ticket.should == @klass
   end
-  
+
   it "should be able to load a single ticket" do
     @ticket = @project.ticket(@ticket_id)
     @ticket.should be_an_instance_of(@klass)
     @ticket.id.should == @ticket_id
   end
-  
+
   it "should be able to load a single ticket based on attributes" do
     @ticket = @project.ticket(:id => @ticket_id)
     @ticket.should be_an_instance_of(@klass)
     @ticket.id.should == @ticket_id
   end
-  
+
   it "should be able to update and save a ticket" do
     @ticket = @project.ticket(@ticket_id)
     #@ticket.save.should == nil
     @ticket.description = 'hello'
     @ticket.save.should == true
   end
-  
+
   it "should be able to create a ticket" do
     @ticket = @project.ticket!(:title => 'Ticket #12', :description => 'Body')
     @ticket.should be_an_instance_of(@klass)
   end
 
-  it "should be able to load all tickets based on attributes using updated_at field" do 
+  it "should be able to load all tickets based on attributes using updated_at field" do
     @ticket = @project.ticket(@ticket_id)
     tickets = @project.tickets(:updated_at => @ticket.updated_at)
     tickets.should be_an_instance_of(Array)
     tickets.first.should be_an_instance_of(@klass)
   end
 
-  it "shoule be able to load all tickets based on attributes using created_at field" do 
+  it "shoule be able to load all tickets based on attributes using created_at field" do
     @ticket = @project.ticket(@ticket_id)
     tickets = @project.tickets(:created_at => @ticket.created_at)
     tickets.should be_an_instance_of(Array)
     tickets.first.should be_an_instance_of(@klass)
   end
 
-  it "should return the requested_by field" do 
+  it "should return the requested_by field" do
     @ticket = @project.ticket(@ticket_id)
     @ticket.requestor.should == 'Hong Quach'
   end
-  
+
+  it "should be able to update a ticket" do
+    @ticket = @project.ticket(@ticket_id)
+    @ticket.title = "Hello World"
+    @ticket.save.should be_true
+  end
+
+  it "should have all contract fields for tickets" do 
+    @ticket = @project.ticket(@ticket_id)
+    @ticket.title.should_not be_nil
+    @ticket.description.should_not be_nil
+    @ticket.status.should_not be_nil
+    @ticket.priority.should_not be_nil
+    @ticket.resolution.should_not be_nil
+    @ticket.created_at.should_not be_nil
+    @ticket.updated_at.should_not be_nil
+    @ticket.assignee.should_not be_nil
+    @ticket.requestor.should_not be_nil
+    @ticket.project_id.should_not be_nil
+  end
+
 end

data/ticketmaster-pivotal.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "ticketmaster-pivotal"
-  s.version = "0.6.3"
+  s.version = "0.7.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["HybridGroup"]
-  s.date = "2012-03-27"
+  s.date = "2012-04-11"
   s.description = "This is a ticketmaster provider for interacting with Pivotal Tracker ."
   s.email = "hong.quach@abigfisch.com"
   s.extra_rdoc_files = [
@@ -55,14 +55,14 @@ Gem::Specification.new do |s|
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<ticketmaster>, ["= 0.6.10"])
+      s.add_runtime_dependency(%q<ticketmaster>, ["~> 0.7"])
       s.add_development_dependency(%q<rspec>, ["~> 2.8"])
       s.add_development_dependency(%q<bundler>, ["~> 1.1"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.6"])
       s.add_development_dependency(%q<simplecov>, ["~> 0.5"])
       s.add_development_dependency(%q<rcov>, ["~> 1.0"])
     else
-      s.add_dependency(%q<ticketmaster>, ["= 0.6.10"])
+      s.add_dependency(%q<ticketmaster>, ["~> 0.7"])
       s.add_dependency(%q<rspec>, ["~> 2.8"])
       s.add_dependency(%q<bundler>, ["~> 1.1"])
       s.add_dependency(%q<jeweler>, ["~> 1.6"])
@@ -70,7 +70,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<rcov>, ["~> 1.0"])
     end
   else
-    s.add_dependency(%q<ticketmaster>, ["= 0.6.10"])
+    s.add_dependency(%q<ticketmaster>, ["~> 0.7"])
     s.add_dependency(%q<rspec>, ["~> 2.8"])
     s.add_dependency(%q<bundler>, ["~> 1.1"])
     s.add_dependency(%q<jeweler>, ["~> 1.6"])

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ticketmaster-pivotal
 version: !ruby/object:Gem::Version 
-  hash: 1
+  hash: 3
   prerelease: 
   segments: 
   - 0
-  - 6
-  - 3
-  version: 0.6.3
+  - 7
+  - 0
+  version: 0.7.0
 platform: ruby
 authors: 
 - HybridGroup
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-03-27 00:00:00 Z
+date: 2012-04-11 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: ticketmaster
@@ -24,14 +24,13 @@ dependencies:
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ~>
       - !ruby/object:Gem::Version 
-        hash: 19
+        hash: 5
         segments: 
         - 0
-        - 6
-        - 10
-        version: 0.6.10
+        - 7
+        version: "0.7"
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   name: rspec