diff --git a/lib/vimaly/client.rb b/lib/vimaly/client.rb index c79dc3c..79e11ea 100644 --- a/lib/vimaly/client.rb +++ b/lib/vimaly/client.rb @@ -45,7 +45,7 @@ def update_ticket(id, other_fields) response = put("/tickets/#{id}", ticket.to_json(custom_field_name_map, true)) case response.status when 200..299 - true + id else log_warn "status: #{response.status}" log_warn " #{response.inspect}" diff --git a/test/client_test.rb b/test/client_test.rb index 2ccc38e..2b8e988 100644 --- a/test/client_test.rb +++ b/test/client_test.rb @@ -98,22 +98,24 @@ class ClientTest < Minitest::Test should "succeed for standard params" do stub_update_ticket - @client.update_ticket( - 123, - title: 'updated title', - description: 'updated description' + id = @client.update_ticket( + 123, + title: 'updated title', + description: 'updated description' ) + assert_equal 123, id end should "succeed for custom params" do stub_update_ticket_custom - @client.update_ticket( - 123, - title: 'updated title', - description: 'updated description', - 'Last seen': Date.new(2016,1,20) + id = @client.update_ticket( + 123, + title: 'updated title', + description: 'updated description', + 'Last seen': Date.new(2016,1,20) ) + assert_equal 123, id end end # updating tickets