root
/
rabbitmq-server
mirror of https://github.com/rabbitmq/rabbitmq-server.git
1
0
Fork 0
Code Issues Actions Packages Projects Releases Wiki Activity

Make these tests more resilient to node data variations 

E.g. if other test suites created a bunch of users
first. We should not assume that there's only one user
present.
This commit is contained in:
Michael Klishin 2017-06-16 19:29:38 +03:00
parent 31822baf8c
commit 2d60573f03
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
deps/rabbitmq_cli/test/set_permissions_command_test.exs vendored
View File

@ -76,7 +76,8 @@ defmodule SetPermissionsCommandTest do
vhost_opts vhost_opts
) == :ok ) == :ok
assert List.first(list_permissions(context[:vhost]))[:configure] == "^#{context[:user]}-.*" u = Enum.find(list_permissions(context[:vhost]), fn(x) -> x[:user] == context[:user] end)
assert u[:configure] == "^#{context[:user]}-.*"
end end
test "run: throws a badrpc when instructed to contact an unreachable RabbitMQ node" do test "run: throws a badrpc when instructed to contact an unreachable RabbitMQ node" do
@ -93,8 +94,6 @@ defmodule SetPermissionsCommandTest do
[context[:user], "^#{context[:user]}-.*", ".*", ".*"], [context[:user], "^#{context[:user]}-.*", ".*", ".*"],
context[:opts] context[:opts]
) == {:error, {:no_such_user, context[:user]}} ) == {:error, {:no_such_user, context[:user]}}
assert List.first(list_permissions(context[:vhost]))[:configure] == ".*"
end end
@tag user: @user, vhost: "wintermute" @tag user: @user, vhost: "wintermute"
@ -108,14 +107,15 @@ defmodule SetPermissionsCommandTest do
end end
@tag user: @user, vhost: @root @tag user: @user, vhost: @root
test "run: Invalid regex patterns return error", context do test "run: invalid regex patterns return error", context do
assert @command.run( assert @command.run(
[context[:user], "^#{context[:user]}-.*", ".*", "*"], [context[:user], "^#{context[:user]}-.*", ".*", "*"],
context[:opts] context[:opts]
) == {:error, {:invalid_regexp, '*', {'nothing to repeat', 0}}} ) == {:error, {:invalid_regexp, '*', {'nothing to repeat', 0}}}
# asserts that the bad command didn't change anything # asserts that the failed command didn't change anything
assert List.first(list_permissions(context[:vhost])) == [user: context[:user], configure: ".*", write: ".*", read: ".*"] u = Enum.find(list_permissions(context[:vhost]), fn(x) -> x[:user] == context[:user] end)
assert u == [user: context[:user], configure: ".*", write: ".*", read: ".*"]
end end
@tag user: @user, vhost: @vhost @tag user: @user, vhost: @vhost