jirarest2 0.0.9 → 0.0.10

data/History.txt

@@ -1,3 +1,13 @@
+=== 0.0.10 / 2012-08-02
+
+* 1 major enhancement:
+
+  * Added exceptions for return code 400 and 400 in the main executer
+
+* 1 minor enhancements:
+
+  * Added jira_comment to work with comments. It's not very nice to errors and there might be some work with. This fixes #12
+
 === 0.0.9 / 2012-07-27
 
 * 1 major enhancement:

data/Manifest.txt

@@ -5,6 +5,7 @@ Manifest.txt
 README.txt
 Rakefile
 bin/create_issue.rb
+bin/jira_comment
 bin/jira_create_issue
 bin/jira_create_issue.rb
 bin/jira_watcher
@@ -17,16 +18,19 @@ lib/jirarest2/issue.rb
 lib/jirarest2/madbitconfig.rb
 lib/jirarest2/result.rb
 lib/jirarest2/services.rb
+lib/jirarest2/services/comment.rb
 lib/jirarest2/services/issuelink.rb
 lib/jirarest2/services/issuelinktype.rb
 lib/jirarest2/services/watcher.rb
 lib/jirarest2bin.rb
+test/data/get-comments.txt
 test/data/issuespec.txt
 test/data/test.config.data
 test/data/test.json
 test/data/test.nojson
 test/data/test.nojson1
 test/data/ticket.json
+test/test_comment.rb
 test/test_connect.rb
 test/test_credentials.rb
 test/test_issue.rb

data/README.txt

@@ -22,11 +22,13 @@ There are scripts to create new issues with watchers and link those to existing
    * Still in the alpha stages. The classes are still pretty volatile.
    * jira_create_issue allows you to create new issues with watchers and link those to existing issues
    * jira_watcher allows you to manipulate watchers on existing issues
+   * jira_comment allows you to add read and update comments. (The latter only if you know the right comment id)
 
 == SYNOPSIS:
 
 jira_create_issue -h
 jira_watcher -h
+jira_comment -h
 
 == REQUIREMENTS:
 

data/bin/jira_comment

@@ -0,0 +1,156 @@
+#!/usr/bin/env ruby
+
+#    Copyright (C) 2012 Cyril Bitterich
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU General Public License as published by
+#    the Free Software Foundation, either version 3 of the License, or
+#    (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU General Public License for more details.
+#
+#    You should have received a copy of the GNU General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+
+require "jirarest2bin"
+require "pp"
+
+# Check for the minimum ruby version we need
+Jirarest2Bin::check_ruby_version
+
+class ParseOptions
+
+=begin
+  Parse the command line parameters
+  @return [Array] returns two hashes. The first one contains the options for the issue, the second one for the execution of the script
+=end
+  def self.parse(args)
+    issueopts = OpenStruct.new
+    scriptopts = OpenStruct.new
+    scriptopts.configfile = "~/.jiraconfig"
+    scriptopts.ofs = "\n"
+      
+    
+    opts = OptionParser.new do |opts|
+      opts.banner = "Usage: #{__FILE__} [options]"
+      scriptopts = Jirarest2Bin::defaultoptions(opts,scriptopts)
+
+      opts.on("-i", "--issue ISSUE,ISSUE,...", Array, "Issues to work on") do |i|
+        issueopts.issues = i
+      end
+
+      opts.on("-d", "--id COMMENTID", "ID number of the comment (for remove and set/update)") do |i| 
+        issueopts.commentid = i
+      end
+
+      opts.on("-t", "--text TEXT", "Text for the comment to add/update") do |w|
+        issueopts.text = w
+      end
+      
+      opts.on("-c", "--command [command]",[:add,:remove,:view,:get,:set,:update], "What to do with the comment(s) on the issues (add, remove,set)") do |c|
+        c = :set if c == :update
+        c = :get if c == :view
+        scriptopts.command = c
+      end
+      
+      opts.on("-O", "--field-seperator CHAR", "Output field separator. Set this if you want the output to be formatted in one line for example.") do |ofs|
+        scriptopts.ofs = ofs
+      end
+      
+      opts.on("-v", "--verbose", "Tell what I did in great detail") do |v|
+        scriptopts.verbose = v
+      end
+    end # OptionParser.new
+    
+    opts.parse!(args)
+    return issueopts, scriptopts
+    
+  end #parse
+end # class
+
+@issueopts, @scriptopts = ParseOptions.parse(ARGV)
+
+# Print the issues to STDOUT
+# We use the instance-varable @scriptopts.ofs here
+# @param [String] issue The Key of the Issue these comments belong to
+# @param [Array(CommentElement)] results An Array of CommentElement 
+def print_results(issue,results)
+  output = Array.new
+  if results.size == 0 then
+    output << "There are no comments for #{issue}."
+  else
+    output << "Comment(s) for #{issue}:"
+  end
+  results.each { |result| 
+    output << result.text
+    output <<  "Created on #{result.cdate.to_s}"
+    output << "Last written by: #{result.author} on #{result.mdate.to_s}"
+  }
+  puts output.join(@scriptopts.ofs)
+  
+end
+
+# Calls the issues and get the comments of each issue in return.
+# @param [Array] issues An Array containing the issuekeys
+# @param [Fixnum] id Identidication number of the comment to view
+def view_comments(issues,id = nil)
+  connection = Jirarest2Bin::get_connection(@scriptopts,@connection)
+  result = Array.new
+  if id.nil? then
+    issues.each { |issue|
+      connectedissue = Comment.new(connection,issue)
+      result << connectedissue.get
+      print_results(issue,result.flatten.compact)
+    }
+  else
+    connectedissue = Comment.new(connection,issues[0],id)
+    result << connectedissue.get
+    print_results(issues,result.flatten.compact)
+  end
+end
+
+# Calls the issues and runs the needed work on it with the given users by opening the connection to the server, looking for the issue identified by the key and executing the string
+# @param [Array] issues Issues to work with
+# @param [Array] text Text to work with
+# @param [String] method Kind of work to be done
+# @param [Fixnum] id Identifier of the comment
+def work(issues,text,method,id = nil)
+  connection = Jirarest2Bin::get_connection(@scriptopts,@connection)
+  issues.each{ |issue|
+    connectedissue = Comment.new(connection, issue, id)
+    connectedissue.send method, text
+    if @scriptopts.verbose then
+      if id.nil? then
+        puts "Used #{method} on issue #{issue}" 
+      else
+        puts "Used #{method} on the comment #{id} on issue #{issue}" 
+      end
+    end
+  }
+end
+
+if @scriptopts.writeconf then
+  Jirarest2Bin::write_configfile(@scriptopts)
+end
+
+Jirarest2Bin::required_argument("issue") if @issueopts.issues.nil?
+case @scriptopts.command
+when :add
+  Jirarest2Bin::required_argument("text") if @issueopts.text.nil?
+  work(@issueopts.issues,@issueopts.text,"add")
+when :remove
+  Jirarest2Bin::required_argument("id") if @issueopts.commentid.nil?
+  work(@issueopts.issues,"","delete",@issueopts.commentid)
+when :set
+  Jirarest2Bin::required_argument("id") if @issueopts.commentid.nil?
+  Jirarest2Bin::required_argument("text") if @issueopts.text.nil?
+  work(@issueopts.issues,@issueopts.text,"update",@issueopts.commentid)
+when :get
+  view_comments(@issueopts.issues, @issueopts.commentid)
+else
+  puts "Don't know what to do. Please set -c"
+end

data/bin/jira_create_issue

@@ -199,13 +199,9 @@ def get_jsondata_from_file
   content = Hash.new
   if  data["metadata"] then # We have metadata and content
     content = data["content"]
-    pp "metadata"
     @issueopts.project = data["metadata"]["project"] if data["metadata"]["project"]
     @issueopts.issue = data["metadata"]["issue"] if data["metadata"]["issue"]
-    pp data["metadata"]["watcher"].class
-    pp data["metadata"]["watcher"]
     @issueopts.watchers = data["metadata"]["watcher"] if data["metadata"]["watcher"]
-    pp data["metadata"]["link"]
     @scriptopts.linkdestkey, @scriptopts.linktype = data["metadata"]["link"].flatten if data["metadata"]["link"]
   else # We have the simple format that carries only the content
     content = data

data/lib/jirarest2.rb

@@ -17,7 +17,7 @@
 #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 
-VERSION = "0.0.9"
+VERSION = "0.0.10"
 
 require_relative "jirarest2/connect"
 require_relative "jirarest2/issue"
@@ -26,3 +26,4 @@ require_relative "jirarest2/exceptions"
 require_relative "jirarest2/services/watcher"
 require_relative "jirarest2/services"
 require_relative "jirarest2/services/issuelink"
+require_relative "jirarest2/services/comment"

data/lib/jirarest2/connect.rb

@@ -44,9 +44,8 @@ class Connect
   def execute(operation,uritail,data)
     uri = nil
     uri = URI(@CONNECTURL+uritail)
-    
     if data != "" then
-      if operation != "Post" then # POST carries the payload in the body that's why we have to wait
+      if ! (operation == "Post" || operation == "Put") then # POST carries the payload in the body that's why we have to wait
         uri.query = URI.encode_www_form(data)
       end
     end
@@ -57,27 +56,33 @@ class Connect
     req["Content-Type"] = "application/json;charset=UTF-8"
 
     if data != "" then
-      if operation == "Post" then # POST carries the payload in the body
+      if (operation == "Post" || operation == "Put") then # POST and PUT carry the payload in the body
         @payload = data.to_json
         req.body = @payload
       end
     end
     
+
     # Ask the server
     result = Net::HTTP.start(uri.host, uri.port) {|http|
       http.request(req)
     }
+
     # deal with output
     case result
-    when Net::HTTPUnauthorized #No login-credentials oder wrong ones.
+    when Net::HTTPBadRequest # 400
+      raise Jirarest2::BadRequestError, result.body
+    when Net::HTTPUnauthorized # 401 No login-credentials oder wrong ones.
       raise Jirarest2::AuthentificationError, result.body
-    when Net::HTTPForbidden #Captcha-Time
-      #      pp res.get_fields("x-authentication-denied-reason")
-      # Result: ["CAPTCHA_CHALLENGE; login-url=http://localhost:8080/login.jsp"]
-      result.get_fields("x-authentication-denied-reason")[0] =~ /.*login-url=(.*)/
-      raise Jirarest2::AuthentificationCaptchaError, $1
-    when Net::HTTPNotFound
+    when Net::HTTPForbidden # 403
+      if result.get_fields("x-authentication-denied-reason")[0] =~ /.*login-url=(.*)/ then #Captcha-Time
+        raise Jirarest2::AuthentificationCaptchaError, $1
+      else
+        raise Jirarest2::ForbiddenError, result.body
+      end
+    when Net::HTTPNotFound # 404
       raise Jirarest2::NotFoundError, result.body
+
     end
     
     return Jirarest2::Result.new(result)
@@ -90,7 +95,10 @@ class Connect
     begin 
       begin
         ret = (execute("Get","dashboard","").code == "200")
-      rescue Jirarest2::NotFoundError
+# TODO is the 404 really possible?
+      rescue Jirarest2::NotFoundError 
+        return false
+      rescue Jirarest2::BadRequestError
         return false
       end
     end

data/lib/jirarest2/exceptions.rb

@@ -19,10 +19,14 @@
 module Jirarest2
   ## connections.rb
 
-  # Authentification failed 
+  # 400 - 
+  class BadRequestError < StandardError ; end
+  # 401 Authentification failed 
   class AuthentificationError < StandardError ; end
-  # Authentification failed and JIRA(tm) requires a login with captcha to continue
+  # 403 Authentification failed and JIRA(tm) requires a login with captcha to continue
   class AuthentificationCaptchaError < StandardError ; end
+  # 403 Authentification failed
+  class ForbiddenError < StandardError ; end
   # 404 - Results in HTML body - not JSON
   class NotFoundError < StandardError ; end
   # Could not heal URI

data/lib/jirarest2/services.rb

@@ -28,7 +28,8 @@ class Services
   end
 
   # Send the GET request
-  # @param [Hash] data to be sent to Connection.execute
+  # @param [Hash,String] data to be sent to Connection.execute
+  # @return [Hash] The result as constructed by Connection.execute transformed from a json hash
   def get(data = "")
     return @connection.execute("Get",@uritail,data).result
   end
@@ -36,15 +37,23 @@ class Services
 
   # Send the POST request
   # @param [Hash] data to be sent to Connection.execute
+  # @return [Result] The result as constructed by Connection.execute
   def post(data = "")
     return @connection.execute("Post",@uritail,data)
-  end  
-
+  end
   
   # Send the DELETE request
   # @param [Hash] data to be sent to Connection.execute
+  # @return [Result] The result as constructed by Connection.execute
   def delete(data = "")
     return @connection.execute("Delete",@uritail,data)
   end
 
+  # Send the PUT request
+  # @param [HASH] data to be sent to Connection.execute
+  # @return [Hash] The result as constructed by Connection.execute transformed from a json hash
+  def put(data = "")
+    return @connection.execute("Put",@uritail,data).result
+  end
+
 end #class

data/lib/jirarest2/services/comment.rb

@@ -0,0 +1,118 @@
+# Comment class
+#    Copyright (C) 2012 Cyril Bitterich
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU General Public License as published by
+#    the Free Software Foundation, either version 3 of the License, or
+#    (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU General Public License for more details.
+#
+#    You should have received a copy of the GNU General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+
+require "time"
+require "jirarest2/connect"
+require "jirarest2/services"
+
+#TODO give nicer answers in return to work
+
+# One single Comment
+# TODO Maybe rewrite the initalize and then get a few other parameters like the issue-id or the comment-id
+class CommentElement 
+  #The author of the comment
+  # @return [String]
+  attr_reader :author
+  #text of the comment
+  # @return [String]
+  attr_reader :text
+  #creation date
+  # @return [Time]
+  attr_reader :cdate
+  #last modify date
+  # @return [Time]
+  attr_reader :mdate
+
+  # create one instance of an Comment
+  # @param [String] author (Last) author of the comment
+  # @param [String] text Text of the comment
+  # @param [Time,String] cdate Creation date of the comment
+  # @param [Time,String] mdate Date of last change to the comment
+  def  initialize(author,text,cdate = Time.now,mdate = Time.now)
+    @author = author
+    @text = text
+    #  parse time if needed
+    cdate = Time.parse(cdate) if cdate.instance_of?(String)
+    @cdate = cdate
+    mdate = Time.parse(mdate) if mdate.instance_of?(String)
+    @mdate = mdate
+  end
+end
+
+class Comment < Services
+#TODO See to documentation for "DELETE" as we use the superclass here and it will not be shown
+
+  #  Set our uritail
+  # @param [Connection] connection 
+  # @param [String] issueid The id or key of the issue in question
+  def initialize(connection, issueid, commentid = nil)
+    if commentid then
+      @uritail = "issue/#{issueid}/comment/#{commentid}"
+    else
+      @uritail = "issue/#{issueid}/comment"
+    end
+    super(connection)
+  end
+
+  # Add a comment to an issue
+  # @param [String] text to add
+  # @return [Result] The result as constructed by Connection.execute
+  def add(text)
+    post({"body" => text})
+  end
+
+  # Split the returned hash and fill the CommentElement
+  # @param [Hash] result The json based hash of one Comment
+  # @return [CommentElement] One comment
+  private
+  def create_element(result)
+    text = result["body"]
+    author = result["updateAuthor"]["displayName"]
+    ctime = result["created"]
+    mtime = result["updated"]
+    return CommentElement.new(author,text,ctime,mtime)          
+  end
+
+  public
+  # Get a certain comment
+  # @param [String] data Additional data to send via GET
+  # @return [Nil] If there is no comment in the Project
+  # @return [Array(CommentElement)] If there is one or more than one result - TODO See if this is going to be changed for a special type that keeps startAt, maxResults and total
+  def get(data = "")
+    result = super("")
+    if result["comments"].nil? then
+      return [create_element(result)]
+    elsif result["comments"].empty? then
+      return nil
+    else
+      resultarray = Array.new
+      result["comments"].each { |singleresult|
+         resultarray << create_element(singleresult)
+      }
+      return resultarray
+    end
+  end
+  
+  # Update an comment
+  # @param [String] text The new text for the comment
+  # @return [CommentElement] The new comment
+  def update(text)
+    result =  put({"body" => text})
+    return CommentElement.new(result["updateAuthor"]["displayName"], result["body"], result["created"], result["updated"])
+  end
+    
+end # Comment

data/test/data/get-comments.txt

@@ -0,0 +1,5 @@
+HTTP/1.1 200 OK
+Server: Apache-Coyote/1.1
+Content-Type: application/json;charset=UTF-8
+
+{"startAt":0,"maxResults":4,"total":4,"comments":[{"self":"http://localhost:2990/jira/rest/api/2/issue/10116/comment/10223","id":"10223","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"1st comment","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-29T18:43:22.654+0200","updated":"2012-07-29T18:43:22.654+0200"},{"self":"http://localhost:2990/jira/rest/api/2/issue/10116/comment/10224","id":"10224","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"2nd comment","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-29T18:43:36.648+0200","updated":"2012-07-29T18:43:36.648+0200"},{"self":"http://localhost:2990/jira/rest/api/2/issue/10116/comment/10225","id":"10225","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"3rd comment","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-29T18:43:47.769+0200","updated":"2012-07-29T18:43:47.769+0200"},{"self":"http://localhost:2990/jira/rest/api/2/issue/10116/comment/10226","id":"10226","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"4th comment","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-29T18:44:03.321+0200","updated":"2012-07-29T18:44:03.321+0200","visibility":{"type":"role","value":"Users"}}]}

data/test/data/issuespec.txt

@@ -1,14 +1,5 @@
 HTTP/1.1 200 OK
 Server: Apache-Coyote/1.1
-X-AREQUESTID: 1205x110x1
-Set-Cookie: JSESSIONID=3D7C4DCDAD212D6D5227E08E5059A347; Path=/
-X-Seraph-LoginReason: OK
-Set-Cookie: atlassian.xsrf.token=BP8Q-WXN6-SKX3-NB5M|09078c4ba23629ddc418efa11cc75a215f98ce1d|lin; Path=/jira
-X-ASESSIONID: 1lodmqg
-X-AUSERNAME: test
-Cache-Control: no-cache, no-store, no-transform
 Content-Type: application/json;charset=UTF-8
-Transfer-Encoding: chunked
-Date: Mon, 23 Jul 2012 18:05:26 GMT
 
-{"expand":"projects","projects":[{"expand":"issuetypes","self":"http://localhost:2990/jira/rest/api/2/project/MFTP","id":"10000","key":"MFTP","name":"My first Test Project","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/projectavatar?size=small&pid=10000&avatarId=10011","48x48":"http://localhost:2990/jira/secure/projectavatar?pid=10000&avatarId=10011"},"issuetypes":[{"self":"http://localhost:2990/jira/rest/api/2/issuetype/6","id":"6","description":"An own issue type","iconUrl":"http://localhost:2990/jira/images/icons/ico_epic.png","name":"My issue type","subtask":false,"expand":"fields","fields":{"summary":{"required":true,"schema":{"type":"string","system":"summary"},"name":"Summary","operations":["set"]},"timetracking":{"required":false,"schema":{"type":"timetracking","system":"timetracking"},"name":"Time Tracking","operations":["set","edit"]},"customfield_10102":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textfield","customId":10102},"name":"projects","operations":["set"]},"customfield_10101":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textfield","customId":10101},"name":"Issue Type","operations":["set"]},"customfield_10100":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textfield","customId":10100},"name":"issuetype","operations":["set"]},"issuetype":{"required":true,"schema":{"type":"issuetype","system":"issuetype"},"name":"Issue Type","operations":[],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/issuetype/6","id":"6","description":"An own issue type","iconUrl":"http://localhost:2990/jira/images/icons/ico_epic.png","name":"My issue type","subtask":false}]},"customfield_10006":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:select","customId":10006},"name":"List select","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10015","value":"Räuber","id":"10015"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10016","value":"Kabale und Liebe","id":"10016"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10017","value":"Faust","id":"10017"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10018","value":"Landleben","id":"10018"}]},"customfield_10005":{"required":false,"schema":{"type":"array","items":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:multiselect","customId":10005},"name":"Multi Select","operations":["add","set","remove"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10023","value":"Glocke","id":"10023"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10024","value":"Kabale und Liebe","id":"10024"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10025","value":"Schiller","id":"10025"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10026","value":"Göthe","id":"10026"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10027","value":"Faust","id":"10027"}]},"labels":{"required":false,"schema":{"type":"array","items":"string","system":"labels"},"name":"Labels","autoCompleteUrl":"http://localhost:2990/jira/rest/api/1.0/labels/suggest?query=","operations":["add","set","remove"]},"resolution":{"required":false,"schema":{"type":"resolution","system":"resolution"},"name":"Resolution","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/resolution/1","name":"Fixed","id":"1"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/2","name":"Won't Fix","id":"2"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/3","name":"Duplicate","id":"3"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/4","name":"Incomplete","id":"4"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/5","name":"Cannot Reproduce","id":"5"}]},"attachment":{"required":false,"schema":{"type":"array","items":"attachment","system":"attachment"},"name":"Attachment","operations":[]},"project":{"required":true,"schema":{"type":"project","system":"project"},"autoCompleteUrl":"Project","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/project/MFTP","id":"10000","key":"MFTP","name":"My first Test Project","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/projectavatar?size=small&pid=10000&avatarId=10011","48x48":"http://localhost:2990/jira/secure/projectavatar?pid=10000&avatarId=10011"}}]},"versions":{"required":false,"schema":{"type":"array","items":"version","system":"versions"},"name":"Affects Version/s","operations":["set","add","remove"],"allowedValues":[]},"environment":{"required":false,"schema":{"type":"string","system":"environment"},"name":"Environment","operations":["set"]},"priority":{"required":false,"schema":{"type":"priority","system":"priority"},"name":"Priority","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/priority/1","iconUrl":"http://localhost:2990/jira/images/icons/priority_blocker.gif","name":"Blocker","id":"1"},{"self":"http://localhost:2990/jira/rest/api/2/priority/2","iconUrl":"http://localhost:2990/jira/images/icons/priority_critical.gif","name":"Critical","id":"2"},{"self":"http://localhost:2990/jira/rest/api/2/priority/3","iconUrl":"http://localhost:2990/jira/images/icons/priority_major.gif","name":"Major","id":"3"},{"self":"http://localhost:2990/jira/rest/api/2/priority/4","iconUrl":"http://localhost:2990/jira/images/icons/priority_minor.gif","name":"Minor","id":"4"},{"self":"http://localhost:2990/jira/rest/api/2/priority/5","iconUrl":"http://localhost:2990/jira/images/icons/priority_trivial.gif","name":"Trivial","id":"5"}]},"description":{"required":false,"schema":{"type":"string","system":"description"},"name":"Description","operations":["set"]},"customfield_10001":{"required":false,"schema":{"type":"datetime","custom":"com.atlassian.jira.plugin.system.customfieldtypes:datetime","customId":10001},"name":"Date Time Field","operations":["set"]},"customfield_10002":{"required":false,"schema":{"type":"date","custom":"com.atlassian.jira.plugin.system.customfieldtypes:datepicker","customId":10002},"name":"Date Picker","operations":["set"]},"customfield_10003":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textarea","customId":10003},"name":"Großes Text","operations":["set"]},"customfield_10004":{"required":false,"schema":{"type":"array","items":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:multicheckboxes","customId":10004},"name":"Multi Checkboxes","operations":["add","set","remove"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10019","value":"Göthe","id":"10019"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10020","value":"Schiller","id":"10020"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10021","value":"Heine","id":"10021"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10022","value":"Kafka","id":"10022"}]},"components":{"required":false,"schema":{"type":"array","items":"component","system":"components"},"name":"Component/s","operations":["add","set","remove"],"allowedValues":[]},"customfield_10000":{"required":false,"schema":{"type":"array","items":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:cascadingselect","customId":10000},"name":"Cascading Select Test","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10000","value":"English","id":"10000","children":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10003","value":"One","id":"10003"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10004","value":"Two","id":"10004"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10005","value":"Three","id":"10005"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10006","value":"Four","id":"10006"}]},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10001","value":"German","id":"10001","children":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10007","value":"Eins","id":"10007"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10008","value":"zwei","id":"10008"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10009","value":"drEi","id":"10009"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10010","value":"vier","id":"10010"}]},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10002","value":"ISO","id":"10002","children":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10011","value":"Unaone","id":"10011"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10012","value":"Bissotwo","id":"10012"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10013","value":"Terrathree","id":"10013"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10014","value":"Kartefour","id":"10014"}]}]}}}]}]}
+{"expand":"projects","projects":[{"expand":"issuetypes","self":"http://localhost:2990/jira/rest/api/2/project/MFTP","id":"10000","key":"MFTP","name":"My first Test Project","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/projectavatar?size=small&pid=10000&avatarId=10011","48x48":"http://localhost:2990/jira/secure/projectavatar?pid=10000&avatarId=10011"},"issuetypes":[{"self":"http://localhost:2990/jira/rest/api/2/issuetype/6","id":"6","description":"An own issue type","iconUrl":"http://localhost:2990/jira/images/icons/ico_epic.png","name":"My issue type","subtask":false,"expand":"fields","fields":{"summary":{"required":true,"schema":{"type":"string","system":"summary"},"name":"Summary","operations":["set"]},"timetracking":{"required":false,"schema":{"type":"timetracking","system":"timetracking"},"name":"Time Tracking","operations":["set","edit"]},"customfield_10102":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textfield","customId":10102},"name":"projects","operations":["set"]},"customfield_10101":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textfield","customId":10101},"name":"Issue Type","operations":["set"]},"customfield_10100":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textfield","customId":10100},"name":"issuetype","operations":["set"]},"issuetype":{"required":true,"schema":{"type":"issuetype","system":"issuetype"},"name":"Issue Type","operations":[],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/issuetype/6","id":"6","description":"An own issue type","iconUrl":"http://localhost:2990/jira/images/icons/ico_epic.png","name":"My issue type","subtask":false}]},"customfield_10006":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:select","customId":10006},"name":"List select","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10015","value":"Räuber","id":"10015"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10016","value":"Kabale und Liebe","id":"10016"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10017","value":"Faust","id":"10017"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10018","value":"Landleben","id":"10018"}]},"customfield_10005":{"required":false,"schema":{"type":"array","items":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:multiselect","customId":10005},"name":"Multi Select","operations":["add","set","remove"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10023","value":"Glocke","id":"10023"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10024","value":"Kabale und Liebe","id":"10024"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10025","value":"Schiller","id":"10025"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10026","value":"Göthe","id":"10026"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10027","value":"Faust","id":"10027"}]},"labels":{"required":false,"schema":{"type":"array","items":"string","system":"labels"},"name":"Labels","autoCompleteUrl":"http://localhost:2990/jira/rest/api/1.0/labels/suggest?query=","operations":["add","set","remove"]},"resolution":{"required":false,"schema":{"type":"resolution","system":"resolution"},"name":"Resolution","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/resolution/1","name":"Fixed","id":"1"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/2","name":"Won't Fix","id":"2"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/3","name":"Duplicate","id":"3"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/4","name":"Incomplete","id":"4"},{"self":"http://localhost:2990/jira/rest/api/2/resolution/5","name":"Cannot Reproduce","id":"5"}]},"attachment":{"required":false,"schema":{"type":"array","items":"attachment","system":"attachment"},"name":"Attachment","operations":[]},"project":{"required":true,"schema":{"type":"project","system":"project"},"autoCompleteUrl":"Project","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/project/MFTP","id":"10000","key":"MFTP","name":"My first Test Project","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/projectavatar?size=small&pid=10000&avatarId=10011","48x48":"http://localhost:2990/jira/secure/projectavatar?pid=10000&avatarId=10011"}}]},"versions":{"required":false,"schema":{"type":"array","items":"version","system":"versions"},"name":"Affects Version/s","operations":["set","add","remove"],"allowedValues":[]},"environment":{"required":false,"schema":{"type":"string","system":"environment"},"name":"Environment","operations":["set"]},"priority":{"required":false,"schema":{"type":"priority","system":"priority"},"name":"Priority","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/priority/1","iconUrl":"http://localhost:2990/jira/images/icons/priority_blocker.gif","name":"Blocker","id":"1"},{"self":"http://localhost:2990/jira/rest/api/2/priority/2","iconUrl":"http://localhost:2990/jira/images/icons/priority_critical.gif","name":"Critical","id":"2"},{"self":"http://localhost:2990/jira/rest/api/2/priority/3","iconUrl":"http://localhost:2990/jira/images/icons/priority_major.gif","name":"Major","id":"3"},{"self":"http://localhost:2990/jira/rest/api/2/priority/4","iconUrl":"http://localhost:2990/jira/images/icons/priority_minor.gif","name":"Minor","id":"4"},{"self":"http://localhost:2990/jira/rest/api/2/priority/5","iconUrl":"http://localhost:2990/jira/images/icons/priority_trivial.gif","name":"Trivial","id":"5"}]},"description":{"required":false,"schema":{"type":"string","system":"description"},"name":"Description","operations":["set"]},"customfield_10001":{"required":false,"schema":{"type":"datetime","custom":"com.atlassian.jira.plugin.system.customfieldtypes:datetime","customId":10001},"name":"Date Time Field","operations":["set"]},"customfield_10002":{"required":false,"schema":{"type":"date","custom":"com.atlassian.jira.plugin.system.customfieldtypes:datepicker","customId":10002},"name":"Date Picker","operations":["set"]},"customfield_10003":{"required":false,"schema":{"type":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:textarea","customId":10003},"name":"Großes Text","operations":["set"]},"customfield_10004":{"required":false,"schema":{"type":"array","items":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:multicheckboxes","customId":10004},"name":"Multi Checkboxes","operations":["add","set","remove"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10019","value":"Göthe","id":"10019"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10020","value":"Schiller","id":"10020"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10021","value":"Heine","id":"10021"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10022","value":"Kafka","id":"10022"}]},"components":{"required":false,"schema":{"type":"array","items":"component","system":"components"},"name":"Component/s","operations":["add","set","remove"],"allowedValues":[]},"customfield_10000":{"required":false,"schema":{"type":"array","items":"string","custom":"com.atlassian.jira.plugin.system.customfieldtypes:cascadingselect","customId":10000},"name":"Cascading Select Test","operations":["set"],"allowedValues":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10000","value":"English","id":"10000","children":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10003","value":"One","id":"10003"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10004","value":"Two","id":"10004"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10005","value":"Three","id":"10005"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10006","value":"Four","id":"10006"}]},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10001","value":"German","id":"10001","children":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10007","value":"Eins","id":"10007"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10008","value":"zwei","id":"10008"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10009","value":"drEi","id":"10009"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10010","value":"vier","id":"10010"}]},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10002","value":"ISO","id":"10002","children":[{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10011","value":"Unaone","id":"10011"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10012","value":"Bissotwo","id":"10012"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10013","value":"Terrathree","id":"10013"},{"self":"http://localhost:2990/jira/rest/api/2/customFieldOption/10014","value":"Kartefour","id":"10014"}]}]}}}]}]}

data/test/test_comment.rb

@@ -0,0 +1,88 @@
+require "minitest/autorun"
+require "webmock/minitest"
+require "jirarest2/connect"
+require "jirarest2/credentials"
+require "jirarest2/services/comment"
+
+class TestComment < MiniTest::Unit::TestCase
+  def setup
+    cred = Credentials.new("http://localhost:2990/jira/rest/api/2/","test","1234")
+    @con = Connect.new(cred)
+  end
+
+  def test_get_comment_filled
+    comment = Comment.new(@con,"SP-15")
+    raw_response_file = File.new(File.dirname(__FILE__)+"/data/get-comments.txt")
+    stub_request(:get, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-15/comment").with(:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(raw_response_file)
+    result =  comment.get
+    assert_equal 4,result.size
+    assert_equal "2nd comment", result[1].text
+    assert_equal true, Time.parse("2012-07-29 18:44:03 +0200").ctime == result[3].mdate.ctime
+  end
+
+  def test_get_comment_empty
+    comment = Comment.new(@con,"SP-14")
+    stub_request(:get, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-14/comment").with(:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 200, :body => '{"startAt":0,"maxResults":0,"total":0,"comments":[]}', :headers => {})
+    assert_equal nil, comment.get
+  end
+
+  def test_add_comment_fail
+    stub_request(:post, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-3/comment").with(:body => "{\"body\":\"\"}", :headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 400, :body => '{"errorMessages":[],"errors":{"comment":"Comment body can not be empty!"}}', :headers => {})
+    comment = Comment.new(@con,"SP-3")
+    assert_raises(Jirarest2::BadRequestError) {
+      comment.add("")
+    }
+    stub_request(:post, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SPP-3/comment").with(:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 404, :body => '{"errorMessages":["Issue Does Not Exist"],"errors":{}}', :headers => {})
+    comment = Comment.new(@con,"SPP-3")
+    assert_raises(Jirarest2::NotFoundError) {
+      comment.add("Text")
+    }
+  end
+
+  def test_add_comment_success
+    stub_request(:post, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-3/comment").with(:body => "{\"body\":\"Text for the long run\"}",:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 201, :body => '{"self":"http://localhost:2990/jira/rest/api/2/issue/10104/comment/10110","id":"10110","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"Text for the long run","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-27T20:36:07.832+0200","updated":"2012-07-27T20:36:07.832+0200"}', :headers => {})
+
+    comment = Comment.new(@con,"SP-3")
+    assert_match /,"body":"Text for the long run","updateAuthor"/, comment.add("Text for the long run").body
+    assert_instance_of Jirarest2::Result, comment.add("Text for the long run")
+  end
+
+  def test_get_special_comment_fail
+    comment = Comment.new(@con,"SP-3",12345)
+    stub_request(:get, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-3/comment/12345").with(:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 404, :body => '{"errorMessages":["Can not find a comment for the id: 12345."],"errors":{}}', :headers => {})
+    assert_raises(Jirarest2::NotFoundError) {
+      comment.get()
+    }
+  end
+
+
+  def test__get_special_comment_success
+    comment = Comment.new(@con,"SP-3",10109)
+    stub_request(:get, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-3/comment/10109").with(:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 200, :body => '{"self":"http://localhost:2990/jira/rest/api/2/issue/10104/comment/10109","id":"10109","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"Text for the long run","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-27T20:36:07.651+0200","updated":"2012-07-27T20:36:07.651+0200"}', :headers => {})
+    result = comment.get
+    assert_equal "Text for the long run", result[0].text
+    assert_equal "Test User", result[0].author
+    assert_instance_of CommentElement, result[0]
+    assert_instance_of Time, result[0].mdate
+  end
+
+
+  def test_replace_comment
+    comment = Comment.new(@con,"SP-3",10210)
+    stub_request(:put, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-3/comment/10210").with(:body => "{\"body\":\"New Comment\"}",:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 200, :body => '{"self":"http://localhost:2990/jira/rest/api/2/issue/10104/comment/10210","id":"10210","author":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"body":"New Comment","updateAuthor":{"self":"http://localhost:2990/jira/rest/api/2/user?username=test","name":"test","emailAddress":"jira-test@localhost","avatarUrls":{"16x16":"http://localhost:2990/jira/secure/useravatar?size=small&avatarId=10122","48x48":"http://localhost:2990/jira/secure/useravatar?avatarId=10122"},"displayName":"Test User","active":true},"created":"2012-07-29T17:31:32.230+0200","updated":"2012-07-29T18:06:22.943+0200"}', :headers => {})
+    
+    result = comment.update("New Comment")
+    assert_equal "New Comment", result.text
+    assert_equal "Test User", result.author
+    assert_instance_of Time, result.mdate  
+
+  end
+
+  
+  def test_delete_comment
+    comment = Comment.new(@con,"SP-3",10208)
+    stub_request(:delete, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-3/comment/10208").with(:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 204 )
+    assert_equal "204", comment.delete.code
+  end
+
+end # class

data/test/test_connect.rb

@@ -20,13 +20,9 @@ class TestConnect < MiniTest::Unit::TestCase
   
   def test_executePOST
     stub_request(:post, "http://test:1234@localhost:2990/jira/rest/api/2/search/").with(:body => "{\"jql\":\"project = MFTP\",\"startAt\":0,\"maxResults\":4}",:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 200, :body => '{"expand":"schema,names","startAt":0,"maxResults":4,"total":9,"issues":[{"expand":"editmeta,renderedFields,transitions,changelog,operations","id":"10102","self":"http://localhost:2990/jira/rest/api/2/issue/10102","key":"MFTP-9","fields":{"summary":"AnotherissueatSunJul1516","progress":{"progress":0,"total":0}}}]}', :headers => {})
-#    WebMock.disable!
 
     query={"jql"=>"project = MFTP", "startAt"=>0, "maxResults"=>4 }
     assert_equal 4, @con.execute("Post","search/",query).result["maxResults"]
-#   WebMock.enable!
-
-    pp @con.execute("Post","search/",query).result["maxResults"]
   end
 
   def test_check_uri_true

data/test/test_result.rb

@@ -17,7 +17,9 @@ class TestResult < MiniTest::Unit::TestCase
 
   def test_code_400
     stub_request(:get, "http://test:1234@localhost:2990/jira/rest/api/2/issue/createmeta/").with(:headers => {'Content-Type'=>'application/json;charset=UTF-8'}).to_return(:status => 400, :body => "", :headers => {})
-    assert_equal "400", @con.execute("Get","issue/createmeta/","").code
+    assert_raises(Jirarest2::BadRequestError) { 
+      @con.execute("Get","issue/createmeta/","").code
+    }
   end
 
 

data/test/test_watcher.rb

@@ -43,7 +43,7 @@ class TestWatcher < MiniTest::Unit::TestCase
   def test_add_watcher_success
     stub_request(:post, "http://test:1234@localhost:2990/jira/rest/api/2/issue/SP-1/watchers").with(:body => "\"cebit\"",:headers => {'Accept'=>'*/*', 'Content-Type'=>'application/json;charset=UTF-8', 'User-Agent'=>'Ruby'}).to_return(:status => 204, :headers => {})
     watchers = Watcher.new(@con, "SP-1")
-    assert true, watchers.add_watcher("cebit")
+    assert_equal true, watchers.add_watcher("cebit")
   end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jirarest2
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
   prerelease: 
 platform: ruby
 authors:
@@ -50,7 +50,7 @@ cert_chain:
   -----END CERTIFICATE-----
 
 '
-date: 2012-07-26 00:00:00.000000000 Z
+date: 2012-08-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -145,6 +145,7 @@ email:
 - cebit-jirarest@gunnet.de
 executables:
 - create_issue.rb
+- jira_comment
 - jira_create_issue
 - jira_create_issue.rb
 - jira_watcher
@@ -161,6 +162,7 @@ files:
 - README.txt
 - Rakefile
 - bin/create_issue.rb
+- bin/jira_comment
 - bin/jira_create_issue
 - bin/jira_create_issue.rb
 - bin/jira_watcher
@@ -173,16 +175,19 @@ files:
 - lib/jirarest2/madbitconfig.rb
 - lib/jirarest2/result.rb
 - lib/jirarest2/services.rb
+- lib/jirarest2/services/comment.rb
 - lib/jirarest2/services/issuelink.rb
 - lib/jirarest2/services/issuelinktype.rb
 - lib/jirarest2/services/watcher.rb
 - lib/jirarest2bin.rb
+- test/data/get-comments.txt
 - test/data/issuespec.txt
 - test/data/test.config.data
 - test/data/test.json
 - test/data/test.nojson
 - test/data/test.nojson1
 - test/data/ticket.json
+- test/test_comment.rb
 - test/test_connect.rb
 - test/test_credentials.rb
 - test/test_issue.rb
@@ -224,6 +229,7 @@ test_files:
 - test/test_madbitconfig.rb
 - test/test_connect.rb
 - test/test_issuelinktype.rb
+- test/test_comment.rb
 - test/test_credentials.rb
 - test/test_issue.rb
 - test/test_result.rb