[MNG-6275] Maven Embedder compatible fix

- Need to use PARENT_CLASSLOADER so that when used by embedder we get a
consistent view of the available classes

- Restores test case from f047ea1437
This commit is contained in:
Stephen Connolly 2017-08-24 11:33:01 +02:00
parent c9a288d8b1
commit 27a2bda3f4
2 changed files with 102 additions and 1 deletions

View File

@ -120,7 +120,7 @@ public class DefaultClassRealmManager
{
try
{
ClassRealm classRealm = world.newRealm( realmId, null );
ClassRealm classRealm = world.newRealm( realmId, PARENT_CLASSLOADER );
if ( logger.isDebugEnabled() )
{

View File

@ -0,0 +1,101 @@
package org.apache.maven.classrealm;
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/
import java.util.ServiceLoader;
import javax.script.ScriptEngineFactory;
import org.apache.maven.model.Model;
import org.apache.maven.model.Plugin;
import org.codehaus.plexus.ContainerConfiguration;
import org.codehaus.plexus.PlexusConstants;
import org.codehaus.plexus.PlexusTestCase;
import org.codehaus.plexus.classworlds.realm.ClassRealm;
import org.junit.Test;
public class DefaultClassRealmManagerTest extends PlexusTestCase
{
private ClassRealmManager classRealmManager;
@Override
protected void setUp()
throws Exception
{
super.setUp();
this.classRealmManager = lookup( ClassRealmManager.class );
}
@Override
protected void customizeContainerConfiguration( ContainerConfiguration configuration )
{
configuration.setClassPathScanning( PlexusConstants.SCANNING_INDEX );
}
@Test
public void testMNG6275_pluginRealmDefaultParentClassLoader()
{
Plugin plugin = new Plugin();
plugin.setVersion( "VERSION" );
ClassLoader parent = null;
ClassRealm pluginRealm = classRealmManager.createPluginRealm( plugin, parent, null, null, null );
ServiceLoader<ScriptEngineFactory> sef = ServiceLoader.load( ScriptEngineFactory.class, pluginRealm );
assertTrue( sef.iterator().hasNext() );
}
@Test
public void testMNG6275_extensionRealmDefaultParentClassLoader()
{
Plugin extension = new Plugin();
extension.setVersion( "VERSION" );
ClassRealm extensionRealm = classRealmManager.createExtensionRealm( extension, null );
ServiceLoader<ScriptEngineFactory> sef = ServiceLoader.load( ScriptEngineFactory.class, extensionRealm );
assertTrue( sef.iterator().hasNext() );
}
@Test
public void testMNG6275_projectRealmDefaultParentClassLoader()
{
Model model = new Model();
ClassRealm projectRealm = classRealmManager.createProjectRealm( model, null );
ServiceLoader<ScriptEngineFactory> sef = ServiceLoader.load( ScriptEngineFactory.class, projectRealm );
assertTrue( sef.iterator().hasNext() );
}
@Test
public void testMNG6275_mavenApiRealmDefaultParentClassLoader()
{
ClassRealm mavenApiRealm = classRealmManager.getMavenApiRealm();
ServiceLoader<ScriptEngineFactory> sef = ServiceLoader.load( ScriptEngineFactory.class, mavenApiRealm );
assertTrue( sef.iterator().hasNext() );
}
@Test
public void testMNG6275_coreRealmDefaultParentClassLoader()
{
ClassRealm coreRealm = classRealmManager.getCoreRealm();
ServiceLoader<ScriptEngineFactory> sef = ServiceLoader.load( ScriptEngineFactory.class, coreRealm );
assertTrue( sef.iterator().hasNext() );
}
}