Hi Robbie,
        Thanks for the clarification. I was confused as there were other tests 
that didn't match the module, but I will go with your suggestion. 

Regards,

Weston
On May 3, 2012, at 11:31 AM, Robbie Gemmell wrote:

> Unit tests for a class in <module_x> should generally live in
> <module_x>/src/test/<etc to match the implementation class package>,
> so for PropertiesFileInitialContextFactory we would usually go for:
> 
> java/client/src/test/java/org/apache/qpid/jndi/PropertiesFileInitialContextFactoryTest.java
> 
> Robbie
> 
> On 3 May 2012 16:18, Weston M. Price <[email protected]> wrote:
>> I have a fix for https://issues.apache.org/jira/browse/QPID-3971. The fix is 
>> fairly simple, but I wanted to create a unit test for the 
>> PropertiesFileInitialContextFactory as we don't seem to have one. I am sort 
>> of confused about where the appropriate test for this should go. Looking at 
>> the tree, and since this doesn't actually require Broker connectivity, my 
>> best guess would be in:
>> 
>> java/client/src/test/java/org/apache/qpid/client
>> 
>> I would create a jndi directory and the test. The class itself lives in the 
>> client code so this would seem to make sense.
>> 
>> Thoughts?
>> 
>> Regards,
>> 
>> Weston
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to