taskmapper-redmine 0.4.0 → 0.5.0

data/VERSION

@@ -1 +1 @@
-0.4.0
+0.5.0

data/lib/provider/comment.rb

@@ -1,13 +1,55 @@
 module TaskMapper::Provider
   module Redmine
-    # The comment class for taskmapper-yoursystem
-    #
-    # Do any mapping between taskmapper and your system's comment model here
-    # versions of the ticket.
-    #
+    # The comment class for taskmapper-redmine
     class Comment < TaskMapper::Provider::Base::Comment
-      # declare needed overloaded methods here
-      
+      API = RedmineAPI::Issue
+
+      def initialize(ticket_id, *object)
+        if object.first
+          object = object.first
+          unless object.is_a? Hash
+            author = object.respond_to?(:user) ? object.user : object.author
+            hash = {:id => object.id.to_i,
+                    :author => author.name,
+                    :body => object.notes,
+                    :update_at => object.created_on,
+                    :created_at => object.created_on,
+                    :ticket_id => ticket_id
+                    }
+          else
+            hash = object
+          end
+          super hash
+        end
+      end
+
+      def self.find_by_id(project_id, ticket_id, id)
+        self.search(ticket_id).select { |journal| journal.id == id.to_i }.first
+      end
+
+      def self.find_by_attributes(project_id, ticket_id, attributes = {})
+        search_by_attribute(self.search(ticket_id), attributes)
+      end
+
+      def self.search(ticket_id)
+        API.find(ticket_id).journals.inject([]) do |arr, journal|
+          arr << self.new(ticket_id, journal) if journal.notes.present?
+          arr
+        end
+      end
+
+      def self.create(*options)
+        attributes = options.first
+        self.new(attributes[:ticket_id], create_journal_for_issue(attributes))
+      end
+
+      private 
+      def self.create_journal_for_issue(attributes)
+        journal = API.find(attributes[:ticket_id])
+        journal.notes = attributes[:body]
+        journal.save
+        journal
+      end
     end
   end
 end

data/lib/provider/ticket.rb

@@ -52,7 +52,7 @@ module TaskMapper::Provider
       end
 
       def self.find_by_id(project_id, ticket_id)
-       self.new API.find(:first, :id => ticket_id)
+       self.new API.find(ticket_id)
       end
 
       def self.find_by_attributes(project_id, attributes = {})
@@ -73,21 +73,6 @@ module TaskMapper::Provider
         to_issue.new? ? to_issue.save : update
       end
       
-      def comments
-        warn "Redmine doesn't support comments"
-        []
-      end
-
-      def comment
-        warn "Redmine doesn't support comments"
-        nil
-      end
-
-      def comment!
-        warn "Redmine doesn't support comments"
-        []
-      end
-      
       private
         def find_issue
           issue = API.find id, :params => {:project_id => self.project_id}

data/lib/redmine/redmine-api.rb

@@ -31,6 +31,19 @@ module RedmineAPI
   end
 
   class Issue < Base
+    def self.find(*arguments)
+      scope   = arguments.slice!(0)
+      options = arguments.slice!(0) || {}
+      # By including journals, we can get Notes aka Comments
+      # RedmineAPI::Issue.find(2180, :params => {:include => 'journals'})
+      get_comments = {:include => 'journals'}
+      if options[:params].nil?
+        options[:params] = get_comments
+      else
+        options[:params].merge!(get_comments)
+      end
+      super scope, options
+    end
   end
 
   class Project < Base

data/spec/comments_spec.rb

@@ -1,5 +1,71 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe TaskMapper::Provider::Redmine::Comment do
-  it "should have specs for comments"
+
+  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
+
+  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
+  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
+  end
 end

data/spec/fixtures/issues/1.xml

@@ -1 +1 @@
-<?xml version="1.0" encoding="UTF-8"?><issue><id>1</id><project name="test-repo" id="1"/><tracker name="Bug" id="1"/><status name="New" id="1"/><priority name="Normal" id="4"/><author name="Redmine Admin" id="1"/><subject>test-issue</subject><description></description><start_date>2010-12-22</start_date><due_date></due_date><done_ratio>0</done_ratio><estimated_hours></estimated_hours><spent_hours>0.0</spent_hours><created_on>2010-12-22T17:55:58-06:00</created_on><updated_on>2010-12-22T17:55:58-06:00</updated_on></issue>
+<?xml version="1.0" encoding="UTF-8"?><issue><id>1</id><project name="test-repo" id="1"/><tracker name="Bug" id="1"/><status name="New" id="1"/><priority name="Normal" id="4"/><author name="Redmine Admin" id="1"/><subject>test-issue</subject><description></description><start_date>2010-12-22</start_date><due_date></due_date><done_ratio>0</done_ratio><estimated_hours></estimated_hours><spent_hours>0.0</spent_hours><created_on>2010-12-22T17:55:58-06:00</created_on><updated_on>2010-12-22T17:55:58-06:00</updated_on><journals type="array"><journal id="1"><user name="Redmine Admin" id="1"/><notes>hello there boys and girls</notes><created_on>2012-05-08T11:02:18+10:00</created_on><details type="array"><detail property="attr" name="status_id"><old_value>1</old_value><new_value>2</new_value></detail><detail property="attr" name="assigned_to_id"><old_value></old_value><new_value>1</new_value></detail><detail property="attr" name="done_ratio"><old_value>0</old_value><new_value>50</new_value></detail></details></journal><journal id="2"><user name="Redmine Admin" id="1"/><notes>Hello World</notes><created_on>2012-05-17T06:26:15+10:00</created_on><details type="array"><detail property="attr" name="status_id"><old_value>2</old_value><new_value>3</new_value></detail><detail property="attr" name="done_ratio"><old_value>50</old_value><new_value>100</new_value></detail></details></journal><journal id="6626"><user name="Redmine User" id="20"/><notes></notes><created_on>2011-03-23T20:08:02-07:00</created_on><details type="array"><detail name="fixed_version_id" property="attr"><old_value></old_value><new_value>118</new_value></detail></details></journal></journals></issue>

data/spec/tickets_spec.rb

@@ -8,9 +8,9 @@ describe TaskMapper::Provider::Redmine::Ticket do
     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?include=journals&project_id=1', headers, fixture_for('issues'), 200
       mock.get '/issues.xml', headers, fixture_for('issues'), 200
-      mock.get '/issues/1.xml', headers, fixture_for('issues/1'), 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

data/taskmapper-redmine.gemspec

@@ -0,0 +1,78 @@
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = "taskmapper-redmine"
+  s.version = "0.5.0"
+
+  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.description = "Allows taskmapper to interact with Your System."
+  s.email = "rafael@hybridgroup.com"
+  s.extra_rdoc_files = [
+    "LICENSE",
+    "README.md"
+  ]
+  s.files = [
+    ".rbenv-gemsets",
+    ".rvmrc",
+    ".travis.yml",
+    "Gemfile",
+    "Gemfile.lock",
+    "LICENSE",
+    "README.md",
+    "Rakefile",
+    "VERSION",
+    "lib/provider/api-extensions.rb",
+    "lib/provider/comment.rb",
+    "lib/provider/project.rb",
+    "lib/provider/redmine.rb",
+    "lib/provider/ticket.rb",
+    "lib/redmine/redmine-api.rb",
+    "lib/taskmapper-redmine.rb",
+    "spec/comments_spec.rb",
+    "spec/fixtures/issues.xml",
+    "spec/fixtures/issues/1.xml",
+    "spec/fixtures/projects.xml",
+    "spec/fixtures/projects/test-repo12.xml",
+    "spec/projects_spec.rb",
+    "spec/spec.opts",
+    "spec/spec_helper.rb",
+    "spec/taskmapper-redmine_spec.rb",
+    "spec/tickets_spec.rb",
+    "taskmapper-redmine.gemspec",
+    "ticketmaster-redmine.gemspec"
+  ]
+  s.homepage = "http://github.com/hybridgroup/taskmapper-redmine"
+  s.require_paths = ["lib"]
+  s.rubygems_version = "1.8.17"
+  s.summary = "taskmapper Provider for Redmine"
+
+  if s.respond_to? :specification_version then
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+      s.add_runtime_dependency(%q<taskmapper>, ["~> 0.8"])
+      s.add_development_dependency(%q<rspec>, ["~> 2.8"])
+      s.add_development_dependency(%q<jeweler>, ["~> 1.6"])
+      s.add_development_dependency(%q<simplecov>, ["~> 0.5"])
+      s.add_development_dependency(%q<rcov>, ["~> 1.0"])
+    else
+      s.add_dependency(%q<taskmapper>, ["~> 0.8"])
+      s.add_dependency(%q<rspec>, ["~> 2.8"])
+      s.add_dependency(%q<jeweler>, ["~> 1.6"])
+      s.add_dependency(%q<simplecov>, ["~> 0.5"])
+      s.add_dependency(%q<rcov>, ["~> 1.0"])
+    end
+  else
+    s.add_dependency(%q<taskmapper>, ["~> 0.8"])
+    s.add_dependency(%q<rspec>, ["~> 2.8"])
+    s.add_dependency(%q<jeweler>, ["~> 1.6"])
+    s.add_dependency(%q<simplecov>, ["~> 0.5"])
+    s.add_dependency(%q<rcov>, ["~> 1.0"])
+  end
+end
+

metadata

@@ -2,7 +2,7 @@
 name: taskmapper-redmine
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors: 
 - Rafael George
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-05-14 00:00:00 Z
+date: 2012-06-20 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: taskmapper
@@ -103,6 +103,7 @@ files:
 - spec/spec_helper.rb
 - spec/taskmapper-redmine_spec.rb
 - spec/tickets_spec.rb
+- taskmapper-redmine.gemspec
 - ticketmaster-redmine.gemspec
 homepage: http://github.com/hybridgroup/taskmapper-redmine
 licenses: []