Skip to content

Commit

Permalink
Fix rabbit_db_queue_SUITE:update_decorators case
Browse files Browse the repository at this point in the history
This test called `rabbit_db_queue:update_decorators/1` which doesn't
exist - instead it can call `update_decorators/2` with an empty list.
This commit also adds the test to the `all_tests/0` list - it being
absent is why this wasn't caught before.

(cherry picked from commit 49c645a)
  • Loading branch information
the-mikedavis authored and mergify[bot] committed Aug 16, 2024
1 parent 49cf4a1 commit c129a9c
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion deps/rabbit/test/rabbit_db_queue_SUITE.erl
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ all_tests() ->
set,
delete,
update,
update_decorators,
exists,
get_all_durable,
get_all_durable_by_type,
Expand Down Expand Up @@ -323,7 +324,7 @@ update_decorators1(_Config) ->
?assertEqual({ok, Q}, rabbit_db_queue:get(QName)),
?assertEqual(undefined, amqqueue:get_decorators(Q)),
%% Not really testing we set a decorator, but at least the field is being updated
?assertEqual(ok, rabbit_db_queue:update_decorators(QName)),
?assertEqual(ok, rabbit_db_queue:update_decorators(QName, [])),
{ok, Q1} = rabbit_db_queue:get(QName),
?assertEqual([], amqqueue:get_decorators(Q1)),
passed.
Expand Down

0 comments on commit c129a9c

Please sign in to comment.