[ARVADOS] updated: 1.3.0-1005-g22722aab8

[Git user]

Summary of changes:
 services/api/test/integration/collections_api_test.rb | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

       via  22722aab8f5aaf33d75975738d7e4f9c46150722 (commit)
      from  d289f9642981adc8b332a03caf31fe2a93fd313f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 22722aab8f5aaf33d75975738d7e4f9c46150722
Author: Lucas Di Pentima <ldipentima at veritasgenetics.com>
Date:   Wed Jun 5 17:52:48 2019 -0300

    15306: Fixes tests to POST form & json encoded params.
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima at veritasgenetics.com>

diff --git a/services/api/test/integration/collections_api_test.rb b/services/api/test/integration/collections_api_test.rb
index 7bbae6e35..86195fba7 100644
--- a/services/api/test/integration/collections_api_test.rb
+++ b/services/api/test/integration/collections_api_test.rb
@@ -285,8 +285,9 @@ class CollectionsApiTest < ActionDispatch::IntegrationTest
       expired_col = collections(:expired_collection)
       assert expired_col.is_trashed
       # Try #index first
-      get "/arvados/v1/collections",
+      post "/arvados/v1/collections",
           params: {
+            :_method => 'GET',
             :include_trash => param,
             :filters => [['uuid', '=', expired_col.uuid]].to_json
           },
@@ -295,9 +296,9 @@ class CollectionsApiTest < ActionDispatch::IntegrationTest
       assert_not_nil json_response['items']
       assert_equal truthiness, json_response['items'].collect {|c| c['uuid']}.include?(expired_col.uuid)
       # Try #show next
-      get "/arvados/v1/collections/#{expired_col.uuid}",
+      post "/arvados/v1/collections/#{expired_col.uuid}",
         params: {
-          :format => :json,
+          :_method => 'GET',
           :include_trash => param,
         },
         headers: auth(:active)
@@ -345,11 +346,12 @@ class CollectionsApiTest < ActionDispatch::IntegrationTest
       expired_col = collections(:expired_collection)
       assert expired_col.is_trashed
       params = [
+        ['_method', 'GET'],
         ['include_trash', param],
         ['filters', [['uuid','=',expired_col.uuid]].to_json],
       ]
       # Try #index first
-      get "/arvados/v1/collections",
+      post "/arvados/v1/collections",
         params: URI.encode_www_form(params),
         headers: {
           "Content-type" => "application/x-www-form-urlencoded"
@@ -358,8 +360,8 @@ class CollectionsApiTest < ActionDispatch::IntegrationTest
       assert_not_nil json_response['items']
       assert_equal truthiness, json_response['items'].collect {|c| c['uuid']}.include?(expired_col.uuid)
       # Try #show next
-      get "/arvados/v1/collections/#{expired_col.uuid}",
-        params: URI.encode_www_form([['include_trash', param]]),
+      post "/arvados/v1/collections/#{expired_col.uuid}",
+        params: URI.encode_www_form([['_method', 'GET'],['include_trash', param]]),
         headers: {
           "Content-type" => "application/x-www-form-urlencoded"
         }.update(auth(:active))

-----------------------------------------------------------------------


hooks/post-receive
--