ludo-roart 0.1.11

data/spec/roart/core/hash_spec.rb

@@ -0,0 +1,22 @@
+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
+
+  it 'should use our content formatter for strings' do
+    payload = {:subject => 'A new ticket', :queue => 'My queue', :text => "A text"}
+    Roart::ContentFormatter.should_receive(:format_string).at_least(:once)
+    payload.to_content_format
+  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,93 @@
+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 'reading a ticket' do
+		
+		before do
+			@page = File.open(File.join(File.dirname(__FILE__), %w[ .. test_data ticket.txt])).readlines.join
+			@page = @page.split("\n")
+      @page.extend(Roart::TicketPage)
+		end
+		
+		it 'should include custom fields' do
+			@page.to_hash[:cf_BTN].should == '1035328269'
+		end
+		
+		it 'should be a hash' do
+			@page.to_hash.class.should == HashWithIndifferentAccess
+		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 "search list array" do
+	  before do
+			@array = [['id:234', 'subject:SomeTicket', ], ['id:432','subject:Another']]
+      @array.extend(Roart::TicketPage)
+      @array = @array.to_search_list_array
+		end
+		
+		it "should return an array of hashes" do
+		  @array.first.class.should == HashWithIndifferentAccess
+		end
+		
+		it "should put the search elements into the array" do
+			@array.first[: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 == HashWithIndifferentAccess
+    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,553 @@
+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 => 'some options', :adapter => "www").and_return(true)
+    Roart::Ticket.connection(:some => 'some options', :adapter => "www").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_list_array).with('uri&format=l').and_return([['id:234', 'subject:SomeTicket', ], ['id:432','subject:Another']])
+        Roart::Ticket.send(:get_tickets_from_search_uri, 'uri').size.should == 2
+      end
+      
+      it 'should have full tickets' do
+        Roart::Ticket.should_receive(:page_list_array).with('uri&format=l').and_return([['id:234', 'subject:SomeTicket', ], ['id:432','subject:Another']])
+        Roart::Ticket.send(:get_tickets_from_search_uri, 'uri').each do |ticket|
+          ticket.full.should be_true
+        end
+      end
+
+			it 'should give back an array of arrays of strings' do
+				test_data = File.open(File.dirname(__FILE__) + "/../test_data/search_ticket.txt").readlines.join("\n")
+				connection = mock('connection', :get => test_data)
+	      Roart::Ticket.should_receive(:connection).and_return(connection)
+				Roart::Ticket.send(:page_list_array, 'uri').should == [["id: ticket/3033", "Queue: Canada Express", "Subject: Canada Express", "Status: new"], ["id: ticket/2354", "Queue: Canada Express", "Subject: Canada Express", "Status: open"]]
+			end
+			
+			describe ' search with no matches' do
+    	  it 'should give an empty array for :all' do
+    	    results = 'RT/3.6.6 200 Ok
+
+          No matching results.'
+  				connection = mock('connection', :server => 'server', :get => results)
+  	      Roart::Ticket.should_receive(:connection).and_return(connection)
+    	    Roart::Ticket.send(:get_tickets_from_search_uri, 'uri').should == []
+        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
+      
+      it 'should raise when searching for an id with no results' do
+        results = 'RT/3.6.6 200 Ok
+
+        No matching results.'
+        connection = mock('connection', :server => 'server', :get => results)
+        Roart::Ticket.stub!(:connection).and_return(connection)
+        lambda {Roart::Ticket.send(:get_ticket_by_id, 123)}.should raise_error("No ticket matching search criteria found.")
+      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 = {:queue => 'My Queue', :subject => 'A New Ticket'}
+        post_data.update(:id => 'ticket/new')
+        post_data = to_content_format(post_data)
+        mock_connection = mock('connection')
+        Roart::Ticket.should_receive(:connection).twice.and_return(mock_connection)
+        mock_connection.should_receive(:server).and_return('uri')
+        mock_connection.should_receive(:post).
+          with('uri/REST/1.0/ticket/new', {:content => post_data}).
+          and_return(response_body)
+      end
+
+      let(:ticket) { Roart::Ticket.new(@payload) }
+
+      context "on success" do
+        let(:response_body) { "RT/3.6.6 200 Ok\n\n# Ticket 267783 created." }
+
+        it 'should be able to save a new ticket' do
+          ticket.new_record?.should be_true
+          ticket.save
+          ticket.new_record?.should be_false
+        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.class.should == Roart::Ticket
+          ticket.save
+          ticket.id.should == 267783
+        end
+
+        it 'should NOT have errors' do
+          ticket.save
+          ticket.errors.should be_empty
+        end
+
+        it 'should be saved' do
+          ticket.save
+          ticket.saved.should == true
+        end
+      end
+
+      context "on failure" do
+        let(:response_body) { "RT/4.0.5 200 Ok\n\n# Could not create ticket.\n# Status 'huj' isn't a valid status for tickets in this queue." }
+
+        it 'should return an unsaved ticket' do
+          ticket.class.should == Roart::Ticket
+          ticket.save
+          ticket.id.should == 'ticket/new'
+          ticket.should be_new_record
+        end
+
+        it 'should return false' do
+          ticket.save.should == false
+        end
+
+        it 'should have errors' do
+          ticket.save
+          ticket.errors.size.should == 1
+          ticket.errors[:base].should == ["# Status 'huj' isn't a valid status for tickets in this queue."]
+        end
+      end
+    
+    end
+    
+    describe 'updating tickets' do
+    
+      before do 
+        @payload = {:queue => 'My Queue', :subject => 'An Old Ticket'}.with_indifferent_access
+        @post_data = {:queue => 'My Queue', :subject => 'An Old Ticket'}.with_indifferent_access
+        @payload[:subject] = 'A New 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)
+        @mock_connection.should_receive(:post).
+          with('uri/REST/1.0/ticket/1/edit', {:content => @post_data}).
+          and_return(response_body)
+      end
+      
+      context "on success" do
+        let(:response_body) { "RT/3.6.6 200 Ok\n\n# Ticket 267783 updated." }
+
+        it 'should be able to update a ticket' do
+          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
+          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
+          ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+          ticket.subject = 'An Old Ticket'
+          ticket.save
+          ticket.subject.should == 'An Old Ticket'
+        end
+      end
+
+      context "on 502 failure" do
+        let(:response_body) { "RT/3.6.6 502 Not OK\n\n# Dead gateway" }
+
+        it 'should raise an error on failed save' do
+          ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+          ticket.subject = 'An Old Ticket'
+          expect { ticket.save }.to raise_error(Roart::TicketSystemError)
+        end
+      end
+      
+      context "on 200 failure" do
+        let(:response_body) { "RT/3.6.6 200 Probably OK\n\n# Could not update ticket\n# Queue is missing" }
+
+        it 'should set errors' do
+          ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+          ticket.subject = 'An Old Ticket'
+          expect { ticket.save }.not_to raise_error
+          ticket.errors.size.should == 1
+          ticket.errors[:base].should == ["# Queue is missing"]
+        end
+      end
+
+      context "on 400 failure" do
+        let(:response_body) { "RT/3.6.6 400 Not OK\n\n# U's A SUKKA, FOO!." }
+
+        it 'should set errors' do
+          ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+          ticket.subject = 'An Old Ticket'
+          expect { ticket.save }.not_to raise_error
+          ticket.errors.size.should == 1
+          ticket.errors[:base].should == ["# U's A SUKKA, FOO!."]
+        end
+      end
+
+      context "on 409 failure" do
+        let(:response_body) { "RT/4.0.5 409 Syntax Error\n# Ticket 13 updated.\n# status2: Unknown field.\n\nid: \nStatus2: huj"}
+
+        it 'should set errors' do
+          ticket = Roart::Ticket.send(:instantiate, @payload.update(:id => 1))
+          ticket.subject = 'An Old Ticket'
+          expect { ticket.save }.not_to raise_error
+          ticket.errors.size.should == 2
+          ticket.errors[:base].should == [
+            "# Ticket 13 updated.",
+            "# status2: Unknown field."
+          ]
+        end
+      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