ludo-roart 0.1.14 → 0.1.15

data/Gemfile

@@ -0,0 +1,9 @@
+source :rubygems
+
+gemspec
+
+gem 'rspec', '2.8.0'
+
+gem 'activesupport', '3.0.12'
+
+gem 'i18n'

data/Gemfile.lock

@@ -0,0 +1,57 @@
+PATH
+  remote: .
+  specs:
+    roart (0.1.9)
+      activesupport (>= 2.0.0)
+      mechanize (>= 1.0.0)
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    activesupport (3.0.12)
+    bones (3.8.0)
+      little-plugger (~> 1.1.3)
+      loquacious (~> 1.9.1)
+      rake (>= 0.8.7)
+    diff-lcs (1.1.3)
+    domain_name (0.5.2)
+      unf (~> 0.0.3)
+    i18n (0.6.0)
+    little-plugger (1.1.3)
+    loquacious (1.9.1)
+    mechanize (2.3)
+      domain_name (~> 0.5, >= 0.5.1)
+      mime-types (~> 1.17, >= 1.17.2)
+      net-http-digest_auth (~> 1.1, >= 1.1.1)
+      net-http-persistent (~> 2.5, >= 2.5.2)
+      nokogiri (~> 1.4)
+      ntlm-http (~> 0.1, >= 0.1.1)
+      webrobots (~> 0.0, >= 0.0.9)
+    mime-types (1.17.2)
+    net-http-digest_auth (1.2)
+    net-http-persistent (2.5.2)
+    nokogiri (1.5.2)
+    ntlm-http (0.1.1)
+    rake (0.9.2.2)
+    rspec (2.8.0)
+      rspec-core (~> 2.8.0)
+      rspec-expectations (~> 2.8.0)
+      rspec-mocks (~> 2.8.0)
+    rspec-core (2.8.0)
+    rspec-expectations (2.8.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.8.0)
+    unf (0.0.5)
+      unf_ext
+    unf_ext (0.0.4)
+    webrobots (0.0.13)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  activesupport (= 3.0.12)
+  bones (>= 2.5.1)
+  i18n
+  roart!
+  rspec (= 2.8.0)

data/lib/roart/attachment.rb

@@ -0,0 +1,37 @@
+module Roart
+
+  class AttachmentCollection < Array
+    def to_payload
+      hash = Hash.new
+      self.each_with_index do |attach, index|
+        hash["attachment_#{index+1}"] = attach.file
+      end
+      hash
+    end
+  end
+
+  class Attachment < Struct.new(:name, :file)
+
+    def path
+      name
+    end
+
+    def read
+      file.read
+    end
+
+    def self.detect(*args)
+      AttachmentCollection.new Array(args.compact).flatten.map { |file|
+        if file.is_a?(File)
+          Attachment.new(File.basename(file.path), file)
+        elsif file.is_a?(String)
+          Attachment.new(File.basename(file), File.open(file, 'rb'))
+        elsif file.respond_to?(:open, :original_filename)
+          Attachment.new(file.original_filename, file.open)
+        end
+      }.compact
+    end
+
+  end
+
+end

data/lib/roart/core/content_formatter.rb

@@ -0,0 +1,12 @@
+module Roart
+  class ContentFormatter
+    # The following is only based on quick trial&error on my part. The RT Api (at least in 3.6) does not have good documentation.
+    # Strings in a RT request:
+    #  - are not allowed to contain '\r'
+    #  - must use equally padded new-lines to distinguish them from the beginning of a new field (we use 2-space padding)
+    def self.format_string(string)
+      string.gsub("\r", '').gsub("\n", "\n  ")
+    end
+
+  end
+end

data/lib/roart/ticket.rb

@@ -78,7 +78,7 @@ module Roart
     #   tix.comment("This is a comment", :attachments => [params[:attachment_1], params[:attachment_2]])
     #
     def comment(comment, opt = {})
-      comment = {:text => comment, :action => 'Correspond'}.merge(opt)
+      comment = {:text => comment, :action => 'Comment'}.merge(opt)
 
       uri = "#{self.class.connection.server}/REST/1.0/ticket/#{self.id}/comment"
 

data/roart.gemspec

@@ -2,15 +2,16 @@
 
 Gem::Specification.new do |s|
   s.name = %q{ludo-roart}
-  s.version = "0.1.14"
+  s.version = "0.1.15"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["PJ Davis"]
   s.date = %q{2010-09-15}
   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/search_ticket.txt", "spec/test_data/single_history.txt", "spec/test_data/ticket.txt"]
-  s.files = ["History.txt", "README.rdoc", "Rakefile", "lib/roart.rb", "lib/roart/callbacks.rb", "lib/roart/connection.rb", "lib/roart/connection_adapter.rb", "lib/roart/connection_adapters/mechanize_adapter.rb", "lib/roart/core/hash.rb", "lib/roart/errors.rb", "lib/roart/history.rb", "lib/roart/roart.rb", "lib/roart/ticket.rb", "lib/roart/ticket_page.rb", "lib/roart/validations.rb", "roart.gemspec", "spec/roart/callbacks_spec.rb", "spec/roart/connection_adapter_spec.rb", "spec/roart/connection_spec.rb", "spec/roart/core/hash_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/validation_spec.rb", "spec/roart_spec.rb", "spec/spec_helper.rb", "spec/test_data/full_history.txt", "spec/test_data/search_ticket.txt", "spec/test_data/single_history.txt", "spec/test_data/ticket.txt"]
+  s.extra_rdoc_files = Dir['{lib,spec}/**/*.txt', "[a-zA-Z]*.txt", "[a-zA-Z]*.rdoc"]
+  s.files = Dir['{lib,spec}/**/*', "[a-zA-Z]*"]
+  s.test_files = Dir['spec/**/*']
   s.homepage = %q{http://github.com/pjdavis/roart}
   s.rdoc_options = ["--main", "README.rdoc"]
   s.require_paths = ["lib"]

data/spec/roart/attachment_spec.rb

@@ -0,0 +1,65 @@
+require File.join(File.dirname(__FILE__), %w[ .. spec_helper])
+
+describe Roart::Attachment do
+
+  context "instance" do
+    subject { described_class.new("zero", File.open("/dev/zero", "rb")) }
+
+    it { should respond_to(:path) }
+    it { should respond_to(:read) }
+    it { should respond_to(:name) }
+  end
+
+  describe ".detect" do
+
+    it "should create collection from attachments array" do
+      described_class.detect(['/dev/null', '/dev/zero']).should be_a(Enumerable)
+      described_class.detect(['/dev/null', '/dev/zero']).first.should be_a(Roart::Attachment)
+    end
+
+    it "should create collection from attachments as params" do
+      described_class.detect('/dev/null', '/dev/zero').should be_a(Enumerable)
+      described_class.detect(['/dev/null', '/dev/zero']).first.should be_a(Roart::Attachment)
+    end
+
+    it "should create collection from one attachment" do
+      described_class.detect('/dev/null').should be_a(Enumerable)
+      described_class.detect(['/dev/null', '/dev/zero']).first.should be_a(Roart::Attachment)
+    end
+
+    it "should create using String filename" do
+      @attachment = described_class.detect('/dev/null').first
+      @attachment.file.should be_a(File)
+    end
+
+    it "should create using File descriptor" do
+      @attachment = described_class.detect(File.open('/dev/null', 'rb')).first
+      @attachment.file.should be_a(File)
+    end
+
+    it "should create using Tempfile instance" do
+      @tempfile = mock(:tempfile, :original_filename => "test name", :open => "file descriptor")
+      @attachment = described_class.detect(@tempfile).first
+      @attachment.file.should == "file descriptor"
+    end
+
+  end
+
+  describe Roart::AttachmentCollection, ".to_payload" do
+    subject { Roart::Attachment.detect(['/dev/null', '/dev/zero']) }
+
+    it "should return a Hash" do
+      subject.to_payload.should be_a(Hash)
+    end
+
+    it "should generate keys for attachments" do
+      subject.to_payload.should have_key("attachment_1")
+      subject.to_payload.should have_key("attachment_2")
+    end
+
+    it "should contain file descriptors" do
+      subject.to_payload["attachment_1"].should be_a(File)
+    end
+  end
+
+end

data/spec/roart/core/content_formatter_spec.rb

@@ -0,0 +1,23 @@
+require File.join(File.dirname(__FILE__), %w[ .. .. spec_helper])
+
+describe 'content formatter' do
+  before do
+    @formatter = Roart::ContentFormatter
+  end
+
+  describe 'formatting strings' do
+    it "should remove \\r characters" do
+      @formatter.format_string("a\r\ntext with\ndifferent\rnew\r\nlines").should_not include("\r")
+    end
+
+    it "should pad new lines with 2 spaces" do
+      @formatter.format_string("a\ntext with\n\ndifferent\nnew\n  \nlines").should == "a\n" +
+                                                                                      "  text with\n" +
+                                                                                      "  \n" +
+                                                                                      "  different\n" +
+                                                                                      "  new\n" +
+                                                                                      "    \n" +
+                                                                                      "  lines"
+    end
+  end
+end

data/spec/test_data/lorem.txt

@@ -0,0 +1 @@
+Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ludo-roart
 version: !ruby/object:Gem::Version 
-  hash: 7
+  hash: 5
   prerelease: 
   segments: 
   - 0
   - 1
-  - 14
-  version: 0.1.14
+  - 15
+  version: 0.1.15
 platform: ruby
 authors: 
 - PJ Davis
@@ -72,21 +72,20 @@ executables: []
 extensions: []
 
 extra_rdoc_files: 
-- History.txt
-- README.rdoc
 - spec/test_data/full_history.txt
+- spec/test_data/lorem.txt
 - spec/test_data/search_ticket.txt
 - spec/test_data/single_history.txt
 - spec/test_data/ticket.txt
-files: 
 - History.txt
 - README.rdoc
-- Rakefile
-- lib/roart.rb
+files: 
+- lib/roart/attachment.rb
 - lib/roart/callbacks.rb
 - lib/roart/connection.rb
 - lib/roart/connection_adapter.rb
 - lib/roart/connection_adapters/mechanize_adapter.rb
+- lib/roart/core/content_formatter.rb
 - lib/roart/core/hash.rb
 - lib/roart/errors.rb
 - lib/roart/history.rb
@@ -94,10 +93,12 @@ files:
 - lib/roart/ticket.rb
 - lib/roart/ticket_page.rb
 - lib/roart/validations.rb
-- roart.gemspec
+- lib/roart.rb
+- spec/roart/attachment_spec.rb
 - spec/roart/callbacks_spec.rb
 - spec/roart/connection_adapter_spec.rb
 - spec/roart/connection_spec.rb
+- spec/roart/core/content_formatter_spec.rb
 - spec/roart/core/hash_spec.rb
 - spec/roart/history_spec.rb
 - spec/roart/roart_spec.rb
@@ -107,9 +108,16 @@ files:
 - spec/roart_spec.rb
 - spec/spec_helper.rb
 - spec/test_data/full_history.txt
+- spec/test_data/lorem.txt
 - spec/test_data/search_ticket.txt
 - spec/test_data/single_history.txt
 - spec/test_data/ticket.txt
+- Gemfile
+- Gemfile.lock
+- History.txt
+- Rakefile
+- README.rdoc
+- roart.gemspec
 homepage: http://github.com/pjdavis/roart
 licenses: []
 
@@ -144,5 +152,22 @@ rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: Interface for working with Request Tracker (RT) tickets inspired by ActiveRecord
-test_files: []
-
+test_files: 
+- spec/roart/attachment_spec.rb
+- spec/roart/callbacks_spec.rb
+- spec/roart/connection_adapter_spec.rb
+- spec/roart/connection_spec.rb
+- spec/roart/core/content_formatter_spec.rb
+- spec/roart/core/hash_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/validation_spec.rb
+- spec/roart_spec.rb
+- spec/spec_helper.rb
+- spec/test_data/full_history.txt
+- spec/test_data/lorem.txt
+- spec/test_data/search_ticket.txt
+- spec/test_data/single_history.txt
+- spec/test_data/ticket.txt