taskmapper-redmine 0.5.0 → 0.5.1

data/VERSION

@@ -1 +1 @@
-0.5.0
+0.5.1

data/lib/provider/comment.rb

@@ -23,6 +23,10 @@ module TaskMapper::Provider
         end
       end
 
+      def save
+        raise TaskMapper::Exception.new "Redmine API doesn't support comment updates"
+      end
+
       def self.find_by_id(project_id, ticket_id, id)
         self.search(ticket_id).select { |journal| journal.id == id.to_i }.first
       end
@@ -51,5 +55,14 @@ module TaskMapper::Provider
         journal
       end
     end
+
+    class Net::HTTP
+      def send(*args)
+        puts "request: #{args.inspect}"
+        resp = super 
+        puts "response: #{resp.inspect}"
+        resp
+      end
+    end
   end
 end

data/spec/comments_spec.rb

@@ -5,67 +5,82 @@ describe TaskMapper::Provider::Redmine::Comment do
   let(:project_id) { 1 }
   let(:ticket_id)  { 1 }
   let(:comment_id) { 1 }
-  let(:klass)      { TaskMapper::Provider::Redmine::Comment }
-  let(:taskmapper) { TaskMapper.new :redmine, :server => 'http://demo.redmine.org/', :token => 'abcdefghijk' }
-  let(:project)    { taskmapper.project(project_id) }
-
-  before(:all) do
-    headers = {'X-Redmine-API-Key' => 'abcdefghijk', 'Accept' => 'application/xml'}
-    headers_post_put = {'X-Redmine-API-Key' => 'abcdefghijk', 'Content-Type' => 'application/xml'}
-    ActiveResource::HttpMock.respond_to do |mock|
-      mock.get '/projects.xml', headers, fixture_for('projects'), 200
-      mock.get '/projects/1.xml', headers, fixture_for('projects/test-repo12'), 200
-      mock.get '/issues.xml?project_id=1', headers, fixture_for('issues'), 200
-      mock.get '/issues.xml', headers, fixture_for('issues'), 200
-      mock.get '/issues/1.xml?include=journals', headers, fixture_for('issues/1'), 200
-      mock.put '/issues/1.xml', headers_post_put, '', 200
-      mock.post '/issues.xml', headers_post_put, '', 200
-    end
-  end
+  let(:comment_class) { TaskMapper::Provider::Redmine::Comment }
+  let(:tm) { TaskMapper.new :redmine, :server => 'http://demo.redmine.org/', :token => 'abcdefghijk' }
+  let(:headers_auth) { {'X-Redmine-API-Key' => 'abcdefghijk' } }
+  let(:headers_get) { headers_auth.merge('Accept' => 'application/xml') }
+  let(:headers_post) { headers_auth.merge('Content-Type' => 'application/xml') }
 
   before(:each) do
-    @ticket = project.ticket(ticket_id)
   end
 
-  it "should be able to load all comments" do
-    comments = @ticket.comments
-    comments.size.should eq(2)
-    comments.should be_an_instance_of(Array)
-    comments.first.should be_an_instance_of(klass)
-  end
-  
-  it "should be able to load all comments based on 'id's" do
-    comments = @ticket.comments([comment_id])
-    comments.should be_an_instance_of(Array)
-    comments.first.should be_an_instance_of(klass)
-    comments.first.id.should == comment_id
-  end
-  
-  it "should be able to load all comments based on attributes" do
-    comments = @ticket.comments(:id => comment_id)
-    comments.should be_an_instance_of(Array)
-    comments.first.should be_an_instance_of(klass)
-  end
-  
-  it "should be able to load a comment based on id" do
-    comment = @ticket.comment(comment_id)
-    comment.should be_an_instance_of(klass)
-    comment.id.should == comment_id
-  end
-  
-  it "should be able to load a comment based on attributes" do
-    comment = @ticket.comment(:id => comment_id)
-    comment.should be_an_instance_of(klass)
-  end
-  
-  it "should return the class" do
-    @ticket.comment.should == klass
+  describe "Retrieving comments" do 
+    before(:each) do 
+      ActiveResource::HttpMock.respond_to do |mock|
+        mock.get '/projects/1.xml', headers_get, fixture_for('projects/test-repo12'), 200
+        mock.get '/issues/1.xml?include=journals', headers_get, fixture_for('issues/1'), 200
+      end
+    end
+    let(:project) { tm.project project_id }
+    let(:ticket) { project.ticket ticket_id }
+
+    context "when #comments" do 
+      subject { ticket.comments } 
+      it { should be_an_instance_of Array }
+      it { should have(2).items }
+    end
+
+    context "when #comments with a list of id's" do 
+      subject { ticket.comments [comment_id] }
+      it { should be_an_instance_of Array }
+      it { should have(1).items }
+    end
+
+    context "when #comments with a comment attribute" do 
+      subject { ticket.comments :id => comment_id }
+      it { should be_an_instance_of Array }
+      it { should have(1).items }
+    end
+
+    describe "Retrieve a single comment" do 
+      context "when #comment with an id" do 
+        subject { ticket.comment comment_id } 
+        it { should be_an_instance_of comment_class }
+      end
+
+      context "when #comment with an attribute" do 
+        subject { ticket.comment :id => comment_id }
+        it { should be_an_instance_of comment_class }
+      end
+    end
   end
-  
-  it "should be able to create a comment for a given task" do
-    comment = @ticket.comment!(:body => 'hello there boys and girls')
-    comment.should be_an_instance_of(klass)
-    comment.ticket_id.should_not be_nil
-    comment.ticket_id.should_not == 0
+
+  describe "Create and update" do 
+    before(:each) do 
+      ActiveResource::HttpMock.respond_to do |mock|
+        mock.get '/projects/1.xml', headers_get, fixture_for('projects/test-repo12'), 200
+        mock.get '/issues/1.xml?include=journals', headers_get, fixture_for('issues/1'), 200
+        mock.put '/issues/1.xml', headers_post, '', 200
+      end
+    end
+    let(:project) { tm.project project_id }
+    let(:ticket) { project.ticket ticket_id }
+
+    context "when #comment!" do 
+      subject { ticket.comment! :body => 'hello there boys and girls' }
+      it { should be_an_instance_of comment_class }
+      context "when #ticket_id" do 
+        subject { ticket.comment!(:body => 'hello there boys and girls').ticket_id }
+        it { should_not be_nil }
+        it { should_not be_zero }
+      end
+    end
+
+    context "when #save" do 
+      let(:comment) { ticket.comment(comment_id) }
+      let(:changed_comment) { comment.body = 'change'; comment }
+      subject { lambda { changed_comment.save } }
+      it { should raise_error "Redmine API doesn't support comment updates" }
+    end
   end
 end

data/spec/tickets_spec.rb

@@ -1,64 +1,81 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe TaskMapper::Provider::Redmine::Ticket do
-  before(:each) do 
-    headers = {'X-Redmine-API-Key' => 'abcdefghijk', 'Accept' => 'application/xml'}
-    headers_post_put = {'X-Redmine-API-Key' => 'abcdefghijk', 'Content-Type' => 'application/xml'}
-    @project_id = 1
-    ActiveResource::HttpMock.respond_to do |mock|
-      mock.get '/projects.xml', headers, fixture_for('projects'), 200
-      mock.get '/projects/1.xml', headers, fixture_for('projects/test-repo12'), 200
-      mock.get '/issues.xml?include=journals&project_id=1', headers, fixture_for('issues'), 200
-      mock.get '/issues.xml', headers, fixture_for('issues'), 200
-      mock.get '/issues/1.xml?include=journals', headers, fixture_for('issues/1'), 200
-      mock.put '/issues/1.xml', headers_post_put, '', 200
-      mock.post '/issues.xml', headers_post_put, '', 200
+  let(:headers_auth) { {'X-Redmine-API-Key' => 'abcdefghijk'} }
+  let(:headers_get) { headers_auth.merge('Accept' => 'application/xml') }
+  let(:headers_post_put) { headers_auth.merge('Content-Type' => 'application/xml') }
+  let(:project_id) { 1 }
+  let(:tm) { TaskMapper.new :redmine, :server => 'http://demo.redmine.org/', :token => 'abcdefghijk' }
+  let(:ticket_class) { TaskMapper::Provider::Redmine::Ticket }
+  let(:ticket_id) { 1 }
+
+  describe "Retrieving tickets" do 
+    before(:each) do 
+      ActiveResource::HttpMock.respond_to do |mock|
+        mock.get '/projects/1.xml', headers_get, fixture_for('projects/test-repo12'), 200
+        mock.get '/issues.xml?include=journals&project_id=1', headers_get, fixture_for('issues'), 200
+        mock.get '/issues/1.xml?include=journals', headers_get, fixture_for('issues/1'), 200
+      end
     end
-  
-    @taskmapper = TaskMapper.new :redmine, :server => 'http://demo.redmine.org/', :token => 'abcdefghijk'
-    @project = @taskmapper.project(@project_id)
-    @klass = TaskMapper::Provider::Redmine::Ticket
-  end
+    let(:project) { tm.project project_id }
 
-  it "should be able to load all tickets" do 
-    @project.tickets.should be_an_instance_of(Array)
-    @project.tickets.first.should be_an_instance_of(@klass)
-  end
+    context "when #tickets" do 
+      subject { project.tickets } 
+      it { should be_an_instance_of Array }
 
-  it "should be able to load all tickets based on an array of id's" do
-    @tickets = @project.tickets([1])
-    @tickets.should be_an_instance_of(Array)
-    @tickets.first.should be_an_instance_of(@klass)
-    @tickets.first.title.should == 'test-issue'
-  end
+      context "when #tickets.first" do 
+        subject { project.tickets.first } 
+        it { should be_an_instance_of ticket_class }
+      end
+    end
 
-  it "should be able to load all tickets based on attributes" do
-    @tickets = @project.tickets(:id => 1)
-    @tickets.should be_an_instance_of(Array)
-    @tickets.first.should be_an_instance_of(@klass)
-    @tickets.first.title.should == 'test-issue'
-  end
+    context "when #tickets with an array of id's" do 
+      subject { project.tickets [ticket_id] }
+      it { should be_an_instance_of Array }
+    end
+
+    context "when #tickets with attributes" do 
+      subject { project.tickets :id => ticket_id }
+      it { should be_an_instance_of Array }
+    end
 
-  it "should return the ticket class" do
-    @project.ticket.should == @klass
+    describe "Retrieving a single ticket" do 
+      context "when #ticket with an id" do 
+        subject { project.ticket ticket_id }
+        it { should be_an_instance_of ticket_class }
+      end
+
+      context "when #ticket with attributes" do 
+        subject { project.ticket :id => ticket_id }
+        it { should be_an_instance_of ticket_class }
+      end
+    end
   end
 
-  it "should be able to load a single ticket" do
-    @ticket = @project.ticket(1)
-    @ticket.should be_an_instance_of(@klass)
-    @ticket.title.should == 'test-issue'
+  describe "Create and Update" do
+    before(:each) do 
+      ActiveResource::HttpMock.respond_to do |mock|
+        mock.get '/projects/1.xml', headers_get, fixture_for('projects/test-repo12'), 200
+        mock.post '/issues.xml', headers_post_put, '', 200
+      end
+    end
+    let(:project) { tm.project project_id }
+
+    context "when #ticket!" do 
+      subject { project.ticket! :title => 'Ticket #12', :description => 'Body'}
+      it { should be_an_instance_of ticket_class }
+    end
+    
+    context "when #save" do 
+      subject { project.ticket ticket_id }
+    end
   end
 
   it "should be able to update and save a ticket" do
+    pending
     @ticket = @project.ticket(1)
     @ticket.description = 'hello'
     @ticket.save.should == true
   end
 
-  it "should be able to create a ticket" do 
-    @ticket = @project.ticket!(:title => 'Ticket #12', :description => 'Body')
-    @ticket.should be_an_instance_of(@klass)
-    @ticket.project_id.should_not be_nil
-  end
-
 end

data/taskmapper-redmine.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "taskmapper-redmine"
-  s.version = "0.5.0"
+  s.version = "0.5.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Rafael George"]
-  s.date = "2012-06-20"
+  s.date = "2012-07-04"
   s.description = "Allows taskmapper to interact with Your System."
   s.email = "rafael@hybridgroup.com"
   s.extra_rdoc_files = [
@@ -48,7 +48,7 @@ Gem::Specification.new do |s|
   ]
   s.homepage = "http://github.com/hybridgroup/taskmapper-redmine"
   s.require_paths = ["lib"]
-  s.rubygems_version = "1.8.17"
+  s.rubygems_version = "1.8.24"
   s.summary = "taskmapper Provider for Redmine"
 
   if s.respond_to? :specification_version then

metadata

@@ -1,82 +1,104 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: taskmapper-redmine
-version: !ruby/object:Gem::Version 
+version: !ruby/object:Gem::Version
+  version: 0.5.1
   prerelease: 
-  version: 0.5.0
 platform: ruby
-authors: 
+authors:
 - Rafael George
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-06-20 00:00:00 Z
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2012-07-04 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: taskmapper
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        version: "0.8"
+      - !ruby/object:Gem::Version
+        version: '0.8'
   type: :runtime
   prerelease: false
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.8'
+- !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        version: "2.8"
+      - !ruby/object:Gem::Version
+        version: '2.8'
   type: :development
   prerelease: false
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.8'
+- !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        version: "1.6"
+      - !ruby/object:Gem::Version
+        version: '1.6'
   type: :development
   prerelease: false
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.6'
+- !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        version: "0.5"
+      - !ruby/object:Gem::Version
+        version: '0.5'
   type: :development
   prerelease: false
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.5'
+- !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        version: "1.0"
+      - !ruby/object:Gem::Version
+        version: '1.0'
   type: :development
   prerelease: false
-  version_requirements: *id005
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.0'
 description: Allows taskmapper to interact with Your System.
 email: rafael@hybridgroup.com
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - LICENSE
 - README.md
-files: 
+files:
 - .rbenv-gemsets
 - .rvmrc
 - .travis.yml
@@ -107,30 +129,29 @@ files:
 - ticketmaster-redmine.gemspec
 homepage: http://github.com/hybridgroup/taskmapper-redmine
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 1055099377
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: taskmapper Provider for Redmine
 test_files: []
-