projects
/
arvados.git
/ blobdiff
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
16007: Update group-sync tool for new restrictions on roles
[arvados.git]
/
services
/
api
/
test
/
integration
/
groups_test.rb
diff --git
a/services/api/test/integration/groups_test.rb
b/services/api/test/integration/groups_test.rb
index e45dd4eb5213954631314dd3fdf57cf4ebc44beb..7021761278d72143c277b06622504eae3c593334 100644
(file)
--- a/
services/api/test/integration/groups_test.rb
+++ b/
services/api/test/integration/groups_test.rb
@@
-193,23
+193,28
@@
class NonTransactionalGroupsTest < ActionDispatch::IntegrationTest
assert_response :success
end
assert_response :success
end
- test "create request with async=true d
efers
permissions update" do
- Rails.configuration.
async_permissions_update_i
nterval = 1 # second
+ test "create request with async=true d
oes not defer
permissions update" do
+ Rails.configuration.
API.AsyncPermissionsUpdateI
nterval = 1 # second
name = "Random group #{rand(1000)}"
assert_equal nil, Group.find_by_name(name)
name = "Random group #{rand(1000)}"
assert_equal nil, Group.find_by_name(name)
+ # Following the implementation of incremental permission updates
+ # (#16007) the async flag is now a no-op. Permission changes are
+ # visible immediately.
+
# Trigger the asynchronous permission update by using async=true parameter.
post "/arvados/v1/groups",
params: {
group: {
# Trigger the asynchronous permission update by using async=true parameter.
post "/arvados/v1/groups",
params: {
group: {
- name: name
+ name: name,
+ group_class: "project"
},
async: true
},
headers: auth(:active)
assert_response 202
},
async: true
},
headers: auth(:active)
assert_response 202
- # The group exists
on the database, but it's not accessible yet.
+ # The group exists
in the database
assert_not_nil Group.find_by_name(name)
get "/arvados/v1/groups",
params: {
assert_not_nil Group.find_by_name(name)
get "/arvados/v1/groups",
params: {
@@
-218,7
+223,7
@@
class NonTransactionalGroupsTest < ActionDispatch::IntegrationTest
},
headers: auth(:active)
assert_response 200
},
headers: auth(:active)
assert_response 200
- assert_equal
0
, json_response['items_available']
+ assert_equal
1
, json_response['items_available']
# Wait a bit and try again.
sleep(1)
# Wait a bit and try again.
sleep(1)