[arvados] updated: 2.6.0-96-gff843a40d
git repository hosting
git at public.arvados.org
Wed May 3 17:50:43 UTC 2023
Summary of changes:
sdk/python/arvados-v1-discovery.json | 6 ++++++
services/api/test/functional/arvados/v1/schema_controller_test.rb | 2 +-
services/api/test/integration/collections_api_test.rb | 4 ++--
3 files changed, 9 insertions(+), 3 deletions(-)
via ff843a40de882d22dd2a5e408c77a2fa4720cc7d (commit)
from 385d9ac5f47bec5c5a5fc9770c74b1a7d8dd2974 (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 ff843a40de882d22dd2a5e408c77a2fa4720cc7d
Author: Peter Amstutz <peter.amstutz at curii.com>
Date: Wed May 3 13:50:31 2023 -0400
20470: Fix tests
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>
diff --git a/sdk/python/arvados-v1-discovery.json b/sdk/python/arvados-v1-discovery.json
index 2fb9678c3..f015978bd 100644
--- a/sdk/python/arvados-v1-discovery.json
+++ b/sdk/python/arvados-v1-discovery.json
@@ -3055,6 +3055,12 @@
"description": "",
"location": "query"
},
+ "select": {
+ "type": "array",
+ "description": "Attributes of each object to return in the response.",
+ "required": false,
+ "location": "query"
+ },
"distinct": {
"type": "boolean",
"required": false,
diff --git a/services/api/test/functional/arvados/v1/schema_controller_test.rb b/services/api/test/functional/arvados/v1/schema_controller_test.rb
index f96f1af53..65a2b64b8 100644
--- a/services/api/test/functional/arvados/v1/schema_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/schema_controller_test.rb
@@ -83,7 +83,7 @@ class Arvados::V1::SchemaControllerTest < ActionController::TestCase
group_index_params = discovery_doc['resources']['groups']['methods']['index']['parameters']
group_contents_params = discovery_doc['resources']['groups']['methods']['contents']['parameters']
- assert_equal group_contents_params.keys.sort, (group_index_params.keys - ['select'] + ['uuid', 'recursive', 'include', 'include_old_versions']).sort
+ assert_equal group_contents_params.keys.sort, (group_index_params.keys + ['uuid', 'recursive', 'include', 'include_old_versions']).sort
recursive_param = group_contents_params['recursive']
assert_equal 'boolean', recursive_param['type']
diff --git a/services/api/test/integration/collections_api_test.rb b/services/api/test/integration/collections_api_test.rb
index baca6e278..1ac5eb5c7 100644
--- a/services/api/test/integration/collections_api_test.rb
+++ b/services/api/test/integration/collections_api_test.rb
@@ -95,7 +95,7 @@ class CollectionsApiTest < ActionDispatch::IntegrationTest
},
headers: auth(:active)
assert_response 422
- assert_match(/Invalid attribute.*bogus/, json_response['errors'].join(' '))
+ assert_match(/ArgumentError: Attribute selection list cannot be empty/, json_response['errors'].join(' '))
end
test "get index with select= (invalid attribute type) responds 422" do
@@ -106,7 +106,7 @@ class CollectionsApiTest < ActionDispatch::IntegrationTest
},
headers: auth(:active)
assert_response 422
- assert_match(/Invalid attribute.*bogus/, json_response['errors'].join(' '))
+ assert_match(/ArgumentError: Attribute selection list cannot be empty/, json_response['errors'].join(' '))
end
test "controller 404 response is json" do
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list