You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Marnie McCormack (JIRA)" <qp...@incubator.apache.org> on 2008/10/30 12:27:44 UTC
[jira] Updated: (QPID-1271) Add VirtualHostFactory and
VirtualHostRegistry.getDefaultVirtualHost()
[ https://issues.apache.org/jira/browse/QPID-1271?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Marnie McCormack updated QPID-1271:
-----------------------------------
Fix Version/s: (was: M4)
Moving items not being worked on afaik out of M4 Fix Version
> Add VirtualHostFactory and VirtualHostRegistry.getDefaultVirtualHost()
> ----------------------------------------------------------------------
>
> Key: QPID-1271
> URL: https://issues.apache.org/jira/browse/QPID-1271
> Project: Qpid
> Issue Type: Improvement
> Components: Java Broker
> Affects Versions: M3
> Reporter: Martin Ritchie
>
> Summary:
> Currently our testing is quite fragile. Either tests rely on their being a "test" virtualhost or they they create a new VirtualHost.
> The first litters our tests with literal values and would require a large update if the "test" vhost was removed.
> The second approach leaves the VirtualHost unregisterd to any ApplicationRegistry and so is not correctly cleaned up by the ApplicationRegistry.remove()
> The addition of a VirtualHostFactory will improve the stability of our testing by ensuring the registration takes place for the instances where a new VirtualHost is required. The addition of the getDefaultVirtualHost() method to the VirtulalHostRegistry would allow the removal of the literal "test" in our current tests.
> The VirtualHostRegistry currently has a setDefaultVirtualHost but does not have a method to retrieve this value which could be used in place of the getVirtualHost("test") that our tests mainly use.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.