[ 
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to