Cannot use with Play 1.2 GWT module and Cobertura about gwtmockito HOT 4 CLOSED

Thanks for helping add support here! I've added the line to support Cobertura by default in 3b88efb.

I'd like to understand your Play issue so that GwtMockito can support it without people having to fork code. Are these the only two lines you added?

classPool.appendClassPath(new LoaderClassPath(Enhancer.class.getClassLoader()));
classPool.appendClassPath(new ApplicationClassesClasspath());

You can extend GwtMockitoTestRunner to override getAdditionalClassPaths , adding your own classpath entries. However, that only supports String right now, not ClassPath objects from Javassist. Are LoaderClassPath and ApplicationClassesClasspath actually ClassPath objects defined by the Play library? If so, it wouldn't be hard to add another overridable version of getAdditionalClassPaths letting you return ClassPath objects.

from gwtmockito.

Are these the only two lines you added?

Not quite, there are two more changes:

protected ClassLoader getParentClassloader() {
    // return ClassLoader.getSystemClassLoader();
    return Play.classloader;
}

And (see GwtMockitoTestRunner.class.getClassLoader() replaced with Play GwtMockitoTestRunner.class.getClassLoader() ):

    @Override
    public Class<?> loadClass(String name) throws ClassNotFoundException {
        // If the class is in a blacklisted package, load it with the default classloader.
        for (String blacklistedPackage : getPackagesToLoadViaStandardClassloader()) {
            if (name.startsWith(blacklistedPackage)) {
                //return GwtMockitoTestRunner.class.getClassLoader().loadClass(name);
                return PlayGwtMockitoTestRunner.class.getClassLoader().loadClass(name);
            }
        }

        // Otherwise load it with our custom classloader.
        return super.loadClass(name);
    }

Are LoaderClassPath and ApplicationClassesClasspath actually ClassPath objects defined by the Play library? If so, it wouldn't be hard to add another overridable version of getAdditionalClassPaths letting you return ClassPath objects.

Yes, I suppose that'd do the trick. Thanks!

from gwtmockito.

Cool, thanks, I'll take a look at adding these as additional customization options before the next release, sounds like your workaround should work for now.

from gwtmockito.

Yes, I'm fine with workaround for now. Just wanted to share it.
Thanks for your effort.

from gwtmockito.