Return-Path: X-Original-To: apmail-qpid-commits-archive@www.apache.org Delivered-To: apmail-qpid-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5FB2217709 for ; Tue, 10 Mar 2015 11:36:23 +0000 (UTC) Received: (qmail 19286 invoked by uid 500); 10 Mar 2015 11:36:17 -0000 Delivered-To: apmail-qpid-commits-archive@qpid.apache.org Received: (qmail 19256 invoked by uid 500); 10 Mar 2015 11:36:17 -0000 Mailing-List: contact commits-help@qpid.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@qpid.apache.org Delivered-To: mailing list commits@qpid.apache.org Received: (qmail 19187 invoked by uid 99); 10 Mar 2015 11:36:17 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 10 Mar 2015 11:36:17 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id D7840E025F; Tue, 10 Mar 2015 11:36:16 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: robbie@apache.org To: commits@qpid.apache.org Date: Tue, 10 Mar 2015 11:36:17 -0000 Message-Id: <87a710da66ba4e65b60cba0ef89294eb@git.apache.org> In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [2/2] qpid-jms git commit: add handler for the detach response, fix sporadic failure from race to close the test peer add handler for the detach response, fix sporadic failure from race to close the test peer Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/f7a12648 Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/f7a12648 Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/f7a12648 Branch: refs/heads/master Commit: f7a12648ca40ed6d740b7e4669da959800271137 Parents: 29706a8 Author: Robert Gemmell Authored: Tue Mar 10 10:34:53 2015 +0000 Committer: Robert Gemmell Committed: Tue Mar 10 10:34:53 2015 +0000 ---------------------------------------------------------------------- .../apache/qpid/jms/integration/SessionIntegrationTest.java | 6 ++++++ 1 file changed, 6 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/f7a12648/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/SessionIntegrationTest.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/SessionIntegrationTest.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/SessionIntegrationTest.java index b0ac37a..89f6ef0 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/SessionIntegrationTest.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/SessionIntegrationTest.java @@ -234,9 +234,15 @@ public class SessionIntegrationTest extends QpidJmsTestCase { try { if (topic) { testPeer.expectAndRefuseTempTopicCreationAttach(AmqpError.UNAUTHORIZED_ACCESS, "Not Authorized to create temp topics.", false); + //Expect the detach response to the test peer after refusal. + testPeer.expectDetach(true, false, false); + session.createTemporaryTopic(); } else { testPeer.expectAndRefuseTempQueueCreationAttach(AmqpError.UNAUTHORIZED_ACCESS, "Not Authorized to create temp queues.", false); + //Expect the detach response to the test peer after refusal. + testPeer.expectDetach(true, false, false); + session.createTemporaryQueue(); } fail("Should have thrown security exception"); --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org For additional commands, e-mail: commits-help@qpid.apache.org