fix(personality insights): Fix bug with content type to be accepted

Author: maxnussbaum

lib/ibm_watson/watson_service.rb

@@ -123,7 +123,7 @@ def request(args)
     args[:json] = args[:data].merge(args[:json]) if args[:data].respond_to?(:merge)
     args[:json] = args[:data] if args[:json].empty? || (args[:data].instance_of?(String) && !args[:data].empty?)
     args[:json].delete_if { |_k, v| v.nil? } if args[:json].instance_of?(Hash)
-    args[:headers]["Accept"] = "application/json" if args[:accept_json]
+    args[:headers]["Accept"] = "application/json" if args[:accept_json] && args[:headers]["Accept"].nil?
     args[:headers]["Content-Type"] = "application/json" unless args[:headers].key?("Content-Type")
     args[:json] = args[:json].to_json if args[:json].instance_of?(Hash)
     args[:headers].delete_if { |_k, v| v.nil? } if args[:headers].instance_of?(Hash)

test/test_helper.rb

@@ -12,6 +12,9 @@
         File.basename(src_file.filename) == "version.rb"
       end
 
+      lib_proc = proc { |file| !file.filename.include?("service_extensions") }
+      add_group "Services", "/lib/", &lib_proc
+      add_group "Patches", "/lib/ibm_watson/service_extensions/"
       command_name "Minitest"
     end
   end

test/unit/test_personality_insights_v3.rb

@@ -105,7 +105,7 @@ def test_json_to_csv
       .with(
         body: personality_text,
         headers: {
-          "Accept" => "application/json",
+          "Accept" => "text/csv",
           "Authorization" => "Basic dXNlcm5hbWU6cGFzc3dvcmQ=",
           "Content-Type" => "application/json",
           "Host" => "gateway.watsonplatform.net"

test/unit/test_vcap_using_personality_insights.rb

@@ -87,7 +87,7 @@ def test_json_to_csv
       .with(
         body: personality_text,
         headers: {
-          "Accept" => "application/json",
+          "Accept" => "text/csv",
           "Authorization" => "Basic dXNlcm5hbWU6cGFzc3dvcmQ=",
           "Content-Type" => "application/json",
           "Host" => "gateway.watsonplatform.net"