You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Keith Wall (JIRA)" <ji...@apache.org> on 2012/09/09 22:20:07 UTC

[jira] [Commented] (QPID-4284) Remove system test dependencies on development and localhost virtual hosts

    [ https://issues.apache.org/jira/browse/QPID-4284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13451673#comment-13451673 ] 

Keith Wall commented on QPID-4284:
----------------------------------

I believe the patch for this change is complete, but running the tests with this patch reveals what I think is an existing problem (Test ClientAcknowledgeTest on the java-dby.0-9-1 profile).

{code}
Testcase: testClientAckWithLargeFlusherPeriod took 4.401 sec
        FAILED
Unexpected message is received expected:<2> but was:<1>
junit.framework.AssertionFailedError: Unexpected message is received expected:<2> but was:<1>
        at org.apache.qpid.test.unit.ack.ClientAcknowledgeTest.testClientAckWithLargeFlusherPeriod(ClientAcknowledgeTest.java:80)
        at org.apache.qpid.test.utils.QpidBrokerTestCase.runBare(QpidBrokerTestCase.java:242)
        at org.apache.qpid.test.utils.QpidTestCase.run(QpidTestCase.java:137)
{code}



It appears to me that the Broker is failing to process the message ack, and for this reason, the wrong message (message 1) is redelivered at restart.

{code}


{code}

                
> Remove system test dependencies on development and localhost virtual hosts
> --------------------------------------------------------------------------
>
>                 Key: QPID-4284
>                 URL: https://issues.apache.org/jira/browse/QPID-4284
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>            Reporter: Keith Wall
>            Assignee: Keith Wall
>             Fix For: 0.19
>
>         Attachments: 0001-QPID-4284-Remove-system-test-dependencies-on-develop.patch, TEST-org.apache.qpid.test.unit.ack.ClientAcknowledgeTest.testClientAckWithLargeFlusherPeriod-derby-0-9-1.out
>
>
> Currently the system tests rely on three virtualhost (namely test, development and localhost) being defined in the system test configuration.  This arrangement has the disadvantage that each system test must initialise three virtual hosts and for persistent stores (derby/BDB), this adds significant time to the test run.  On the whole, tests use only the 'test' virtualhost.
> This change will remove the dependency on the  development and localhost virtual hosts.   It will also remove the remaining few dependencies on the hardcoded queues "ping" and "queue".
> In addition to the time saving, this change will help when we come to eliminate the configuration files entirely.
>  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org