Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Defaults.class's class loader instead of thread context loader #249

Open
wants to merge 2 commits into
base: microprofile-proto
from

Conversation

Projects
None yet
1 participant
@andymc12
Copy link

commented Apr 27, 2019

In my test case, I needed Jackson to be loadable from the Defaults class rather than the thread context class loader. This required a small change to the ClassUtils class, and since my test framework uses Java 2 security, I added some doPrivs to ensure it works.

I also updated the Microprofile API dependency to use the new repo's groupId rather than the sandbox.

The testExplicitPlusDefaultMappers(io.leangen.graphql.SchemaGeneratorConfigurationTest) test failed for me after these changes, but it failed before my changes too, so I don't think I did anything to break it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.