roart 0.1.4

data/roart.gemspec

@@ -0,0 +1,37 @@
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = %q{roart}
+  s.version = "0.1.4"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["PJ Davis"]
+  s.date = %q{2009-08-19}
+  s.description = %q{Interface for working with Request Tracker (RT) tickets inspired by ActiveRecord.}
+  s.email = %q{pj.davis@gmail.com}
+  s.extra_rdoc_files = ["History.txt", "README.rdoc", "spec/test_data/full_history.txt", "spec/test_data/single_history.txt"]
+  s.files = ["History.txt", "README.rdoc", "Rakefile", "lib/roart.rb", "lib/roart/callbacks.rb", "lib/roart/connection.rb", "lib/roart/core/array.rb", "lib/roart/core/hash.rb", "lib/roart/core/string.rb", "lib/roart/errors.rb", "lib/roart/history.rb", "lib/roart/roart.rb", "lib/roart/ticket.rb", "lib/roart/ticket_page.rb", "roart.gemspec", "spec/roart/callbacks_spec.rb", "spec/roart/connection_spec.rb", "spec/roart/core/array_spec.rb", "spec/roart/core/hash_spec.rb", "spec/roart/core/string_spec.rb", "spec/roart/history_spec.rb", "spec/roart/roart_spec.rb", "spec/roart/ticket_page_spec.rb", "spec/roart/ticket_spec.rb", "spec/roart_spec.rb", "spec/spec_helper.rb", "spec/test_data/full_history.txt", "spec/test_data/single_history.txt"]
+  s.has_rdoc = true
+  s.homepage = %q{http://github.com/pjdavis/roart}
+  s.rdoc_options = ["--main", "README.rdoc"]
+  s.require_paths = ["lib"]
+  s.rubyforge_project = %q{roart}
+  s.rubygems_version = %q{1.3.2}
+  s.summary = %q{Interface for working with Request Tracker (RT) tickets inspired by ActiveRecord}
+
+  if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
+      s.add_runtime_dependency(%q<mechanize>, [">= 0.9.0"])
+      s.add_development_dependency(%q<bones>, [">= 2.5.1"])
+    else
+      s.add_dependency(%q<mechanize>, [">= 0.9.0"])
+      s.add_dependency(%q<bones>, [">= 2.5.1"])
+    end
+  else
+    s.add_dependency(%q<mechanize>, [">= 0.9.0"])
+    s.add_dependency(%q<bones>, [">= 2.5.1"])
+  end
+end

data/spec/roart/callbacks_spec.rb

@@ -0,0 +1,49 @@
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe 'ticket callbacks' do
+
+  describe 'create callbacks' do
+    
+    before do
+      post_data = @payload = {:subject => 'A New Ticket', :queue => 'My Queue'}
+      post_data.update(:id => 'ticket/new')
+      post_data = to_content_format(post_data)
+      mock_connection = mock('connection')
+      mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/new', {:content => post_data}).and_return("RT/3.6.6 200 Ok\n\n# Ticket 267783 created.")
+      mock_connection.should_receive(:server).and_return('uri')
+      Roart::Ticket.should_receive(:connection).twice.and_return(mock_connection)
+    end
+    
+    it 'should call before_create callback' do
+      
+      ticket = Roart::Ticket.new(@payload)
+      ticket.should_receive(:before_create)
+      ticket.should_receive(:after_create)
+      ticket.save
+    end
+    
+  end
+  
+  describe 'update callbacks' do
+    
+    before do
+      @post_data = @payload = {:subject => 'A New Ticket', :queue => 'My Queue'}
+      @post_data[:subject] = 'An Old Ticket'
+      @post_data = to_content_format(@post_data)
+      @mock_connection = mock('connection')
+      @mock_connection.should_receive(:server).and_return('uri')
+      Roart::Ticket.should_receive(:connection).twice.and_return(@mock_connection)
+    end
+    
+    it 'should call before_update callbacks' do
+      @mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/1/edit', {:content => @post_data}).and_return("RT/3.6.6 200 Ok\n\n# Ticket 267783 updated.")
+      ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+      ticket.instance_variable_set("@saved", false)
+      ticket.should_receive(:before_update)
+      ticket.should_receive(:after_update)
+      ticket.save
+    end
+    
+  end
+
+end

data/spec/roart/connection_spec.rb

@@ -0,0 +1,7 @@
+
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe "Connection" do
+  
+  
+end 

data/spec/roart/core/array_spec.rb

@@ -0,0 +1,13 @@
+require File.join(File.dirname(__FILE__), %w[ .. .. spec_helper])
+
+describe 'array extentions' do
+  
+  it 'should turn an array to a nil valued hash' do
+    array = %w[key1 key2]
+    hash = array.to_hash
+    hash.has_key?(:key1).should be_true
+    hash.has_key?(:key2).should be_true
+    hash[:key1].should be_nil
+  end
+  
+end

data/spec/roart/core/hash_spec.rb

@@ -0,0 +1,16 @@
+require File.join(File.dirname(__FILE__), %w[ .. .. spec_helper])
+
+describe 'hash extentions' do
+
+  it 'should format the content correctly' do
+    payload = {:subject => "A New Ticket", :queue => 'My Queue'}
+    payload.to_content_format.include?("Subject: A New Ticket").should be_true
+    payload.to_content_format.include?("Queue: My Queue").should be_true
+  end
+  
+  it 'should handel custom fields' do
+    payload = {:cf_stuff => 'field'}
+    payload.to_content_format.should == "CF-Stuff: field"
+  end
+  
+end

data/spec/roart/core/string_spec.rb

@@ -0,0 +1,13 @@
+require File.join(File.dirname(__FILE__), %w[ .. .. spec_helper])
+
+describe 'string extentions' do
+
+  it 'should underscore a word' do
+    'SomeGuy'.underscore.should == 'some_guy'
+  end
+  
+  it 'should camelcase a word' do
+    'some_guy'.camelize.should == 'SomeGuy'
+  end
+
+end

data/spec/roart/history_spec.rb

@@ -0,0 +1,60 @@
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe "History" do
+  
+  it 'should have a ticket' do
+    search_array = ['1:subject']
+    search_array.extend(Roart::TicketPage)
+    full_ticket = Roart::Ticket.send(:instantiate, {:id => 1, :subject => 'subject', :full => true})
+    @ticket = Roart::Ticket.send(:instantiate, search_array.to_search_array ).first
+    Roart::History.should_receive(:get_page).and_return('200')    
+
+    history = @ticket.histories
+    history.ticket.should == @ticket
+    
+  end
+  
+  describe 'getting ticket history' do
+    
+    it 'should create the crect URI' do
+      connection = mock('connection', :rest_path => 'REST/1.0/')
+      myclass = mock('class', :connection => connection)
+      ticket = mock(:ticket, :id => 1, :class => myclass)
+      hash = mock(:options_hash, :[] => ticket)
+      Roart::History.should_receive(:default_options).and_return(hash)
+      Roart::History.send(:uri_for, ticket).should == 'REST/1.0/ticket/1/history?format=l'
+    end
+  end
+  
+  describe 'reading history pages' do
+    
+    before do
+      @page = File.open(File.join(File.dirname(__FILE__), %w[ .. test_data full_history.txt])).readlines.join
+      @ticket = mock('ticket')
+      Roart::History.should_receive(:get_page).and_return(@page)
+      @histories = Roart::History.default(:ticket => @ticket)
+    end
+    
+    it 'should have the right number of histories' do
+      @histories.size.should == 5
+    end
+    
+    it 'should have a ticket' do
+      @histories.first.ticket.should == @ticket
+    end
+    
+    it 'should have an id' do
+      @histories.first.id.should == 34725
+    end
+    
+    it 'should return itself for all' do
+      @histories.all.should == @histories
+    end
+    
+    it 'should return the ticket' do
+      @histories.ticket.should == @ticket
+    end
+    
+  end
+  
+end 

data/spec/roart/roart_spec.rb

@@ -0,0 +1,11 @@
+
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe "Roart" do
+  
+  it "should raise an error if there aren't required fields" do
+    attributes = {:subject => "Not Enough Stuff"}
+    lambda { Roart::check_keys!(attributes, Roart::Tickets::RequiredAttributes) }.should raise_error(Roart::ArgumentError)
+  end
+  
+end

data/spec/roart/ticket_page_spec.rb

@@ -0,0 +1,59 @@
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe 'ticket page' do
+  
+  describe 'ticket hash' do
+    
+    it 'should convert an array to a hash' do
+      array = ["id : 10", "subject : asdf"]
+      array.extend(Roart::TicketPage)
+      hash = array.to_hash
+      hash.has_key?(:id).should be_true
+      hash[:id].should == 10
+      hash[:subject].should == 'asdf'
+    end
+  
+  end
+  
+  describe 'search array' do
+    
+    before do
+      @array = ["123 : Subject", "234 : Subject"]
+      @array.extend(Roart::TicketPage)
+      @array = @array.to_search_array
+    end
+    
+    it 'should make an array of search results' do
+      @array.size.should == 2
+    end
+    
+    it 'should put search elements into the search array' do
+      @array.first[:id].should == 123
+      @array.last[:id].should == 234
+    end
+    
+  end
+  
+  describe 'ticket history hash' do
+    
+    before do
+      @page = File.open(File.join(File.dirname(__FILE__), %w[ .. test_data single_history.txt])).readlines.join
+      @page = @page.split("\n")
+      @page.extend(Roart::TicketPage)
+    end
+    
+    it 'should give back the hash of history' do
+      @page.to_history_hash.class.should == Hash
+    end
+    
+    it 'should have some content' do
+      @page.to_history_hash[:content].should_not be_nil
+    end
+    
+    it 'should have the create type' do
+      @page.to_history_hash[:type].should == 'Create'
+    end
+    
+  end
+  
+end

data/spec/roart/ticket_spec.rb

@@ -0,0 +1,440 @@
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe "Ticket" do
+  
+  it "should have a connection" do
+    Roart::Connection.should_receive(:new).with('some options').and_return(true)
+    Roart::Ticket.connection('some options').should == true
+  end
+  
+  it "should find the first ticket" do
+    Roart::Ticket.should_receive(:find_initial).with(:my => 'options').and_return('a ticket')
+    Roart::Ticket.find(:first, :my => 'options').should == 'a ticket'
+  end
+  
+  it 'should find all tickets' do
+    Roart::Ticket.should_receive(:find_all).with(:my => 'options').and_return(%w[array of tickets])
+    Roart::Ticket.find(:all, :my => 'options').should == ['array', 'of', 'tickets']
+  end
+  
+  it 'should find a ticket by id' do
+    Roart::Ticket.should_receive(:find_by_ids).with([12345],{}).and_return('a ticket')
+    Roart::Ticket.find(12345).should == 'a ticket'
+  end
+  
+  describe "find initial" do 
+  
+    it 'should set options to include :limit => 1' do 
+      Roart::Ticket.should_receive(:find_all).with({:limit => 1}).and_return(['ticket', 'not seen'])
+      Roart::Ticket.send(:find_initial)
+    end
+    
+    it 'should not overwrite the options hash' do
+      Roart::Ticket.should_receive(:find_all).with({:queue => 'queue', :limit => 1}).and_return(['ticket', 'not seen'])
+      Roart::Ticket.send(:find_initial, :queue => 'queue')      
+    end
+    
+    it 'should return 1 ticket object' do
+      Roart::Ticket.should_receive(:find_all).with({:limit => 1}).and_return(['ticket', 'not seen'])
+      Roart::Ticket.send(:find_initial).should == 'ticket'
+    end
+    
+  end
+  
+  describe 'page array' do
+    
+    it 'should raise an error if not a 200 response' do
+      connection = mock('connection', :get => 'some error message')
+      Roart::Ticket.should_receive(:connection).and_return(connection)
+      lambda do 
+        Roart::Ticket.send(:page_array, 'www.whatever')
+      end.should raise_error(Roart::TicketSystemInterfaceError)
+    end
+    
+    it 'should raise an error if nothing is returned' do
+      connection = mock('connection', :get => nil)
+      Roart::Ticket.should_receive(:connection).and_return(connection)
+      lambda do 
+        Roart::Ticket.send(:page_array, 'www.whatever')
+      end.should raise_error(Roart::TicketSystemError)
+    end
+    
+    it 'should give back an array of strings' do
+      connection = mock('connection', :get => "200 OK\n23:SomeTicket\n33:Another")
+      Roart::Ticket.should_receive(:connection).and_return(connection)
+      Roart::Ticket.send(:page_array, 'uri').should == ['23:SomeTicket', '33:Another']
+    end
+    
+  end
+  
+  describe 'getting tickets from URI' do
+    
+    describe 'search' do
+      
+      it 'should give an array of tickets' do
+        Roart::Ticket.should_receive(:page_array).with('uri').and_return(['23:SomeTicket', '33:Another'])
+        Roart::Ticket.send(:get_tickets_from_search_uri, 'uri').size.should == 2
+      end
+      
+      it 'should not have full tickets' do
+        Roart::Ticket.should_receive(:page_array).with('uri').and_return(['23:SomeTicket', '33:Another'])
+        Roart::Ticket.send(:get_tickets_from_search_uri, 'uri').each do |ticket|
+          ticket.full.should_not be_true
+        end
+      end
+      
+    end
+    
+    describe 'full ticket' do
+      
+      it 'should give a ticket' do
+        Roart::Ticket.should_receive(:page_array).with('uri').and_return(['id:23', 'subject:someticket'])
+        Roart::Ticket.send(:get_ticket_from_uri, 'uri').is_a?(Roart::Ticket).should be_true
+      end
+      
+      it 'should be a full ticket' do
+        Roart::Ticket.should_receive(:page_array).with('uri').and_return(['id:23', 'subject:someticket'])
+        Roart::Ticket.send(:get_ticket_from_uri, 'uri').full.should be_true
+      end
+      
+    end
+    
+    describe 'getting ticket from id' do
+      
+      it 'should give a ticket' do
+        connection = mock('connection', :server => "server")
+        Roart::Ticket.should_receive(:connection).and_return(connection)
+        Roart::Ticket.should_receive(:get_ticket_from_uri).with('server' + '/REST/1.0/ticket/' + 12345.to_s).and_return('a ticket')
+        Roart::Ticket.send(:get_ticket_by_id, 12345).should == 'a ticket'
+      end
+      
+    end
+    
+    describe 'find methods' do
+      
+      it 'should get all tickets from an options hash' do
+        Roart::Ticket.should_receive('construct_search_uri').with('searches').and_return('uri')
+        Roart::Ticket.should_receive('get_tickets_from_search_uri').with('uri').and_return(['tickets'])
+        Roart::Ticket.send(:find_all, 'searches').should == ['tickets']
+      end
+      
+      it 'should find tickets from id' do
+        Roart::Ticket.should_receive(:get_ticket_by_id).with('id').and_return('a ticket')
+        Roart::Ticket.send(:find_by_ids, ['id'], {}).should == 'a ticket'
+      end
+      
+    end
+    
+    describe 'building a search uri' do
+      
+      before do
+        @connection = mock('connection', :server => "server")
+        @search_string = "/REST/1.0/search/ticket?"
+        Roart::Ticket.should_receive(:connection).and_return(@connection)
+        @query = @connection.server + @search_string + 'query= '
+      end
+      
+      describe 'queues' do
+        
+        it 'should include a queue' do
+          Roart::Ticket.send(:construct_search_uri, {:queue => 'myQueue'}).should == @query + "Queue = 'myQueue'"
+        end
+        
+        it 'should take multiple queues' do
+          Roart::Ticket.send(:construct_search_uri, {:queue => ['myQueue', 'another']}).should == @query + "( Queue = 'myQueue' OR Queue = 'another' )"
+        end
+
+      end
+      
+      describe 'dates' do
+      
+        before do 
+          @time = Time.now
+        end
+        
+        it 'should accept a date' do
+          Roart::Ticket.send(:construct_search_uri, {:created => @time}).should == @query + "( Created > '#{dbtime(@time)}' )"
+        end
+        
+        it 'should accept an array of dates' do
+          Roart::Ticket.send(:construct_search_uri, {:created => [@time, @time + 300]}).should == @query + "( Created > '#{dbtime(@time)}' AND Created < '#{dbtime(@time + 300)}' )"
+        end
+        
+        it 'should accept a range of dates' do
+          Roart::Ticket.send(:construct_search_uri, {:created => (@time..(@time + 300))}).should == @query + "( Created > '#{dbtime(@time)}' AND Created < '#{dbtime(@time + 300)}' )"
+        end
+        
+        it 'should accept an array of strings' do
+          Roart::Ticket.send(:construct_search_uri, {:created => %w[cat dog]}).should == @query + "( Created > 'cat' AND Created < 'dog' )"
+        end
+        
+        it 'should accept a string' do
+          Roart::Ticket.send(:construct_search_uri, {:created => 'time'}).should == @query + "( Created > 'time' )"
+        end
+        
+        describe 'date fields' do
+        
+          it 'should search started' do
+            Roart::Ticket.send(:construct_search_uri, {:started => 'time'}).should == @query + "( Started > 'time' )"
+          end
+        
+          it 'should search resolved' do
+            Roart::Ticket.send(:construct_search_uri, {:resolved => 'time'}).should == @query + "( Resolved > 'time' )"
+          end
+        
+          it 'should search told' do
+            Roart::Ticket.send(:construct_search_uri, {:told => 'time'}).should == @query + "( Told > 'time' )"
+          end
+        
+          it 'should search last_updated' do
+            Roart::Ticket.send(:construct_search_uri, {:last_updated => 'time'}).should == @query + "( LastUpdated > 'time' )"
+          end
+        
+          it 'should search starts' do
+            Roart::Ticket.send(:construct_search_uri, {:starts => 'time'}).should == @query + "( Starts > 'time' )"
+          end
+        
+          it 'should search due' do
+            Roart::Ticket.send(:construct_search_uri, {:due => 'time'}).should == @query + "( Due > 'time' )"
+          end
+        
+          it 'should search updated' do
+            Roart::Ticket.send(:construct_search_uri, {:updated => 'time'}).should == @query + "( Updated > 'time' )"
+          end
+        
+        end
+        
+      end
+      
+      describe 'searches' do
+        
+        it 'should accept a string' do 
+          Roart::Ticket.send(:construct_search_uri, {:subject => 'fish'}).should == @query + "Subject LIKE 'fish'"
+        end
+        
+        it 'should accept an array' do 
+          Roart::Ticket.send(:construct_search_uri, {:subject => %w[cramanation station]}).should == @query + "( Subject LIKE 'cramanation' AND Subject LIKE 'station' )"
+        end
+        
+        describe 'search fields' do 
+          
+          it 'should search content' do 
+            Roart::Ticket.send(:construct_search_uri, {:content => 'fish'}).should == @query + "Content LIKE 'fish'"
+          end
+          
+          it 'should search content_type' do 
+            Roart::Ticket.send(:construct_search_uri, {:content_type => 'fish'}).should == @query + "ContentType LIKE 'fish'"
+          end
+          
+          it 'should search file_name' do 
+            Roart::Ticket.send(:construct_search_uri, {:file_name => 'fish'}).should == @query + "FileName LIKE 'fish'"
+          end
+          
+        end
+        
+      end
+      
+      describe 'status' do
+        
+        it 'should accept a string' do
+          Roart::Ticket.send(:construct_search_uri, {:status => 'new'}).should == @query + "Status = 'new'"
+        end
+        
+        it 'should accept a symbol' do
+          Roart::Ticket.send(:construct_search_uri, {:status => :new}).should == @query + "Status = 'new'"
+        end
+        
+        it 'should accept an array' do
+          Roart::Ticket.send(:construct_search_uri, {:status => ['new', 'open']}).should == @query + "( Status = 'new' OR Status = 'open' )"
+        end
+        
+      end
+      
+      describe 'custom_fields' do
+        
+        it 'should accept a hash with string values' do
+          Roart::Ticket.send(:construct_search_uri, {:custom_fields => {:phone => '8675309'}}).should == @query + "'CF.{phone}' = '8675309'"
+        end
+
+        it 'should accept a hash with string values' do
+          Roart::Ticket.send(:construct_search_uri, {:custom_fields => {:phone => ['8675309', '5553265']}}).should == @query + "( 'CF.{phone}' = '8675309' OR 'CF.{phone}' = '5553265' )"
+        end
+        
+      end
+      
+      describe 'multiple find options' do
+        
+        it 'should allow multiple find options' do
+          query = Roart::Ticket.send(:construct_search_uri, {:status => 'new', :queue => 'MyQueue'})
+          query.include?("Status = 'new'").should be_true
+          query.include?("Queue = 'MyQueue'").should be_true
+        end
+        
+        it "should search for :queue => 'A Queue', :status => [:new, :open]" do 
+          query = Roart::Ticket.send(:construct_search_uri, {:queue => 'A Queue', :status => [:new, :open]})
+          query.include?("( Status = 'new' OR Status = 'open' )").should be_true
+          query.include?("Queue = 'A Queue'").should be_true
+        end
+        
+      end
+      
+    end  
+    
+  end
+  
+  describe "the default queue" do
+    
+    before do
+      @connection = mock('connection', :server => "server")
+      @search_string = "/REST/1.0/search/ticket?"
+      @query = @connection.server + @search_string + 'query= '
+      class MyTicket < Roart::Ticket; end
+      MyTicket.instance_variable_set("@default_queue", 'myQueue')
+      MyTicket.should_receive(:connection).and_return(@connection)
+    end
+      
+        
+    it 'should have a default queue' do
+      MyTicket.send(:construct_search_uri, :status => :new).should == @query + "Queue = 'myQueue' AND Status = 'new'"
+    end
+    
+    it 'should search without options and return the default queue' do
+      MyTicket.send(:construct_search_uri).should == @query + "Queue = 'myQueue'"
+    end
+  
+  end
+  
+  describe 'ticket methods' do
+    
+    
+    it 'should change the ticket' do
+      ticket = Roart::Ticket.send(:instantiate,{:subject => 'A New Ticket', :queue => 'My Queue', :id => 1})
+      ticket.subject = 'An Old Ticket'
+      ticket.subject.should == 'An Old Ticket'
+    end
+    
+    it 'should be able to load the full ticket' do
+      search_array = ['1:subject']
+      search_array.extend(Roart::TicketPage)
+      full_ticket = Roart::Ticket.send(:instantiate, {:id => 1, :subject => 'subject', :full => true})
+      ticket = Roart::Ticket.send(:instantiate, search_array.to_search_array ).first
+      Roart::Ticket.should_receive(:find).with(1).and_return(full_ticket)
+      ticket.load_full!
+      ticket.full.should be_true
+    end
+    
+  end
+  
+  describe 'manipulating tickets' do
+    
+    describe 'creating tickets' do
+    
+      before do 
+        post_data = @payload = {:subject => 'A New Ticket', :queue => 'My Queue'}
+        post_data.update(:id => 'ticket/new')
+        post_data = to_content_format(post_data)
+        mock_connection = mock('connection')
+        mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/new', {:content => post_data}).and_return("RT/3.6.6 200 Ok\n\n# Ticket 267783 created.")
+        mock_connection.should_receive(:server).and_return('uri')
+        Roart::Ticket.should_receive(:connection).twice.and_return(mock_connection)
+      end
+    
+      it 'should be able to save a new ticket' do
+        ticket = Roart::Ticket.new(@payload)
+        ticket.save
+      end
+      
+      it  'should be able to create a ticket' do
+        ticket = Roart::Ticket.create(@payload)
+      end
+    
+      it 'should return a newly created ticket' do
+        ticket = Roart::Ticket.new(@payload)
+        ticket.class.should == Roart::Ticket
+        ticket.save
+        ticket.id.should == 267783
+      end
+    
+    end
+    
+    describe 'updating tickets' do
+    
+      before do 
+        @post_data = @payload = {:subject => 'A New Ticket', :queue => 'My Queue'}
+        @post_data[:subject] = 'An Old Ticket'
+        @post_data = to_content_format(@post_data)
+        @mock_connection = mock('connection')
+        @mock_connection.should_receive(:server).and_return('uri')
+        Roart::Ticket.should_receive(:connection).twice.and_return(@mock_connection)
+      end
+      
+      it 'should be able to update a ticket' do
+      @mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/1/edit', {:content => @post_data}).and_return("RT/3.6.6 200 Ok\n\n# Ticket 267783 updated.")
+        ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+        ticket.subject = 'An Old Ticket'
+        ticket.save.should == true
+      end
+
+      it 'should keep the same id' do
+      @mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/1/edit', {:content => @post_data}).and_return("RT/3.6.6 200 Ok\n\n# Ticket 267783 updated.")
+        ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+        ticket.subject = 'An Old Ticket'
+        ticket.save
+        ticket.id.should == 1
+      end
+      
+      it 'should save the ticket' do
+      @mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/1/edit', {:content => @post_data}).and_return("RT/3.6.6 200 Ok\n\n# Ticket 267783 updated.")
+        ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+        ticket.subject = 'An Old Ticket'
+        ticket.save
+        ticket.subject.should == 'An Old Ticket'
+      end
+
+      it 'should raise an error on failed save' do
+        @mock_connection.should_receive(:post).with('uri/REST/1.0/ticket/1/edit', {:content => @post_data}).and_return("RT/3.6.6 400 Not OK\n\n# U's A SUKKA, FOO!.")
+        ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+        ticket.subject = 'An Old Ticket'
+        lambda {ticket.save}.should raise_error
+      end
+      
+    end
+  
+  end
+  
+  describe 'histories' do
+    
+    before(:each) do
+      search_array = ['1:subject']
+      search_array.extend(Roart::TicketPage)
+      full_ticket = Roart::Ticket.send(:instantiate, {:id => 1, :subject => 'subject', :full => true})
+      @mock_connection = mock('connection')
+      @mock_connection.should_receive(:get).with('uri').and_return('200')
+      @ticket = Roart::Ticket.send(:instantiate, search_array.to_search_array).first
+      @ticket.class.should_receive(:connection).and_return(@mock_connection)
+      Roart::History.should_receive(:uri_for).with(@ticket).and_return('uri')
+    end
+    
+    it 'should return history objects' do
+      @ticket.histories.class.should == Roart::HistoryArray
+    end
+    
+    it 'should have a default of the ticket id' do
+      @ticket.histories.ticket.should == @ticket
+    end
+    
+    it 'should only spawn 1 dup class for each ticket' do
+      @ticket.histories.should === @ticket.histories
+    end
+    
+    it 'should have a last history that is equal to the last value' do
+      @ticket.histories.last.should == @ticket.histories[@ticket.histories.size - 1]
+    end
+    
+    it 'should have count, which is equal to size' do
+      @ticket.histories.count.should == @ticket.histories.size
+    end
+    
+  end
+  
+end