Hoi Rafael,

Perhaps that's an intermittent test error based on the random order that 
packages are loaded.

Anyway, if you got it working without problems, feel free to fix and send a PR! 
I really appreciate you looking after those things. :)

Once your PR is through I will have a look and merge it.

Thanks again and keep the good  work. ;)

Cheers,
Wilder

Sent from my iPhone

> On 22 May 2015, at 23:11, rsafonseca <g...@git.apache.org> wrote:
> 
> Github user rsafonseca commented on the pull request:
> 
>    https://github.com/apache/cloudstack/pull/280#issuecomment-104771495
> 
>    @wilderrodrigues the Xenserver tests are failing in master with a 
> ClassCast Exception :(
> 
>    
> */plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/wrapper/citrix/CitrixRequestWrapper.java*
>    `    Reflections baseWrappers = new 
> Reflections("com.cloud.hypervisor.xenserver.resource.wrapper");
>        @SuppressWarnings("rawtypes")
>        Set<Class<? extends CommandWrapper>> baseSet = 
> baseWrappers.getSubTypesOf(CommandWrapper.class);` 
> 
>    The baseSet ends up becoming the sum of all tests, because the scanner is 
> recursive, so wrapper.xen56, wrapper.xcp, etc will all get included in the 
> set.
> 
>    I've tested fixing it by changing the package for the Citrix* classes to a 
> package that in not parent to the other wrapper's classes and it's working 
> fine.
>    Do you want to fix this your way, or should I send a PR? :)
> 
> 
> 
> 
> 
> 
> 
> ---
> If your project is set up for it, you can reply to this email and have your
> reply appear on GitHub as well. If your project does not have this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---

Reply via email to