Skip to content

Commit 47f99b4

Browse files
MarcialRosalesmergify[bot]
authored andcommitted
Update tests to check against 404 rather than 401
when the user is not authorized to access teh vhost (cherry picked from commit 9c5b68f)
1 parent f491f68 commit 47f99b4

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

deps/rabbitmq_stream_management/test/http_SUITE.erl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ stream_tracking_requires_vhost_access(Config) ->
204204
rabbit_ct_broker_helpers:add_user(Config, User, User),
205205
rabbit_ct_broker_helpers:set_user_tags(Config, 0, User, [management]),
206206
http_get(Config, "/stream/tracking-vh/test-stream/tracking",
207-
User, User, ?NOT_AUTHORISED),
207+
User, User, 404),
208208
%% Permissions granted, must succeed.
209209
rabbit_ct_broker_helpers:set_full_permissions(Config, User, Vhost),
210210
http_get(Config, "/stream/tracking-vh/test-stream/tracking",

deps/rabbitmq_stream_management/test/http_SUITE_data/src/test/java/com/rabbitmq/stream/HttpTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -746,7 +746,7 @@ void permissions() throws Exception {
746746
assertThatThrownBy(
747747
() ->
748748
toMaps(get(client, "/stream" + configuration.endpoint + request.endpoint)))
749-
.hasMessageContaining("401");
749+
.hasMessageContaining("404");
750750
}
751751
}
752752
for (Entry<String, Boolean> request : configuration.vhostConnections.entrySet()) {

0 commit comments

Comments
 (0)