fiddler 0.0.1 → 1.0.1

data/spec/connection_manager_spec.rb

@@ -6,8 +6,10 @@ describe Fiddler::ConnectionManager do
          reset_config
       end
 
+      use_vcr_cassette "root-request"
+
       it "should raise invalid Configuration error for missing configuration" do
-         expect { Fiddler::ConnectionManager.get("https://www.google.com") }.to raise_error(Fiddler::InvalidConfigurationError)
+         expect { Fiddler::ConnectionManager.get("/") }.to raise_error(Fiddler::InvalidConfigurationError)
       end
 
       it "should raise invalid Configuration error for missing credentials if cookies are not enabled" do
@@ -27,6 +29,8 @@ describe Fiddler::ConnectionManager do
          test_config
       end
 
+      use_vcr_cassette "root-request"
+
       it "should not return nil response for get request" do
          Fiddler::ConnectionManager.get("/").should_not be_nil
       end

data/spec/formatters/search_request_formatter_spec.rb

@@ -14,4 +14,17 @@ describe Fiddler::Formatters::SearchRequestFormatter do
       search_hash = Fiddler::Formatters::SearchRequestFormatter.format( { :owner => "jais.cheema" })
       search_hash.keys.should include(:format)
    end
+
+   it "should return proper query for request" do
+      Fiddler::Formatters::SearchRequestFormatter.format( { :owner => "jais.cheema" })["query"].should eql("Owner LIKE 'jais.cheema'")
+   end
+
+   it "should return proper query for array values for a condition" do
+      Fiddler::Formatters::SearchRequestFormatter.format( { :status => [:open, :resolved] })["query"].should eql("( Status = 'open' OR Status = 'resolved' )")
+   end
+
+   it "should return proper query for mixed conditions" do
+      search_hash = Fiddler::Formatters::SearchRequestFormatter.format( :owner => "jais.cheema", :status => [:open, :resolved], :queue => "Advanced Support")
+      search_hash["query"].should eql("Queue = 'Advanced Support' AND ( Status = 'open' OR Status = 'resolved' ) AND Owner LIKE 'jais.cheema'")
+   end
 end 

data/spec/parsers/attachment_parser_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Fiddler::Parsers::AttachmentParser do
+   before do
+      test_config
+   end
+   
+   it "should return Fiddler::Attachment object for parse single call" do
+      response = Fiddler::ConnectionManager.get("/ticket/3399/attachments/47494")
+      Fiddler::Parsers::AttachmentParser.parse_single(response).should be_a_kind_of(Fiddler::Attachment)
+   end
+
+   it "should be able to handle the image attachments" do
+      response = Fiddler::ConnectionManager.get("/ticket/3399/attachments/57344")
+      Fiddler::Parsers::AttachmentParser.parse_single(response).should be_a_kind_of(Fiddler::Attachment)
+   end
+
+   it "should give me a properly encoded png image for attachment" do
+      response = Fiddler::ConnectionManager.get("/ticket/3399/attachments/57336/content")
+      content = Fiddler::Parsers::AttachmentParser.parse_content(response)
+      File.open("12.jpg","w") { |f| f.write(content) }
+   end
+
+   it "should give me a properly encoded file for any kind of attachment" do
+      response = Fiddler::ConnectionManager.get("/ticket/3399/attachments/57348/content")
+      content = Fiddler::Parsers::AttachmentParser.parse_content(response)
+      File.open("test.graffle","w") { |f| f.write(content) }
+   end
+end

data/spec/parsers/base_parser_spec.rb

@@ -5,6 +5,8 @@ describe Fiddler::Parsers::BaseParser do
       test_config
    end
 
+   use_vcr_cassette "root-request"
+
    it "should not raise error for valid request" do
       response = Fiddler::ConnectionManager.get("/")
       expect { Fiddler::Parsers::BaseParser.check_response_code(response) }.to_not raise_error
@@ -21,4 +23,18 @@ describe Fiddler::Parsers::BaseParser do
 
       Fiddler::Parsers::BaseParser.check_response_code(response).length.should eql(response_array.length-1)
    end
+
+   it "should return proper number of tokens for length response" do
+      response = <<-eos
+some response
+--
+some second response
+--
+some thrid response
+--
+the final response
+      eos
+      response = response.split("\n")
+      Fiddler::Parsers::BaseParser.tokenize_response(response).count.should eql(4)
+   end
 end

data/spec/parsers/history_parser_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe Fiddler::Parsers::HistoryParser do
+
+   before do
+      test_config
+   end
+
+   use_vcr_cassette "get-history"
+
+   it "should return Fiddler::History object for parse single call" do
+      response = Fiddler::ConnectionManager.get("/ticket/4101/history/id/73108")
+      Fiddler::Parsers::HistoryParser.parse_single(response).should be_a_kind_of(Fiddler::History)
+   end
+
+   use_vcr_cassette "ticket-histories"
+
+   it "should return Array of Histories for parse multiple call" do
+      response = Fiddler::ConnectionManager.get("/ticket/4200/history?format=l")
+      histories = Fiddler::Parsers::HistoryParser.parse_multiple(response)
+      histories.should be_a_kind_of(Array)
+      histories.first.should be_a_kind_of(Fiddler::History)
+   end
+
+   use_vcr_cassette "ticket-histories-count"
+
+   it "should return a proper number of histories for a ticket" do
+      response = Fiddler::ConnectionManager.get("/ticket/400/history?format=l")
+      histories = Fiddler::Parsers::HistoryParser.parse_multiple(response)
+      histories.count.should eql(13)
+   end
+end

data/spec/parsers/ticket_parser_spec.rb

@@ -4,6 +4,8 @@ describe Fiddler::Parsers::TicketParser do
    before do
       test_config
    end
+
+   use_vcr_cassette "get-tickets"
    
    it "should return the response for proper request minus empty line" do
       response = Fiddler::ConnectionManager.get("/ticket/4200")
@@ -23,8 +25,10 @@ describe Fiddler::Parsers::TicketParser do
       Fiddler::Parsers::TicketParser.parse_single(response).should be_a_kind_of(Fiddler::Ticket)
    end
 
+   use_vcr_cassette "search-tickets"
+
    it "should give array of ticket objects for parse multiple method" do
-      response = Fiddler::ConnectionManager.get("/search/ticket?query=Owner='jais.cheema'")
+      response = Fiddler::ConnectionManager.get("/search/ticket?query=Owner='jais.cheema'&format=l")
       Fiddler::Parsers::TicketParser.parse_multiple(response).should be_a_kind_of(Array)
    end
 end

data/spec/spec_helper.rb

@@ -1,14 +1,23 @@
 require 'rubygems'
 require 'spork'
 require 'rspec'
+require 'vcr'
 
 PROJECT_ROOT = File.expand_path('../..', __FILE__)
 $LOAD_PATH << File.join(PROJECT_ROOT, 'lib')
 
 require 'fiddler'
 
+VCR.configure do |config|
+  config.cassette_library_dir = 'spec/cassettes'
+  config.hook_into :webmock
+  config.default_cassette_options = { :record => :new_episodes }
+  config.allow_http_connections_when_no_cassette = true
+end
+
 Spork.prefork do
    RSpec.configure do |config|
+      config.extend VCR::RSpec::Macros
       config.treat_symbols_as_metadata_keys_with_true_values = true
       config.run_all_when_everything_filtered = true
       config.filter_run :focus

data/spec/ticket_spec.rb

@@ -4,18 +4,200 @@ describe Fiddler::Ticket do
    before do
       test_config
    end
+
+   describe "Creating tickets" do
+      describe "new empty ticket" do
+         before do
+            @ticket = Fiddler::Ticket.new
+         end
+
+         it "should have a valid id" do
+            @ticket.id.should eql("ticket/new")
+         end
+
+         it "should not be valid" do
+            @ticket.should_not be_valid
+         end
+
+         it "should return false for save" do
+            @ticket.save.should be_false
+         end
+
+         it "should populate the errors hash with the errors" do
+            @ticket.errors.full_messages.should_not be_empty
+         end
+      end
+
+      describe "saving tickets" do
+         before :each do
+            @ticket = Fiddler::Ticket.new
+         end
+
+         it "should show the validation errors" do
+            @ticket.should_not be_valid
+            puts @ticket.errors.full_messages.empty?
+         end
+
+         it "should not run the save method" do
+            @ticket.save.should be_false
+         end
+
+         it "should save the ticket with appropriate attributes set" do
+            @ticket.subject = "Trial ticket"
+            @ticket.queue = "General"
+            @ticket.save.should be_true
+            @ticket.id.to_i.should be_kind_of(Integer)
+         end
+
+         it "should be able to update the ticket without modifying id, ignoring text" do
+            @ticket.subject = "Trial ticket - Update"
+            @ticket.queue = "General"
+            @ticket.save.should be_true
+            @ticket.id.to_i.should be_kind_of(Integer)
+            @ticket.subject = "Update Subject after saving the ticket"
+            @ticket.save.should be_true
+         end
+
+         it "should create a ticket with initial commit" do
+            @ticket.subject = "Test ticket with text and requestor"
+            @ticket.queue = "General"
+            @ticket.requestors = "jais.cheema@cybersecure.com.au"
+            @ticket.text = "Creating ticket with requestor and text"
+            @ticket.save.should be_true
+         end
+      end
+   end
+
+   describe "updating tickets" do
+      it "should update the ticket requestors properly" do
+         t = Fiddler::Ticket.get(5234)
+         requestors = "#{t.requestors}, rene@cybersecure.com.au"
+         t.requestors = requestors
+         t.save.should be_true
+         t = Fiddler::Ticket.get(5234)
+         t.requestors.should eql(requestors)
+      end
+   end
+
+   describe "managing requestors" do
+      it "should return array of requestors" do
+         t = Fiddler::Ticket.get(5234)
+         requestors = t.requestors
+         t.requestor_array.should be_a_kind_of(Array)
+         t.requestor_array.should eql(requestors.split(", ").collect { |x| x.strip} )
+      end
+
+      it "should add in a proper requestors" do
+         t = Fiddler::Ticket.get(5233)
+         t.requestor_array = ["jais.cheema@cybersecure.com.au", "rene@cybersecure.com.au"]
+         t.save.should be_true
+         t = Fiddler::Ticket.get(5233)
+         t.requestors.should eql("jais.cheema@cybersecure.com.au, rene@cybersecure.com.au")
+      end
+   end
    
-   it "should find a ticket with given id" do
-      Fiddler::Ticket.get(4200).should be_a_kind_of(Fiddler::Ticket)
+   describe "Replying to tickets" do
+
+      #use_vcr_cassette "reply-to-tickets"
+
+      it "should comment on a ticket" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.comment("nice comment").should be_true
+      end
+
+      it "should purge any useless options given with comment" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.comment("nice comment", :useless => true).should be_true
+      end
+
+      it "should be able to correspond" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.correspond("nice email").should be_true
+      end
+
+      it "should be able to comment with options" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.correspond("nice email", :cc => "jaischeema@gmail.com", :time_worked => 10, :status => :open).should be_true
+      end
+
+      it "should update the status of the ticket if given in options" do
+         ticket = Fiddler::Ticket.get(3300)
+         ticket.comment("Something right here", :status => :stalled)
+         ticket.status.should eql(:stalled)
+      end
    end
 
-   it "should raise exception for invalid id" do
-      expect { Fiddler::Ticket.get(50000) }.to raise_error(Fiddler::TicketNotFoundError)
+   describe "Changing ownership" do
+
+      use_vcr_cassette "change-ownership-steal"
+
+      it "should change the ownership to the current user for steal" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.steal.should_not be_nil
+         ticket.owner.should eql(Fiddler.configuration.username)
+      end
+
+      use_vcr_cassette "change-ownership-untake"
+
+      it "should change the ownership to the Nobody user for untake" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.untake.should_not be_nil
+         ticket.owner.should eql("Nobody")
+      end
+
+      use_vcr_cassette "change-ownership-take"
+
+      it "should change the ownership to the current user for take" do
+         ticket = Fiddler::Ticket.get(4200)
+         ticket.take.should_not be_nil
+         ticket.owner.should eql(Fiddler.configuration.username)
+      end
    end
 
-   describe "executing all method" do
-      it "should return all the user assigned tickets for empty conditions" do
+   describe "searching tickets" do
+      
+      use_vcr_cassette "get-tickets"
+
+      it "should find a ticket with given id" do
+         ticket = Fiddler::Ticket.get(400)
+         ticket.should be_a_kind_of(Fiddler::Ticket)
+         ticket.id.should eql(400)
+      end
+
+      it "should raise exception for invalid id" do
+         expect { Fiddler::Ticket.get(50000) }.to raise_error(Fiddler::TicketNotFoundError)
+      end
+
+      it "should return empty array for empty conditions" do
          Fiddler::Ticket.all.should be_a_kind_of(Array)
+         Fiddler::Ticket.all.length.should eql(0)
+      end
+
+      it "should return all tickets owned by the user for owner query" do
+         tickets = Fiddler::Ticket.all(:owner => "jais.cheema")
+         tickets.each do |ticket|
+            ticket.owner.should eql("jais.cheema")
+         end
+      end
+
+      it "should be able to handle mutliple values for a condition" do
+         tickets = Fiddler::Ticket.all( :status => [:open, :resolved ])
+         tickets.each do |ticket|
+            ticket.status.should match(/open|resolved/)
+         end 
+      end
+   end
+
+   describe "histories" do
+      before do
+         @ticket = Fiddler::Ticket.get(4200)
+      end
+
+      use_vcr_cassette "ticket-histories"
+
+      it "should return an array of history items" do
+         @ticket.histories.should be_a_kind_of(Array)
+         @ticket.histories.first.should be_a_kind_of(Fiddler::History)
       end
    end
 end

metadata

@@ -2,7 +2,7 @@
 name: fiddler
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.1
+  version: 1.0.1
 platform: ruby
 authors: 
   - Jais Cheema
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-10-04 00:00:00 Z
+date: 2012-12-18 00:00:00 Z
 dependencies: 
   - !ruby/object:Gem::Dependency 
     name: httpclient
@@ -29,13 +29,13 @@ dependencies:
     requirement: &id002 !ruby/object:Gem::Requirement 
       none: false
       requirements: 
-        - - "="
+        - - ">="
           - !ruby/object:Gem::Version 
-            version: 3.2.8
+            version: "0"
     type: :runtime
     version_requirements: *id002
   - !ruby/object:Gem::Dependency 
-    name: rspec
+    name: awesome_print
     prerelease: false
     requirement: &id003 !ruby/object:Gem::Requirement 
       none: false
@@ -43,10 +43,10 @@ dependencies:
         - - ">="
           - !ruby/object:Gem::Version 
             version: "0"
-    type: :development
+    type: :runtime
     version_requirements: *id003
   - !ruby/object:Gem::Dependency 
-    name: spork
+    name: active_attr
     prerelease: false
     requirement: &id004 !ruby/object:Gem::Requirement 
       none: false
@@ -54,10 +54,10 @@ dependencies:
         - - ">="
           - !ruby/object:Gem::Version 
             version: "0"
-    type: :development
+    type: :runtime
     version_requirements: *id004
   - !ruby/object:Gem::Dependency 
-    name: watchr
+    name: rspec
     prerelease: false
     requirement: &id005 !ruby/object:Gem::Requirement 
       none: false
@@ -67,6 +67,50 @@ dependencies:
             version: "0"
     type: :development
     version_requirements: *id005
+  - !ruby/object:Gem::Dependency 
+    name: spork
+    prerelease: false
+    requirement: &id006 !ruby/object:Gem::Requirement 
+      none: false
+      requirements: 
+        - - ">="
+          - !ruby/object:Gem::Version 
+            version: "0"
+    type: :development
+    version_requirements: *id006
+  - !ruby/object:Gem::Dependency 
+    name: watchr
+    prerelease: false
+    requirement: &id007 !ruby/object:Gem::Requirement 
+      none: false
+      requirements: 
+        - - ">="
+          - !ruby/object:Gem::Version 
+            version: "0"
+    type: :development
+    version_requirements: *id007
+  - !ruby/object:Gem::Dependency 
+    name: webmock
+    prerelease: false
+    requirement: &id008 !ruby/object:Gem::Requirement 
+      none: false
+      requirements: 
+        - - ">="
+          - !ruby/object:Gem::Version 
+            version: "0"
+    type: :development
+    version_requirements: *id008
+  - !ruby/object:Gem::Dependency 
+    name: vcr
+    prerelease: false
+    requirement: &id009 !ruby/object:Gem::Requirement 
+      none: false
+      requirements: 
+        - - ">="
+          - !ruby/object:Gem::Version 
+            version: "0"
+    type: :development
+    version_requirements: *id009
 description: This is the interface to request tracker based on Roart Gem
 email: 
   - jaischeema@gmail.com
@@ -79,6 +123,8 @@ extra_rdoc_files: []
 files: 
   - lib/fiddler.rb
   - lib/tasks/fiddler_tasks.rake
+  - lib/fiddler/attachment.rb
+  - lib/fiddler/extensions.rb
   - lib/fiddler/version.rb
   - lib/fiddler/errors.rb
   - lib/fiddler/formatters.rb
@@ -86,18 +132,33 @@ files:
   - lib/fiddler/parsers.rb
   - lib/fiddler/configuration.rb
   - lib/fiddler/ticket.rb
+  - lib/fiddler/history.rb
   - lib/fiddler/parsers/ticket_parser.rb
+  - lib/fiddler/parsers/history_parser.rb
   - lib/fiddler/parsers/base_parser.rb
+  - lib/fiddler/parsers/attachment_parser.rb
+  - lib/fiddler/extensions/file.rb
+  - lib/fiddler/extensions/hash.rb
   - lib/fiddler/formatters/search_request_formatter.rb
   - lib/fiddler/formatters/base_formatter.rb
   - spec/spec_helper.rb
   - spec/ticket_spec.rb
+  - spec/attachment_spec.rb
   - spec/configuration_spec.rb
   - spec/config.rb
   - spec/connection_manager_spec.rb
   - spec/formatters/search_request_formatter_spec.rb
   - spec/formatters/base_formatter_spec.rb
+  - spec/cassettes/ticket-histories.yml
+  - spec/cassettes/search-tickets.yml
+  - spec/cassettes/get-tickets.yml
+  - spec/cassettes/root-request.yml
+  - spec/cassettes/reply-to-tickets.yml
+  - spec/cassettes/change-ownership-take.yml
+  - spec/cassettes/ticket-histories-count.yml
   - spec/parsers/base_parser_spec.rb
+  - spec/parsers/history_parser_spec.rb
+  - spec/parsers/attachment_parser_spec.rb
   - spec/parsers/ticket_parser_spec.rb
   - Rakefile
   - README.md
@@ -131,10 +192,20 @@ summary: Interface to Request Tracker based on Roart Gem
 test_files: 
   - spec/spec_helper.rb
   - spec/ticket_spec.rb
+  - spec/attachment_spec.rb
   - spec/configuration_spec.rb
   - spec/config.rb
   - spec/connection_manager_spec.rb
   - spec/formatters/search_request_formatter_spec.rb
   - spec/formatters/base_formatter_spec.rb
+  - spec/cassettes/ticket-histories.yml
+  - spec/cassettes/search-tickets.yml
+  - spec/cassettes/get-tickets.yml
+  - spec/cassettes/root-request.yml
+  - spec/cassettes/reply-to-tickets.yml
+  - spec/cassettes/change-ownership-take.yml
+  - spec/cassettes/ticket-histories-count.yml
   - spec/parsers/base_parser_spec.rb
+  - spec/parsers/history_parser_spec.rb
+  - spec/parsers/attachment_parser_spec.rb
   - spec/parsers/ticket_parser_spec.rb