1 # Copyright (C) The Arvados Authors. All rights reserved.
3 # SPDX-License-Identifier: AGPL-3.0
7 class Arvados::V1::GroupsControllerTest < ActionController::TestCase
9 test "attempt to delete group without read or write access" do
10 authorize_with :active
11 post :destroy, params: {id: groups(:empty_lonely_group).uuid}
15 test "attempt to delete group without write access" do
16 authorize_with :active
17 post :destroy, params: {id: groups(:all_users).uuid}
21 test "get list of projects" do
22 authorize_with :active
23 get :index, params: {filters: [['group_class', '=', 'project']], format: :json}
24 assert_response :success
26 json_response['items'].each do |group|
27 assert_equal 'project', group['group_class']
28 group_uuids << group['uuid']
30 assert_includes group_uuids, groups(:aproject).uuid
31 assert_includes group_uuids, groups(:asubproject).uuid
32 assert_not_includes group_uuids, groups(:system_group).uuid
33 assert_not_includes group_uuids, groups(:private).uuid
36 test "get list of groups that are not projects" do
37 authorize_with :active
38 get :index, params: {filters: [['group_class', '!=', 'project']], format: :json}
39 assert_response :success
41 json_response['items'].each do |group|
42 assert_not_equal 'project', group['group_class']
43 group_uuids << group['uuid']
45 assert_not_includes group_uuids, groups(:aproject).uuid
46 assert_not_includes group_uuids, groups(:asubproject).uuid
47 assert_includes group_uuids, groups(:private).uuid
48 assert_includes group_uuids, groups(:group_with_no_class).uuid
51 test "get list of groups with bogus group_class" do
52 authorize_with :active
54 filters: [['group_class', '=', 'nogrouphasthislittleclass']],
57 assert_response :success
58 assert_equal [], json_response['items']
59 assert_equal 0, json_response['items_available']
62 def check_project_contents_response disabled_kinds=[]
63 assert_response :success
64 assert_operator 2, :<=, json_response['items_available']
65 assert_operator 2, :<=, json_response['items'].count
66 kinds = json_response['items'].collect { |i| i['kind'] }.uniq
67 expect_kinds = %w'arvados#group arvados#specimen arvados#pipelineTemplate arvados#job' - disabled_kinds
68 assert_equal expect_kinds, (expect_kinds & kinds)
70 json_response['items'].each do |i|
71 if i['kind'] == 'arvados#group'
72 assert(i['group_class'] == 'project',
73 "group#contents returned a non-project group")
77 disabled_kinds.each do |d|
78 assert_equal true, !kinds.include?(d)
82 test 'get group-owned objects' do
83 authorize_with :active
84 get :contents, params: {
85 id: groups(:aproject).uuid,
88 check_project_contents_response
91 test "user with project read permission can see project objects" do
92 authorize_with :project_viewer
93 get :contents, params: {
94 id: groups(:aproject).uuid,
97 check_project_contents_response
100 test "list objects across projects" do
101 authorize_with :project_viewer
102 get :contents, params: {
104 filters: [['uuid', 'is_a', 'arvados#specimen']]
106 assert_response :success
107 found_uuids = json_response['items'].collect { |i| i['uuid'] }
108 [[:in_aproject, true],
109 [:in_asubproject, true],
110 [:owned_by_private_group, false]].each do |specimen_fixture, should_find|
112 assert_includes found_uuids, specimens(specimen_fixture).uuid, "did not find specimen fixture '#{specimen_fixture}'"
114 refute_includes found_uuids, specimens(specimen_fixture).uuid, "found specimen fixture '#{specimen_fixture}'"
119 test "list trashed collections and projects" do
120 authorize_with :active
121 get(:contents, params: {
125 ['uuid', 'is_a', ['arvados#collection', 'arvados#group']],
126 ['is_trashed', '=', true],
130 assert_response :success
131 found_uuids = json_response['items'].collect { |i| i['uuid'] }
132 assert_includes found_uuids, groups(:trashed_project).uuid
133 refute_includes found_uuids, groups(:aproject).uuid
134 assert_includes found_uuids, collections(:expired_collection).uuid
135 refute_includes found_uuids, collections(:w_a_z_file).uuid
138 test "list objects in home project" do
139 authorize_with :active
140 get :contents, params: {
143 id: users(:active).uuid
145 assert_response :success
146 found_uuids = json_response['items'].collect { |i| i['uuid'] }
147 assert_includes found_uuids, specimens(:owned_by_active_user).uuid, "specimen did not appear in home project"
148 refute_includes found_uuids, specimens(:in_asubproject).uuid, "specimen appeared unexpectedly in home project"
151 test "user with project read permission can see project collections" do
152 authorize_with :project_viewer
153 get :contents, params: {
154 id: groups(:asubproject).uuid,
157 ids = json_response['items'].map { |item| item["uuid"] }
158 assert_includes ids, collections(:baz_file_in_asubproject).uuid
162 ['collections.name', 'asc', :<=, "name"],
163 ['collections.name', 'desc', :>=, "name"],
164 ['name', 'asc', :<=, "name"],
165 ['name', 'desc', :>=, "name"],
166 ['collections.created_at', 'asc', :<=, "created_at"],
167 ['collections.created_at', 'desc', :>=, "created_at"],
168 ['created_at', 'asc', :<=, "created_at"],
169 ['created_at', 'desc', :>=, "created_at"],
170 ].each do |column, order, operator, field|
171 test "user with project read permission can sort projects on #{column} #{order}" do
172 authorize_with :project_viewer
173 get :contents, params: {
174 id: groups(:asubproject).uuid,
176 filters: [['uuid', 'is_a', "arvados#collection"]],
177 order: "#{column} #{order}"
179 sorted_values = json_response['items'].collect { |item| item[field] }
181 # Here we avoid assuming too much about the database
182 # collation. Both "alice"<"Bob" and "alice">"Bob" can be
183 # correct. Hopefully it _is_ safe to assume that if "a" comes
184 # before "b" in the ascii alphabet, "aX">"bY" is never true for
185 # any strings X and Y.
186 reliably_sortable_names = sorted_values.select do |name|
187 name[0] >= 'a' && name[0] <= 'z'
191 # Preserve order of sorted_values. But do not use &=. If
192 # sorted_values has out-of-order duplicates, we want to preserve
193 # them here, so we can detect them and fail the test below.
194 sorted_values.select! do |name|
195 reliably_sortable_names.include? name
198 assert_sorted(operator, sorted_values)
202 def assert_sorted(operator, sorted_items)
203 actually_checked_anything = false
205 sorted_items.each do |entry|
207 assert_operator(previous, operator, entry,
208 "Entries sorted incorrectly.")
209 actually_checked_anything = true
213 assert actually_checked_anything, "Didn't even find two items to compare."
216 # Even though the project_viewer tests go through other controllers,
217 # I'm putting them here so they're easy to find alongside the other
219 def check_new_project_link_fails(link_attrs)
220 @controller = Arvados::V1::LinksController.new
221 post :create, params: {
223 link_class: "permission",
225 head_uuid: groups(:aproject).uuid,
228 assert_includes(403..422, response.status)
231 test "user with project read permission can't add users to it" do
232 authorize_with :project_viewer
233 check_new_project_link_fails(tail_uuid: users(:spectator).uuid)
236 test "user with project read permission can't add items to it" do
237 authorize_with :project_viewer
238 check_new_project_link_fails(tail_uuid: collections(:baz_file).uuid)
241 test "user with project read permission can't rename items in it" do
242 authorize_with :project_viewer
243 @controller = Arvados::V1::LinksController.new
244 post :update, params: {
245 id: jobs(:running).uuid,
246 name: "Denied test name",
248 assert_includes(403..404, response.status)
251 test "user with project read permission can't remove items from it" do
252 @controller = Arvados::V1::PipelineTemplatesController.new
253 authorize_with :project_viewer
254 post :update, params: {
255 id: pipeline_templates(:two_part).uuid,
257 owner_uuid: users(:project_viewer).uuid,
263 test "user with project read permission can't delete it" do
264 authorize_with :project_viewer
265 post :destroy, params: {id: groups(:aproject).uuid}
269 test 'get group-owned objects with limit' do
270 authorize_with :active
271 get :contents, params: {
272 id: groups(:aproject).uuid,
276 assert_response :success
277 assert_operator 1, :<, json_response['items_available']
278 assert_equal 1, json_response['items'].count
281 test 'get group-owned objects with limit and offset' do
282 authorize_with :active
283 get :contents, params: {
284 id: groups(:aproject).uuid,
289 assert_response :success
290 assert_operator 1, :<, json_response['items_available']
291 assert_equal 0, json_response['items'].count
294 test 'get group-owned objects with additional filter matching nothing' do
295 authorize_with :active
296 get :contents, params: {
297 id: groups(:aproject).uuid,
298 filters: [['uuid', 'in', ['foo_not_a_uuid','bar_not_a_uuid']]],
301 assert_response :success
302 assert_equal [], json_response['items']
303 assert_equal 0, json_response['items_available']
306 %w(offset limit).each do |arg|
307 ['foo', '', '1234five', '0x10', '-8'].each do |val|
308 test "Raise error on bogus #{arg} parameter #{val.inspect}" do
309 authorize_with :active
310 get :contents, params: {
311 :id => groups(:aproject).uuid,
320 test "Collection contents don't include manifest_text" do
321 authorize_with :active
322 get :contents, params: {
323 id: groups(:aproject).uuid,
324 filters: [["uuid", "is_a", "arvados#collection"]],
327 assert_response :success
328 refute(json_response["items"].any? { |c| not c["portable_data_hash"] },
329 "response included an item without a portable data hash")
330 refute(json_response["items"].any? { |c| c.include?("manifest_text") },
331 "response included an item with a manifest text")
334 test 'get writable_by list for owned group' do
335 authorize_with :active
337 id: groups(:aproject).uuid,
340 assert_response :success
341 assert_not_nil(json_response['writable_by'],
342 "Should receive uuid list in 'writable_by' field")
343 assert_includes(json_response['writable_by'], users(:active).uuid,
344 "owner should be included in writable_by list")
347 test 'no writable_by list for group with read-only access' do
348 authorize_with :rominiadmin
350 id: groups(:testusergroup_admins).uuid,
353 assert_response :success
354 assert_equal([json_response['owner_uuid']],
355 json_response['writable_by'],
356 "Should only see owner_uuid in 'writable_by' field")
359 test 'get writable_by list by admin user' do
360 authorize_with :admin
362 id: groups(:testusergroup_admins).uuid,
365 assert_response :success
366 assert_not_nil(json_response['writable_by'],
367 "Should receive uuid list in 'writable_by' field")
368 assert_includes(json_response['writable_by'],
370 "Current user should be included in 'writable_by' field")
373 test 'creating subproject with duplicate name fails' do
374 authorize_with :active
375 post :create, params: {
378 owner_uuid: users(:active).uuid,
379 group_class: 'project',
383 response_errors = json_response['errors']
384 assert_not_nil response_errors, 'Expected error in response'
385 assert(response_errors.first.include?('duplicate key'),
386 "Expected 'duplicate key' error in #{response_errors.first}")
389 test 'creating duplicate named subproject succeeds with ensure_unique_name' do
390 authorize_with :active
391 post :create, params: {
394 owner_uuid: users(:active).uuid,
395 group_class: 'project',
397 ensure_unique_name: true
399 assert_response :success
400 new_project = json_response
401 assert_not_equal(new_project['uuid'],
402 groups(:aproject).uuid,
403 "create returned same uuid as existing project")
404 assert_match(/^A Project \(\d{4}-\d\d-\d\dT\d\d:\d\d:\d\d\.\d{3}Z\)$/,
409 [['owner_uuid', '!=', 'zzzzz-tpzed-xurymjxw79nv3jz'], 200,
410 'zzzzz-d1hrv-subprojpipeline', 'zzzzz-d1hrv-1xfj6xkicf2muk2'],
411 [["pipeline_instances.state", "not in", ["Complete", "Failed"]], 200,
412 'zzzzz-d1hrv-1xfj6xkicf2muk2', 'zzzzz-d1hrv-i3e77t9z5y8j9cc'],
413 [['container_requests.requesting_container_uuid', '=', nil], 200,
414 'zzzzz-xvhdp-cr4queuedcontnr', 'zzzzz-xvhdp-cr4requestercn2'],
415 [['container_requests.no_such_column', '=', nil], 422],
416 [['container_requests.', '=', nil], 422],
417 [['.requesting_container_uuid', '=', nil], 422],
418 [['no_such_table.uuid', '!=', 'zzzzz-tpzed-xurymjxw79nv3jz'], 422],
419 ].each do |filter, expect_code, expect_uuid, not_expect_uuid|
420 test "get contents with '#{filter}' filter" do
421 authorize_with :active
422 get :contents, params: {filters: [filter], format: :json}
423 assert_response expect_code
424 if expect_code == 200
425 assert_not_empty json_response['items']
426 item_uuids = json_response['items'].collect {|item| item['uuid']}
427 assert_includes(item_uuids, expect_uuid)
428 assert_not_includes(item_uuids, not_expect_uuid)
433 test 'get contents with jobs and pipeline instances disabled' do
434 Rails.configuration.API.DisabledAPIs = {'jobs.index'=>{}, 'pipeline_instances.index'=>{}}
436 authorize_with :active
437 get :contents, params: {
438 id: groups(:aproject).uuid,
441 check_project_contents_response %w'arvados#pipelineInstance arvados#job'
444 test 'get contents with low max_index_database_read' do
445 # Some result will certainly have at least 12 bytes in a
447 Rails.configuration.API.MaxIndexDatabaseRead = 12
448 authorize_with :active
449 get :contents, params: {
450 id: groups(:aproject).uuid,
453 assert_response :success
454 assert_not_empty(json_response['items'])
455 assert_operator(json_response['items'].count,
456 :<, json_response['items_available'])
459 test 'get contents, recursive=true' do
460 authorize_with :active
462 id: groups(:aproject).uuid,
466 get :contents, params: params
467 owners = json_response['items'].map do |item|
470 assert_includes(owners, groups(:aproject).uuid)
471 assert_includes(owners, groups(:asubproject).uuid)
474 [false, nil].each do |recursive|
475 test "get contents, recursive=#{recursive.inspect}" do
476 authorize_with :active
478 id: groups(:aproject).uuid,
481 params[:recursive] = false if recursive == false
482 get :contents, params: params
483 owners = json_response['items'].map do |item|
486 assert_includes(owners, groups(:aproject).uuid)
487 refute_includes(owners, groups(:asubproject).uuid)
491 test 'get home project contents, recursive=true' do
492 authorize_with :active
493 get :contents, params: {
494 id: users(:active).uuid,
498 owners = json_response['items'].map do |item|
501 assert_includes(owners, users(:active).uuid)
502 assert_includes(owners, groups(:aproject).uuid)
503 assert_includes(owners, groups(:asubproject).uuid)
506 ### trashed project tests ###
508 [:active, :admin].each do |auth|
509 # project: to query, to untrash, is visible, parent contents listing success
510 [[:trashed_project, [], false, true],
511 [:trashed_project, [:trashed_project], true, true],
512 [:trashed_subproject, [], false, false],
513 [:trashed_subproject, [:trashed_project], true, true],
514 [:trashed_subproject3, [:trashed_project], false, true],
515 [:trashed_subproject3, [:trashed_subproject3], false, false],
516 [:trashed_subproject3, [:trashed_project, :trashed_subproject3], true, true],
517 ].each do |project, untrash, visible, success|
519 test "contents listing #{project} #{untrash} as #{auth}" do
522 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
524 get :contents, params: {
525 id: groups(project).owner_uuid,
529 assert_response :success
530 item_uuids = json_response['items'].map do |item|
534 assert_includes(item_uuids, groups(project).uuid)
536 assert_not_includes(item_uuids, groups(project).uuid)
543 test "contents of #{project} #{untrash} as #{auth}" do
546 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
548 get :contents, params: {
549 id: groups(project).uuid,
553 assert_response :success
559 test "index #{project} #{untrash} as #{auth}" do
562 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
564 get :index, params: {
567 assert_response :success
568 item_uuids = json_response['items'].map do |item|
572 assert_includes(item_uuids, groups(project).uuid)
574 assert_not_includes(item_uuids, groups(project).uuid)
578 test "show #{project} #{untrash} as #{auth}" do
581 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
584 id: groups(project).uuid,
588 assert_response :success
594 test "show include_trash=false #{project} #{untrash} as #{auth}" do
597 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
600 id: groups(project).uuid,
605 assert_response :success
611 test "show include_trash #{project} #{untrash} as #{auth}" do
614 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
617 id: groups(project).uuid,
621 assert_response :success
624 test "index include_trash #{project} #{untrash} as #{auth}" do
627 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
629 get :index, params: {
633 assert_response :success
634 item_uuids = json_response['items'].map do |item|
637 assert_includes(item_uuids, groups(project).uuid)
641 test "delete project #{auth}" do
643 [:trashed_project].each do |pr|
644 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
646 assert !Group.find_by_uuid(groups(:trashed_project).uuid).is_trashed
647 post :destroy, params: {
648 id: groups(:trashed_project).uuid,
651 assert_response :success
652 assert Group.find_by_uuid(groups(:trashed_project).uuid).is_trashed
655 test "untrash project #{auth}" do
657 assert Group.find_by_uuid(groups(:trashed_project).uuid).is_trashed
658 post :untrash, params: {
659 id: groups(:trashed_project).uuid,
662 assert_response :success
663 assert !Group.find_by_uuid(groups(:trashed_project).uuid).is_trashed
666 test "untrash project with name conflict #{auth}" do
668 [:trashed_project].each do |pr|
669 Group.find_by_uuid(groups(pr).uuid).update! is_trashed: false
671 gc = Group.create!({owner_uuid: "zzzzz-j7d0g-trashedproject1",
672 name: "trashed subproject 3",
673 group_class: "project"})
674 post :untrash, params: {
675 id: groups(:trashed_subproject3).uuid,
677 ensure_unique_name: true
679 assert_response :success
680 assert_match /^trashed subproject 3 \(\d{4}-\d\d-\d\d.*?Z\)$/, json_response['name']
683 test "move trashed subproject to new owner #{auth}" do
685 assert_nil Group.readable_by(users(auth)).where(uuid: groups(:trashed_subproject).uuid).first
686 put :update, params: {
687 id: groups(:trashed_subproject).uuid,
689 owner_uuid: users(:active).uuid
694 assert_response :success
695 assert_not_nil Group.readable_by(users(auth)).where(uuid: groups(:trashed_subproject).uuid).first
699 test 'get shared owned by another user' do
700 authorize_with :user_bar_in_sharing_group
702 act_as_system_user do
704 tail_uuid: users(:user_bar_in_sharing_group).uuid,
705 link_class: 'permission',
707 head_uuid: groups(:project_owned_by_foo).uuid)
710 get :shared, params: {:filters => [["group_class", "=", "project"]], :include => "owner_uuid"}
712 assert_equal 1, json_response['items'].length
713 assert_equal json_response['items'][0]["uuid"], groups(:project_owned_by_foo).uuid
715 assert_equal 1, json_response['included'].length
716 assert_equal json_response['included'][0]["uuid"], users(:user_foo_in_sharing_group).uuid
719 test 'get shared, owned by unreadable project' do
720 authorize_with :user_bar_in_sharing_group
722 act_as_system_user do
723 Group.find_by_uuid(groups(:project_owned_by_foo).uuid).update!(owner_uuid: groups(:aproject).uuid)
725 tail_uuid: users(:user_bar_in_sharing_group).uuid,
726 link_class: 'permission',
728 head_uuid: groups(:project_owned_by_foo).uuid)
731 get :shared, params: {:filters => [["group_class", "=", "project"]], :include => "owner_uuid"}
733 assert_equal 1, json_response['items'].length
734 assert_equal json_response['items'][0]["uuid"], groups(:project_owned_by_foo).uuid
736 assert_equal 0, json_response['included'].length
739 test 'get shared, owned by non-project' do
740 authorize_with :user_bar_in_sharing_group
742 act_as_system_user do
743 Group.find_by_uuid(groups(:project_owned_by_foo).uuid).update!(owner_uuid: groups(:group_for_sharing_tests).uuid)
746 get :shared, params: {:filters => [["group_class", "=", "project"]], :include => "owner_uuid"}
748 assert_equal 1, json_response['items'].length
749 assert_equal json_response['items'][0]["uuid"], groups(:project_owned_by_foo).uuid
751 assert_equal 1, json_response['included'].length
752 assert_equal json_response['included'][0]["uuid"], groups(:group_for_sharing_tests).uuid
755 ### contents with exclude_home_project
757 test 'contents, exclude home owned by another user' do
758 authorize_with :user_bar_in_sharing_group
760 act_as_system_user do
762 tail_uuid: users(:user_bar_in_sharing_group).uuid,
763 link_class: 'permission',
765 head_uuid: groups(:project_owned_by_foo).uuid)
767 tail_uuid: users(:user_bar_in_sharing_group).uuid,
768 link_class: 'permission',
770 head_uuid: collections(:collection_owned_by_foo).uuid)
773 get :contents, params: {:include => "owner_uuid", :exclude_home_project => true}
775 assert_equal 2, json_response['items'].length
776 assert_equal json_response['items'][0]["uuid"], groups(:project_owned_by_foo).uuid
777 assert_equal json_response['items'][1]["uuid"], collections(:collection_owned_by_foo).uuid
779 assert_equal 1, json_response['included'].length
780 assert_equal json_response['included'][0]["uuid"], users(:user_foo_in_sharing_group).uuid
783 test 'contents, exclude home, owned by unreadable project' do
784 authorize_with :user_bar_in_sharing_group
786 act_as_system_user do
787 Group.find_by_uuid(groups(:project_owned_by_foo).uuid).update!(owner_uuid: groups(:aproject).uuid)
789 tail_uuid: users(:user_bar_in_sharing_group).uuid,
790 link_class: 'permission',
792 head_uuid: groups(:project_owned_by_foo).uuid)
795 get :contents, params: {:include => "owner_uuid", :exclude_home_project => true}
797 assert_equal 1, json_response['items'].length
798 assert_equal json_response['items'][0]["uuid"], groups(:project_owned_by_foo).uuid
800 assert_equal 0, json_response['included'].length
803 test 'contents, exclude home, owned by non-project' do
804 authorize_with :user_bar_in_sharing_group
806 act_as_system_user do
807 Group.find_by_uuid(groups(:project_owned_by_foo).uuid).update!(owner_uuid: groups(:group_for_sharing_tests).uuid)
810 get :contents, params: {:include => "owner_uuid", :exclude_home_project => true}
812 assert_equal 1, json_response['items'].length
813 assert_equal json_response['items'][0]["uuid"], groups(:project_owned_by_foo).uuid
815 assert_equal 1, json_response['included'].length
816 assert_equal json_response['included'][0]["uuid"], groups(:group_for_sharing_tests).uuid
819 test 'contents, exclude home, with parent specified' do
820 authorize_with :active
822 get :contents, params: {id: groups(:aproject).uuid, :include => "owner_uuid", :exclude_home_project => true}