cstamas opened a new pull request #99: URL: https://github.com/apache/maven-resolver/pull/99
So far with c17902add6435d228038c6ed5d117604dc66bd41 we did run "demo snippets" (can be understood as some sort of "integration testing", but as backward compatibility testing, as it used Maven Resolver provider 3.5.0). As during recent work discovered, there were some problems with three kinds of "bindings" we provide: service locator, guice and sisu. This PR fixes those, and also changes the introduced test, by expanding it to run demos using serviceLocator, Guice and SISU. Changes: - up demo snippets maven-resolver-provider to 3.5.4 (was 3.5.0, contained relevant bugs re binding) - modify booter and demos to be able to select factory - modify existing UT to run same demos with all 3 factories - to be able to retain binary backward compat across all 3 factories, smaller changes were needed in impl module: introduced "deprecated" DefaultSyncContextFactory as well, that delegates to "latest" syncContextFactory. This is must to make things work same way in all factories. Hence, SL, guice and sisu also does this same thing, effective component dependencies are now same across all 3 factories. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
