Skip to content

Commit 959e90e

Browse files
author
Simon MacMullen
committed
How many bugs can there be in this code? Previously we would succeed if exp were true but the server issued ACCESS_REFUSED anyway. Let's fix that too.
1 parent 17d5985 commit 959e90e

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

test/src/com/rabbitmq/client/test/server/Permissions.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -341,9 +341,11 @@ protected void runTest(boolean exp, String name, WithName test)
341341
test.with(name);
342342
assertTrue(msg, exp);
343343
} catch (IOException e) {
344+
assertFalse(msg, exp);
344345
checkShutdownSignal(AMQP.ACCESS_REFUSED, e);
345346
openChannel();
346347
} catch (AlreadyClosedException e) {
348+
assertFalse(msg, exp);
347349
checkShutdownSignal(AMQP.ACCESS_REFUSED, e);
348350
openChannel();
349351
}

0 commit comments

Comments
 (0)