ticketmaster-bugherd 0.0.1 → 0.0.2

data/Gemfile

@@ -4,8 +4,7 @@ source "http://rubygems.org"
 #   gem "activesupport", ">= 2.3.5"
 
 gem "ticketmaster", "~> 0.6.0"
-gem "activesupport", "~> 3.0.0"
-gem "activeresource", "~> 3.0.0"
+gem "bugherd-api", "~> 0.0.3"
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
 group :development do

data/Gemfile.lock

@@ -9,6 +9,9 @@ GEM
       activemodel (= 3.0.9)
       activesupport (= 3.0.9)
     activesupport (3.0.9)
+    bugherd-api (0.0.3)
+      activeresource
+      activesupport
     builder (2.1.2)
     diff-lcs (1.1.2)
     git (1.2.5)
@@ -42,8 +45,7 @@ PLATFORMS
   x86-mingw32
 
 DEPENDENCIES
-  activeresource (~> 3.0.0)
-  activesupport (~> 3.0.0)
+  bugherd-api (~> 0.0.3)
   bundler (~> 1.0.0)
   jeweler (~> 1.6.4)
   rcov

data/README.rdoc

@@ -1,6 +1,31 @@
 = ticketmaster-bugherd
 
-Description goes here.
+This is a provider for ticketmaster[http://ticketrb.com]. It provides interoperability with Bugherd[http://www.bugherd.com] through the ticketmaster gem.
+
+== Usage and Examples
+
+first we have to instance a new ticketmaster instance: 
+ bugherd = TicketMaster.new(:bugherd, :email => 'user@email.com', :password => 'password')
+
+=== Retreiving all projects
+ bugherd.projects
+
+=== Retrieving projects based on an array of id's
+ bugherd.projects([1,2,3])
+
+=== Retrieving projects based on attributes
+ bugherd.projects(:id => 1)
+
+=== Retrieving a single project
+ bugherd.project(1)
+
+=== Retrieving all tickets
+ project = bugherd.project(1)
+ project.tickets
+
+=== Retrieving tickets based on array of id's
+ project = bugherd.project(1)
+ project.tickets([1,2,3,4])
 
 == Contributing to ticketmaster-bugherd
  

data/VERSION

@@ -1 +1 @@
-0.0.1
+0.0.2

data/lib/provider/bugherd.rb

@@ -2,7 +2,7 @@ module TicketMaster::Provider
   # This is the Bugherd Provider for ticketmaster
   module Bugherd
     include TicketMaster::Provider::Base
-    #TICKET_API = Bugherd::Ticket # The class to access the api's tickets
+    TICKET_API = Bugherd::Ticket # The class to access the api's tickets
     PROJECT_API = BugherdAPI::Project # The class to access the api's projects
     
     # This is for cases when you want to instantiate using TicketMaster::Provider::Bugherd.new(auth)
@@ -24,11 +24,7 @@ module TicketMaster::Provider
     
     # declare needed overloaded methods here
     def valid?
-      begin 
-        !BugherdAPI::User.find(:first).nil?
-      rescue
-        false
-      end
+      BugherdAPI::User.find(:all)
     end
 
   end

data/lib/provider/comment.rb

@@ -6,9 +6,37 @@ module TicketMaster::Provider
     # versions of the ticket.
     #
     class Comment < TicketMaster::Provider::Base::Comment
-      #API = Bugherd::Comment # The class to access the api's comments
+      API = BugherdAPI::Comment # The class to access the api's comments
       # declare needed overloaded methods here
       
+      def author
+        author = BugherdAPI::User.find(:all).select do |user|
+          user.id == self[:user_id]
+        end.first
+        "#{author.name} #{author.surname}" 
+      end
+
+      def ticket_id
+        self.prefix_options[:task_id]
+      end
+
+      def project_id
+        self.prefix_options[:project_id]
+      end
+
+      def body
+        self[:text]
+      end
+
+      def self.search(project_id, ticket_id, options = {}, limit = 1000)
+        API.find(:all, :params => {:project_id => project_id, :task_id => ticket_id}).collect do |comment|
+          self.new comment
+        end
+      end
+
+      def self.find_by_id(project_id, ticket_id, id)
+        self.new API.find(id, :params => {:project_id => project_id, :task_id => ticket_id})
+      end
     end
   end
 end

data/lib/provider/ticket.rb

@@ -34,11 +34,6 @@ module TicketMaster::Provider
         end
       end
 
-      def comments(*options)
-        warn "Bugherd API doesn't support comments"
-        []
-      end
-
     end
   end
 end

data/lib/ticketmaster-bugherd.rb

@@ -1,4 +1,5 @@
-require 'lib/bugherd/bugherd-api'
+require 'rubygems'
+require 'bugherd-api'
 
 %w{ bugherd ticket project comment }.each do |f|
   require File.dirname(__FILE__) + '/provider/' + f + '.rb';

data/spec/comments_spec.rb

@@ -1,7 +1,51 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe "TicketMaster::Provider::Bugherd::Comment" do 
+  before(:each) do 
+    @tm = TicketMaster.new(:bugherd, :email => 'user@email.com',
+                           :password => '0000')
+    @klass = TicketMaster::Provider::Bugherd::Comment
+    headers = {'Authorization' => 'Basic dXNlckBlbWFpbC5jb206MDAwMA==', 'Accept' => 'application/xml'}
+    ActiveResource::HttpMock.respond_to do |mock|
+      mock.get '/api_v1/projects/1458.xml', headers, fixture_for('projects/1458', 'xml'), 200
+      mock.get '/api_v1/projects/1458/tasks/4950.xml', headers, fixture_for('tasks/4950', 'xml'), 200
+      mock.get '/api_v1/projects/1458/tasks/4950/comments.xml', headers, fixture_for('comments', 'xml'), 200
+      mock.get '/api_v1/projects/1458/tasks/4950/comments/9760.xml', headers, fixture_for('comments/9760', 'xml'), 200
+      mock.get '/api_v1/users.xml', headers, fixture_for('users', 'xml'), 200
+    end
+    @project_id = 1458
+    @ticket_id = 4950
+    @project = @tm.project(@project_id)
+    @ticket = @project.ticket(@ticket_id)
+  end
 
-  it "should create a comment"
+  it "should retrieve all comments" do 
+    @ticket.comments.should be_an_instance_of(Array)
+    @ticket.comments.first.should be_an_instance_of(@klass)
+  end
 
+  it "should retrieve all comments based on an array of id's" do 
+    comments = @ticket.comments([9760])
+    comments.should be_an_instance_of(Array)
+    comments.first.should be_an_instance_of(@klass)
+  end
+
+  it "should be able to retrieve a single comment based on id" do 
+    comment = @ticket.comment(9760)
+    comment.should be_an_instance_of(@klass)
+  end
+
+  it "should be able to retrieve a single comment based on attributes" do 
+    comment = @ticket.comment(:id => 9760)
+    comment.should be_an_instance_of(@klass)
+  end
+
+  it "should have all valid fields for a comment" do 
+    comment = @ticket.comment(9760)
+    comment.author.should == 'Rafael George'
+    comment.body.should == "I'm working on this already"
+    comment.id.should == 9760
+    comment.ticket_id.should == 4950
+    comment.project_id.should == 1458
+  end
 end

data/spec/fixtures/comments.xml

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<comments type="array">
+  <comment>
+    <id type="integer">9760</id>
+    <created-at type="datetime">2011-08-18T19:57:56Z</created-at>
+    <text>I'm working on this already</text>
+    <user-id type="integer">1798</user-id>
+  </comment>
+</comments>

data/spec/fixtures/comments/9760.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<comment>
+  <id type="integer">9760</id>
+  <created-at type="datetime">2011-08-18T19:57:56Z</created-at>
+  <text>I'm working on this already</text>
+  <user-id type="integer">1798</user-id>
+</comment>

data/spec/projects_spec.rb

@@ -2,10 +2,10 @@ require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe "TicketMaster::Provider::Bugherd::Project" do 
   before(:each) do 
+    headers = {'Authorization' => 'Basic dXNlckBlbWFpbC5jb206MDAwMA==', 'Accept' => 'application/xml'}
     @tm = TicketMaster.new(:bugherd, :email => 'user@email.com',
                            :password => '0000')
     @klass = TicketMaster::Provider::Bugherd::Project
-    headers = {'Authorization' => 'Basic dXNlckBlbWFpbC5jb206MDAwMA==', 'Accept' => 'application/xml'}
     ActiveResource::HttpMock.respond_to do |mock|
       mock.get '/api_v1/projects.xml', headers, fixture_for('projects', 'xml'), 200
       mock.get '/api_v1/projects/1458.xml', headers, fixture_for('/projects/1458', 'xml'), 200

data/spec/ticketmaster-bugherd_spec.rb

@@ -2,33 +2,33 @@ require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe "TicketmasterBugherd" do
   before(:each) do 
-    @tm = TicketMaster.new(:bugherd, :email => 'user@bugherd.com', :password => '123456')
-    headers = {'Authorization' => 'Basic dXNlckBidWdoZXJkLmNvbToxMjM0NTY=', 'Accept' => 'application/xml'}
-    ActiveResource::HttpMock.respond_to do |mock|
-      mock.get '/api_v1/users.xml', headers, fixture_for('users', 'xml'), 200
+    headers = {'Authorization' => 'Basic Z2VvcmdlLnJhZmFlbEBnbWFpbC5jb206MTIzNDU2', 'Accept' => 'application/xml'}
+    ActiveResource::HttpMock.respond_to do |mock| 
+      @tm = TicketMaster.new(:bugherd, :email => 'george.rafael@gmail.com', :password => '123456')
+      ActiveResource::HttpMock.respond_to do |mock|
+        mock.get '/api_v1/users.xml', headers, fixture_for('users', 'xml'), 200
+      end
     end
-  end
 
-  context "Initialization and validation" do 
-    it "should be able to initialize a ticketmaster object" do
-      @tm.should be_an_instance_of(TicketMaster)
-      @tm.should be_a_kind_of(TicketMaster::Provider::Bugherd)
-    end
+    context "Initialization and validation" do 
+      it "should be able to initialize a ticketmaster object" do
+        @tm.should be_an_instance_of(TicketMaster)
+        @tm.should be_a_kind_of(TicketMaster::Provider::Bugherd)
+      end
 
-    it "should validate the ticketmaster instance" do 
-      @tm.valid?.should be_true
+      it "should validate the ticketmaster instance" do 
+        @tm.valid?.should be_true
+      end
     end
-  end
 
-  context "Raise exceptions" do 
-    it "should raise an exception without email or password" do 
-      lambda { @tm = TicketMaster.new(:bugherd, :email => '', :password => '') }.should raise_error
-    end
+    context "Raise exceptions" do 
+      it "should raise an exception without email or password" do 
+        lambda { @tm = TicketMaster.new(:bugherd, :email => '', :password => '') }.should raise_error
+      end
 
-    it "should raise an exception with an user message" do 
-      lambda { @tm = TicketMaster.new(:bugherd, :email => '', :password => '')}.should raise_error("You must provide email and password for authentication")
+      it "should raise an exception with an user message" do 
+        lambda { @tm = TicketMaster.new(:bugherd, :email => '', :password => '')}.should raise_error("You must provide email and password for authentication")
+      end
     end
   end
-
-
 end

data/spec/tickets_spec.rb

@@ -3,7 +3,7 @@ require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 describe "TicketMaster::Provider::Bugherd::Ticket" do 
 
   before(:each) do 
-    headers = { 'Authorization' => 'Basic Z2VvcmdlLnJhZmFlbEBnbWFpbC5jb206MTIzNDU2', 'Accept' => 'application/xml'}
+    headers = {'Authorization' => 'Basic Z2VvcmdlLnJhZmFlbEBnbWFpbC5jb206MTIzNDU2', 'Accept' => 'application/xml'}
     ActiveResource::HttpMock.respond_to do |mock| 
       mock.get '/api_v1/projects/1458.xml', headers, fixture_for('/projects/1458', 'xml'), 200
       mock.get '/api_v1/projects/1458/tasks.xml', headers, fixture_for('tasks', 'xml'), 200

data/ticketmaster-bugherd.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{ticketmaster-bugherd}
-  s.version = "0.0.1"
+  s.version = "0.0.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Rafael George"]
-  s.date = %q{2011-08-22}
+  s.date = %q{2011-08-29}
   s.description = %q{Ticketmaster provider for Bugherd}
   s.email = %q{george.rafael@gmail.com}
   s.extra_rdoc_files = [
@@ -26,7 +26,6 @@ Gem::Specification.new do |s|
     "README.rdoc",
     "Rakefile",
     "VERSION",
-    "lib/bugherd/bugherd-api.rb",
     "lib/provider/bugherd.rb",
     "lib/provider/comment.rb",
     "lib/provider/project.rb",
@@ -34,6 +33,8 @@ Gem::Specification.new do |s|
     "lib/ticketmaster-bugherd.rb",
     "request.xml",
     "spec/comments_spec.rb",
+    "spec/fixtures/comments.xml",
+    "spec/fixtures/comments/9760.xml",
     "spec/fixtures/projects.xml",
     "spec/fixtures/projects/1458.xml",
     "spec/fixtures/tasks.xml",
@@ -56,16 +57,14 @@ Gem::Specification.new do |s|
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<ticketmaster>, ["~> 0.6.0"])
-      s.add_runtime_dependency(%q<activesupport>, ["~> 3.0.0"])
-      s.add_runtime_dependency(%q<activeresource>, ["~> 3.0.0"])
+      s.add_runtime_dependency(%q<bugherd-api>, ["~> 0.0.3"])
       s.add_development_dependency(%q<rspec>, ["~> 2.3.0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.6.4"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
     else
       s.add_dependency(%q<ticketmaster>, ["~> 0.6.0"])
-      s.add_dependency(%q<activesupport>, ["~> 3.0.0"])
-      s.add_dependency(%q<activeresource>, ["~> 3.0.0"])
+      s.add_dependency(%q<bugherd-api>, ["~> 0.0.3"])
       s.add_dependency(%q<rspec>, ["~> 2.3.0"])
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
@@ -73,8 +72,7 @@ Gem::Specification.new do |s|
     end
   else
     s.add_dependency(%q<ticketmaster>, ["~> 0.6.0"])
-    s.add_dependency(%q<activesupport>, ["~> 3.0.0"])
-    s.add_dependency(%q<activeresource>, ["~> 3.0.0"])
+    s.add_dependency(%q<bugherd-api>, ["~> 0.0.3"])
     s.add_dependency(%q<rspec>, ["~> 2.3.0"])
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.6.4"])

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ticketmaster-bugherd
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 27
   prerelease: 
   segments: 
   - 0
   - 0
-  - 1
-  version: 0.0.1
+  - 2
+  version: 0.0.2
 platform: ruby
 authors: 
 - Rafael George
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-08-22 00:00:00 -04:00
+date: 2011-08-29 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -41,34 +41,18 @@ dependencies:
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        hash: 7
+        hash: 25
         segments: 
-        - 3
         - 0
         - 0
-        version: 3.0.0
-  version_requirements: *id002
-  name: activesupport
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  type: :runtime
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 7
-        segments: 
         - 3
-        - 0
-        - 0
-        version: 3.0.0
-  version_requirements: *id003
-  name: activeresource
+        version: 0.0.3
+  version_requirements: *id002
+  name: bugherd-api
   prerelease: false
 - !ruby/object:Gem::Dependency 
   type: :development
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -79,12 +63,12 @@ dependencies:
         - 3
         - 0
         version: 2.3.0
-  version_requirements: *id004
+  version_requirements: *id003
   name: rspec
   prerelease: false
 - !ruby/object:Gem::Dependency 
   type: :development
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -95,12 +79,12 @@ dependencies:
         - 0
         - 0
         version: 1.0.0
-  version_requirements: *id005
+  version_requirements: *id004
   name: bundler
   prerelease: false
 - !ruby/object:Gem::Dependency 
   type: :development
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -111,12 +95,12 @@ dependencies:
         - 6
         - 4
         version: 1.6.4
-  version_requirements: *id006
+  version_requirements: *id005
   name: jeweler
   prerelease: false
 - !ruby/object:Gem::Dependency 
   type: :development
-  requirement: &id007 !ruby/object:Gem::Requirement 
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -125,7 +109,7 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  version_requirements: *id007
+  version_requirements: *id006
   name: rcov
   prerelease: false
 description: Ticketmaster provider for Bugherd
@@ -147,7 +131,6 @@ files:
 - README.rdoc
 - Rakefile
 - VERSION
-- lib/bugherd/bugherd-api.rb
 - lib/provider/bugherd.rb
 - lib/provider/comment.rb
 - lib/provider/project.rb
@@ -155,6 +138,8 @@ files:
 - lib/ticketmaster-bugherd.rb
 - request.xml
 - spec/comments_spec.rb
+- spec/fixtures/comments.xml
+- spec/fixtures/comments/9760.xml
 - spec/fixtures/projects.xml
 - spec/fixtures/projects/1458.xml
 - spec/fixtures/tasks.xml

data/lib/bugherd/bugherd-api.rb

@@ -1,39 +0,0 @@
-require 'rubygems'
-require 'active_support'
-require 'active_resource'
-
-module BugherdAPI
-  class Error < StandardError; end
-
-  class << self
-    def authenticate(email, password)
-      @email = email 
-      @password = password
-      self::Base.user = email
-      self::Base.password = password
-    end
-
-    def resources
-      @resources ||= []
-    end
-  end
-
-  class Base < ActiveResource::Base
-    self.format = :xml
-    self.site = 'http://www.bugherd.com/api_v1/'
-    def self.inherited(base)
-      BugherdAPI.resources << base
-      super
-    end
-  end
-
-  class User < Base
-  end
-
-  class Project < Base
-  end
-
-  class Task < Base
-    self.site += 'projects/:project_id/'
-  end
-end