You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Robbie Gemmell (JIRA)" <ji...@apache.org> on 2011/03/24 16:31:05 UTC
[jira] [Created] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
improve unit testing of SimpleAMQQueue#processQueue()
-----------------------------------------------------
Key: QPID-3167
URL: https://issues.apache.org/jira/browse/QPID-3167
Project: Qpid
Issue Type: Sub-task
Components: Java Tests
Reporter: Robbie Gemmell
Fix For: 0.11
Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org
[jira] [Updated] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
Posted by "Keith Wall (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/QPID-3167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Keith Wall updated QPID-3167:
-----------------------------
Status: Open (was: Ready To Review)
> improve unit testing of SimpleAMQQueue#processQueue()
> -----------------------------------------------------
>
> Key: QPID-3167
> URL: https://issues.apache.org/jira/browse/QPID-3167
> Project: Qpid
> Issue Type: Sub-task
> Components: Java Tests
> Reporter: Robbie Gemmell
> Assignee: Keith Wall
> Fix For: 0.11
>
>
> Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org
[jira] [Resolved] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
Posted by "Robbie Gemmell (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/QPID-3167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Robbie Gemmell resolved QPID-3167.
----------------------------------
Resolution: Fixed
Assignee: Robbie Gemmell (was: Keith Wall)
Resolving this issue, have raised QPID-3183 for the feedback (since it isn't strictly part of the issue at hand as you mentioned).
> improve unit testing of SimpleAMQQueue#processQueue()
> -----------------------------------------------------
>
> Key: QPID-3167
> URL: https://issues.apache.org/jira/browse/QPID-3167
> Project: Qpid
> Issue Type: Sub-task
> Components: Java Tests
> Reporter: Robbie Gemmell
> Assignee: Robbie Gemmell
> Fix For: 0.11
>
>
> Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org
[jira] [Commented] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
Posted by "Keith Wall (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/QPID-3167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13014500#comment-13014500 ]
Keith Wall commented on QPID-3167:
----------------------------------
No comments against your actual changes but:
1) I think we should take this opportunity to add some class javadoc to the newly extracted QueueRunner.
2) In QueueRunner, _logger.error(e) is going to fail to log the underlying(s) in the broker log. This can make problem determination harder. Would be better to use the two argument form error(message,throwable)
3) I notice SubFlushRunner made its instance variables explicitly final. I think QueueRunner ought to have done the same. I think the stricter form is preferable.
I realise all of the above would have applied to code base prior to your change.
> improve unit testing of SimpleAMQQueue#processQueue()
> -----------------------------------------------------
>
> Key: QPID-3167
> URL: https://issues.apache.org/jira/browse/QPID-3167
> Project: Qpid
> Issue Type: Sub-task
> Components: Java Tests
> Reporter: Robbie Gemmell
> Assignee: Keith Wall
> Fix For: 0.11
>
>
> Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org
[jira] [Assigned] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
Posted by "Robbie Gemmell (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/QPID-3167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Robbie Gemmell reassigned QPID-3167:
------------------------------------
Assignee: Keith Wall (was: Robbie Gemmell)
Keith, could you review please? Thanks.
> improve unit testing of SimpleAMQQueue#processQueue()
> -----------------------------------------------------
>
> Key: QPID-3167
> URL: https://issues.apache.org/jira/browse/QPID-3167
> Project: Qpid
> Issue Type: Sub-task
> Components: Java Tests
> Reporter: Robbie Gemmell
> Assignee: Keith Wall
> Fix For: 0.11
>
>
> Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org
[jira] [Assigned] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
Posted by "Robbie Gemmell (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/QPID-3167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Robbie Gemmell reassigned QPID-3167:
------------------------------------
Assignee: Robbie Gemmell
> improve unit testing of SimpleAMQQueue#processQueue()
> -----------------------------------------------------
>
> Key: QPID-3167
> URL: https://issues.apache.org/jira/browse/QPID-3167
> Project: Qpid
> Issue Type: Sub-task
> Components: Java Tests
> Reporter: Robbie Gemmell
> Assignee: Robbie Gemmell
> Fix For: 0.11
>
>
> Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org
[jira] [Updated] (QPID-3167) improve unit testing of
SimpleAMQQueue#processQueue()
Posted by "Robbie Gemmell (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/QPID-3167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Robbie Gemmell updated QPID-3167:
---------------------------------
Status: Ready To Review (was: In Progress)
> improve unit testing of SimpleAMQQueue#processQueue()
> -----------------------------------------------------
>
> Key: QPID-3167
> URL: https://issues.apache.org/jira/browse/QPID-3167
> Project: Qpid
> Issue Type: Sub-task
> Components: Java Tests
> Reporter: Robbie Gemmell
> Assignee: Robbie Gemmell
> Fix For: 0.11
>
>
> Improve unit testing of SimpleAMQQueue#processQueue(), in particular providing coverage of the scenario encountered in the parent JIRA.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org