vigetlabs-unfuzzle 0.1.1 → 0.1.2

data/test/unit/unfuzzle/project_test.rb

@@ -7,16 +7,10 @@ module Unfuzzle
 
       should "be able to return a list of all projects" do
         response = mock_request_cycle :for => '/projects', :data => 'projects'
-
-        elements = Array.new
-
-        response.data.each do |data|
-          element = stub()
-          Unfuzzle::Project.expects(:new).with(data).returns(element)
-          elements << element
-        end
-
-        Project.all.should == elements
+      
+        Unfuzzle::Project.expects(:collection_from).with(response.body, 'projects/project').returns(['project_1', 'project_2'])
+      
+        Project.all.should == ['project_1', 'project_2']
       end
 
       should "be able to find a project by its slug" do
@@ -24,7 +18,7 @@ module Unfuzzle
 
         response = mock_request_cycle :for => "/projects/by_short_name/#{slug}", :data => 'project'
 
-        Unfuzzle::Project.expects(:new).with(response.data).returns('project')
+        Unfuzzle::Project.expects(:new).with(response.body).returns('project')
 
         Project.find_by_slug(slug).should == 'project'
       end
@@ -34,7 +28,7 @@ module Unfuzzle
 
         response = mock_request_cycle :for => "/projects/#{id}", :data => 'project'
 
-        Unfuzzle::Project.expects(:new).with(response.data).returns('project')
+        Unfuzzle::Project.expects(:new).with(response.body).returns('project')
 
         Project.find_by_id(id).should == 'project'
       end
@@ -44,21 +38,21 @@ module Unfuzzle
 
     context "An instance of the Project class" do
 
-      when_populating Project, :from => 'projects' do
-
-        value_for :id,                :is => 1
-        value_for :archived,          :is => false
-        value_for :slug,              :is => 'aa'
-        value_for :name,              :is => 'A Client'
-        value_for :description,       :is => 'A great project'
-        value_for :created_timestamp, :is => '2008-07-28T16:57:10Z'
-        value_for :updated_timestamp, :is => '2009-04-28T18:48:52Z'
-
+      when_populating Project, :from => 'project' do
+      
+        value_for :id,          :is => 1
+        value_for :archived,    :is => false
+        value_for :slug,        :is => 'blip'
+        value_for :name,        :is => 'Blip Bleep Co.'
+        value_for :description, :is => 'This is the project for Blip Bleep Co.'
+        value_for :created_at,  :is => Time.parse('2008-07-28T16:57:10Z')
+        value_for :updated_at,  :is => Time.parse('2009-04-28T18:48:52Z')
+      
       end
 
       context "with a new instance" do
 
-        setup { @project = Project.new(stub()) }
+        setup { @project = Project.new }
 
         should "know that it's archived" do
           @project.stubs(:archived).with().returns(true)
@@ -70,20 +64,6 @@ module Unfuzzle
           @project.archived?.should be(false)
         end
 
-        should "have a create date" do
-          DateTime.expects(:parse).with('2008-07-28T16:57:10Z').returns('create_date')
-
-          @project.stubs(:created_timestamp).with().returns('2008-07-28T16:57:10Z')
-          @project.created_at.should == 'create_date'
-        end
-
-        should "have an update date" do
-          DateTime.expects(:parse).with('2009-04-28T18:48:52Z').returns('update_date')
-
-          @project.stubs(:updated_timestamp).with().returns('2009-04-28T18:48:52Z')
-          @project.updated_at.should == 'update_date'
-        end
-
         should "have a list of associated milestones" do
           id = 1
           Milestone.expects(:find_all_by_project_id).with(id).returns('milestones')

data/test/unit/unfuzzle/request_test.rb

@@ -14,7 +14,7 @@ module Unfuzzle
       
       should "be able to perform a PUT request" do
         request = mock() {|r| r.expects(:put).with().returns('response') }
-        Unfuzzle::Request.expects(:new).with('/projects', '<payload>', :format => :xml).returns(request)
+        Unfuzzle::Request.expects(:new).with('/projects', '<payload>').returns(request)
         
         Unfuzzle::Request.put('/projects', '<payload>').should == 'response'
       end
@@ -23,27 +23,17 @@ module Unfuzzle
     
     context "An instance of the Request class" do
 
-      should "default the data format to JSON" do
-        request = Unfuzzle::Request.new('/projects')
-        request.request_format.should == 'json'
-      end
-
-      should "be able to override the request data format" do
-        request = Unfuzzle::Request.new('/projects', nil, :format => :xml)
-        request.request_format.should == 'xml'
-      end
-      
       should "have an endpoint URI" do
         Unfuzzle.stubs(:subdomain).with().returns('viget')
         
         request = Unfuzzle::Request.new('/projects')
-        request.endpoint_uri.should == URI.parse('http://viget.unfuddle.com/api/v1/projects.json')
+        request.endpoint_uri.should == URI.parse('http://viget.unfuddle.com/api/v1/projects.xml')
       end
       
       should "have an endpoint URI with the appropriate format when specified" do
         Unfuzzle.stubs(:subdomain).with().returns('viget')
         
-        request = Unfuzzle::Request.new('/projects', nil, :format => :xml)
+        request = Unfuzzle::Request.new('/projects', nil)
         request.endpoint_uri.should == URI.parse('http://viget.unfuddle.com/api/v1/projects.xml')
       end
       

data/test/unit/unfuzzle/response_test.rb

@@ -31,34 +31,6 @@ module Unfuzzle
         response.error?.should be(true)
       end
       
-      should "be able to parse the response" do
-        response = Unfuzzle::Response.new(stub())
-        response.expects(:body).with().returns('json')
-        response.stubs(:error?).with().returns(false)
-                
-        JSON.expects(:parse).with('json').returns('data')
-        
-        response.data.should == 'data'
-      end
-      
-      should "cache the parsed data from the response" do
-        response = Unfuzzle::Response.new(stub())
-        response.stubs(:body).with().returns('json')
-        response.stubs(:error?).with().returns(false)
-        
-        JSON.stubs(:parse).with('json').once.returns('data')
-        
-        2.times { response.data }
-      end
-      
-      should "return nil when parsing data if there are errors in the response" do
-        response = Unfuzzle::Response.new(stub())
-        response.expects(:error?).with().returns(true)
-        
-        response.data.should be(nil)
-        
-      end
-      
     end
     
   end

data/test/unit/unfuzzle/severity_test.rb

@@ -0,0 +1,36 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+module Unfuzzle
+  class SeverityTest < Test::Unit::TestCase
+    
+    context "The Severity class" do
+      
+      should "be able to return a severity for a project and severity id" do
+        project_id  = 1
+        severity_id = 1
+        
+        response = mock_request_cycle :for => "/projects/#{project_id}/severities/#{severity_id}", :data => 'severity'
+
+        Unfuzzle::Severity.expects(:new).with(response.body).returns('severity')
+        
+        Unfuzzle::Severity.find_by_project_id_and_severity_id(project_id, severity_id).should == 'severity'
+      end
+      
+    end
+    
+    context "An instance of the Severity class" do
+      
+      when_populating Severity, :from => 'severity' do
+
+        value_for :id,          :is => 1
+        value_for :name,        :is => 'Story'
+        value_for :project_id,  :is => 2
+        value_for :created_at,  :is => Time.parse('2009-04-02T19:44:49Z')
+        value_for :updated_at,  :is => Time.parse('2009-04-02T19:44:49Z')
+        
+      end
+      
+    end
+    
+  end
+end

data/test/unit/unfuzzle/ticket_test.rb

@@ -10,15 +10,9 @@ module Unfuzzle
 
         response = mock_request_cycle :for => "/projects/#{project_id}/tickets", :data => 'tickets'
 
-        tickets = []
+        Unfuzzle::Ticket.expects(:collection_from).with(response.body, 'tickets/ticket').returns(['ticket_1', 'ticket_2'])
 
-        response.data.each do |data|
-          ticket = stub()
-          Ticket.expects(:new).with(data).returns(ticket)
-          tickets << ticket
-        end
-
-        Ticket.find_all_by_project_id(project_id).should == tickets
+        Ticket.find_all_by_project_id(project_id).should == ['ticket_1', 'ticket_2']
       end
 
       should "be able to return a list of tickets for a milestone" do
@@ -27,32 +21,29 @@ module Unfuzzle
 
         response = mock_request_cycle :for => "/projects/#{project_id}/milestones/#{milestone_id}/tickets", :data => 'tickets'
 
-        tickets = []
-
-        response.data.each do |data|
-          ticket = stub()
-          Ticket.expects(:new).with(data).returns(ticket)
-          tickets << ticket
-        end
-
-        Ticket.find_all_by_project_id_and_milestone_id(project_id, milestone_id).should == tickets
+        Unfuzzle::Ticket.expects(:collection_from).with(response.body, 'tickets/ticket').returns(['ticket_1', 'ticket_2'])
+        
+        Ticket.find_all_by_project_id_and_milestone_id(project_id, milestone_id).should == ['ticket_1', 'ticket_2']
       end
 
     end
 
     context "An instance of the Ticket class" do
 
-      when_populating Ticket, :from => 'tickets' do
+      when_populating Ticket, :from => 'ticket' do
 
         value_for :id,                :is => 1
         value_for :project_id,        :is => 1
         value_for :milestone_id,      :is => 1
-        value_for :created_timestamp, :is => '2008-11-25T14:00:19Z'
-        value_for :updated_timestamp, :is => '2008-12-31T15:51:41Z'
-        value_for :number,            :is => 1
+        value_for :severity_id,       :is => 123
+        value_for :priority_id,       :is => 3
+        value_for :component_id,      :is => 1
+        value_for :created_at,        :is => Time.parse('2008-11-25T14:00:19Z')
+        value_for :updated_at,        :is => Time.parse('2008-12-31T15:51:41Z')
+        value_for :number,            :is => '1'
         value_for :title,             :is => 'Ticket #1'
         value_for :description,       :is => 'Do something important'
-        value_for :due_datestamp,     :is => nil
+        value_for :due_on,            :is => nil
         value_for :status,            :is => 'closed'
 
       end
@@ -61,32 +52,13 @@ module Unfuzzle
       should_set_a_value_for :description
       
       context "with a new instance" do
-        setup { @ticket = Ticket.new({}) }
-
-        should "have a create date/time" do
-          DateTime.expects(:parse).with('2008-07-28T16:57:10Z').returns('create_date')
-
-          @ticket.stubs(:created_timestamp).with().returns('2008-07-28T16:57:10Z')
-          @ticket.created_at.should == 'create_date'
-        end
-
-        should "have an update date/time" do
-          DateTime.expects(:parse).with('2009-04-28T18:48:52Z').returns('update_date')
-
-          @ticket.stubs(:updated_timestamp).with().returns('2009-04-28T18:48:52Z')
-          @ticket.updated_at.should == 'update_date'
-        end
+        setup { @ticket = Ticket.new }
 
         should "have a due date" do
-          Date.expects(:parse).with('2008-07-30').returns('due_date')
-
-          @ticket.stubs(:due_datestamp).with().returns('2008-07-30')
-          @ticket.due_on.should == 'due_date'
-        end
-
-        should "not have a due date if there isn't one associated" do
-          @ticket.stubs(:due_datestamp).with().returns(nil)
-          @ticket.due_on.should be(nil)
+          xml = '<ticket><due-on>2009-08-01</due-on></ticket>'
+          
+          ticket = Ticket.new(xml)
+          ticket.due_on.should == Date.parse('2009-08-01')
         end
         
         should "have an associated milestone" do
@@ -98,6 +70,95 @@ module Unfuzzle
           @ticket.milestone.should == 'milestone'
         end
         
+        should "have an associated severity" do
+          project_id  = 1
+          severity_id = 1
+
+          Severity.expects(:find_by_project_id_and_severity_id).with(project_id, severity_id).returns('severity')
+          
+          @ticket.stubs(:project_id).with().returns(project_id)
+          @ticket.stubs(:severity_id).with().returns(severity_id)
+          
+          @ticket.severity.should == 'severity'
+        end
+        
+        should "have no associated severity if it is not set" do
+          @ticket.stubs(:severity_id).with().returns(nil)
+          @ticket.severity.should be(nil)
+        end
+        
+        should "have a severity_name" do
+          @ticket.stubs(:severity).with().returns(stub(:name => 'Story'))
+          @ticket.severity_name.should == 'Story'
+        end
+        
+        should "have no severity_name if there is no severity" do
+          @ticket.stubs(:severity).with().returns(nil)
+          @ticket.severity_name.should be(nil)
+        end
+        
+        should "have an associated priority" do
+          Priority.expects(:new).with(1).returns('priority')
+          
+          @ticket.stubs(:priority_id).with().returns(1)
+          @ticket.priority.should == 'priority'
+        end
+        
+        should "have a priority_name" do
+          @ticket.stubs(:priority).with().returns(stub(:name => 'High'))
+          @ticket.priority_name.should == 'High'
+        end
+        
+        should "have an associated component" do
+          project_id  = 1
+          component_id = 1
+
+          Component.expects(:find_by_project_id_and_component_id).with(project_id, component_id).returns('component')
+          
+          @ticket.stubs(:project_id).with().returns(project_id)
+          @ticket.stubs(:component_id).with().returns(component_id)
+          
+          @ticket.component.should == 'component'
+        end
+
+        should "have no associated component if it is not set" do
+          @ticket.stubs(:component_id).with().returns(nil)
+          @ticket.component.should be(nil)
+        end
+        
+        should "have a component_name" do
+          @ticket.stubs(:component).with().returns(stub(:name => 'Admin'))
+          @ticket.component_name.should == 'Admin'
+        end
+        
+        should "have no component_name if there is no component" do
+          @ticket.stubs(:component).with().returns(nil)
+          @ticket.component_name.should be(nil)
+        end
+        
+        should "be able to generate a hash representation of itself for updating" do
+          @ticket.id           = 1
+          @ticket.project_id   = 2
+          @ticket.milestone_id = 3
+          @ticket.number       = '12'
+          @ticket.title        = 'summary'
+          @ticket.description  = 'description'
+          @ticket.status       = 'closed'
+
+          
+          expected = {
+            'id'           => 1,
+            'project_id'   => 2,
+            'milestone_id' => 3,
+            'number'       => '12',
+            'summary'      => 'summary',
+            'description'  => 'description',
+            'status'       => 'closed'
+          }
+          
+          @ticket.to_hash.should == expected          
+        end
+        
         should "be able to perform an update" do
           @ticket.stubs(:project_id).with().returns(1)
           @ticket.stubs(:id).with().returns(2)
@@ -105,7 +166,7 @@ module Unfuzzle
           resource_path = '/projects/1/tickets/2'
           ticket_xml    = '<ticket />'
           
-          @ticket.stubs(:to_xml).with().returns(ticket_xml)
+          @ticket.stubs(:to_xml).with('ticket').returns(ticket_xml)
           
           Unfuzzle::Request.expects(:put).with(resource_path, ticket_xml).returns('response')
           

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: vigetlabs-unfuzzle
 version: !ruby/object:Gem::Version 
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors: 
 - Patrick Reagan
@@ -9,18 +9,18 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-07-06 00:00:00 -07:00
+date: 2009-07-23 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: json
+  name: graft
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.1.6
+        version: 0.1.1
     version: 
 description: 
 email: patrick.reagan@viget.com
@@ -34,28 +34,37 @@ files:
 - README.rdoc
 - Rakefile
 - lib/unfuzzle
+- lib/unfuzzle/component.rb
 - lib/unfuzzle/milestone.rb
-- lib/unfuzzle/model.rb
+- lib/unfuzzle/priority.rb
 - lib/unfuzzle/project.rb
 - lib/unfuzzle/request.rb
 - lib/unfuzzle/response.rb
+- lib/unfuzzle/severity.rb
 - lib/unfuzzle/ticket.rb
 - lib/unfuzzle/version.rb
 - lib/unfuzzle.rb
 - test/fixtures
-- test/fixtures/milestone.json
-- test/fixtures/milestones.json
-- test/fixtures/project.json
-- test/fixtures/projects.json
-- test/fixtures/tickets.json
+- test/fixtures/component.xml
+- test/fixtures/components.xml
+- test/fixtures/milestone.xml
+- test/fixtures/milestones.xml
+- test/fixtures/project.xml
+- test/fixtures/projects.xml
+- test/fixtures/severities.xml
+- test/fixtures/severity.xml
+- test/fixtures/ticket.xml
+- test/fixtures/tickets.xml
 - test/test_helper.rb
 - test/unit
 - test/unit/unfuzzle
+- test/unit/unfuzzle/component_test.rb
 - test/unit/unfuzzle/milestone_test.rb
-- test/unit/unfuzzle/model_test.rb
+- test/unit/unfuzzle/priority_test.rb
 - test/unit/unfuzzle/project_test.rb
 - test/unit/unfuzzle/request_test.rb
 - test/unit/unfuzzle/response_test.rb
+- test/unit/unfuzzle/severity_test.rb
 - test/unit/unfuzzle/ticket_test.rb
 - test/unit/unfuzzle_test.rb
 has_rdoc: false
@@ -84,6 +93,6 @@ rubyforge_project:
 rubygems_version: 1.2.0
 signing_key: 
 specification_version: 3
-summary: This gem provides an interface to the Unfuddle JSON API
+summary: This gem provides an interface to the Unfuddle XML API
 test_files: []