[MNG-7055] Fix G level metadata handling

maven-compat/pom.xml

@@ -108,6 +108,11 @@ under the License.
       <artifactId>plexus-testing</artifactId>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.mockito</groupId>
+      <artifactId>mockito-core</artifactId>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.apache.maven.wagon</groupId>
       <artifactId>wagon-file</artifactId>

maven-compat/src/test/java/org/apache/maven/artifact/deployer/ArtifactDeployerTest.java

@@ -24,12 +24,15 @@
 import org.apache.maven.artifact.AbstractArtifactComponentTestCase;
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.execution.MavenSession;
+import org.apache.maven.session.scope.internal.SessionScope;
 import org.codehaus.plexus.util.FileUtils;
 import org.junit.jupiter.api.Test;
 
 import static org.codehaus.plexus.testing.PlexusExtension.getBasedir;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.mockito.Mockito.mock;
 
 import javax.inject.Inject;
 
@@ -42,6 +45,9 @@ public class ArtifactDeployerTest
     @Inject
     private ArtifactDeployer artifactDeployer;
 
+    @Inject
+    private SessionScope sessionScope;
+
     protected String component()
     {
         return "deployer";
@@ -51,18 +57,28 @@ protected String component()
     public void testArtifactInstallation()
         throws Exception
     {
-        String artifactBasedir = new File( getBasedir(), "src/test/resources/artifact-install" ).getAbsolutePath();
+        sessionScope.enter();
+        try
+        {
+            sessionScope.seed(MavenSession.class, mock(MavenSession.class));
+
+            String artifactBasedir = new File( getBasedir(), "src/test/resources/artifact-install" ).getAbsolutePath();
 
-        Artifact artifact = createArtifact( "artifact", "1.0" );
+            Artifact artifact = createArtifact( "artifact", "1.0" );
 
-        File file = new File( artifactBasedir, "artifact-1.0.jar" );
-        assertEquals( "dummy", FileUtils.fileRead( file, "UTF-8" ).trim() );
+            File file = new File( artifactBasedir, "artifact-1.0.jar" );
+            assertEquals( "dummy", FileUtils.fileRead( file, "UTF-8" ).trim() );
 
-        artifactDeployer.deploy( file, artifact, remoteRepository(), localRepository() );
+            artifactDeployer.deploy( file, artifact, remoteRepository(), localRepository() );
 
-        ArtifactRepository remoteRepository = remoteRepository();
-        File deployedFile = new File( remoteRepository.getBasedir(), remoteRepository.pathOf( artifact ) );
-        assertTrue( deployedFile.exists() );
-        assertEquals( "dummy", FileUtils.fileRead( deployedFile, "UTF-8" ).trim() );
+            ArtifactRepository remoteRepository = remoteRepository();
+            File deployedFile = new File( remoteRepository.getBasedir(), remoteRepository.pathOf( artifact ) );
+            assertTrue( deployedFile.exists() );
+            assertEquals( "dummy", FileUtils.fileRead( deployedFile, "UTF-8" ).trim() );
+        }
+        finally
+        {
+            sessionScope.exit();
+        }
     }
 }

maven-compat/src/test/java/org/apache/maven/artifact/installer/ArtifactInstallerTest.java

@@ -23,11 +23,14 @@
 
 import org.apache.maven.artifact.AbstractArtifactComponentTestCase;
 import org.apache.maven.artifact.Artifact;
+import org.apache.maven.execution.MavenSession;
+import org.apache.maven.session.scope.internal.SessionScope;
 import org.junit.jupiter.api.Test;
 
 import javax.inject.Inject;
 
 import static org.codehaus.plexus.testing.PlexusExtension.getBasedir;
+import static org.mockito.Mockito.mock;
 
 /**
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
@@ -38,6 +41,9 @@ public class ArtifactInstallerTest
     @Inject
     private ArtifactInstaller artifactInstaller;
 
+    @Inject
+    private SessionScope sessionScope;
+
     protected String component()
     {
         return "installer";
@@ -47,14 +53,24 @@ protected String component()
     public void testArtifactInstallation()
         throws Exception
     {
-        String artifactBasedir = new File( getBasedir(), "src/test/resources/artifact-install" ).getAbsolutePath();
+        sessionScope.enter();
+        try
+        {
+            sessionScope.seed(MavenSession.class, mock(MavenSession.class));
+
+            String artifactBasedir = new File( getBasedir(), "src/test/resources/artifact-install" ).getAbsolutePath();
 
-        Artifact artifact = createArtifact( "artifact", "1.0" );
+            Artifact artifact = createArtifact( "artifact", "1.0" );
 
-        File source = new File( artifactBasedir, "artifact-1.0.jar" );
+            File source = new File( artifactBasedir, "artifact-1.0.jar" );
 
-        artifactInstaller.install( source, artifact, localRepository() );
+            artifactInstaller.install( source, artifact, localRepository() );
 
-        assertLocalArtifactPresent( artifact );
+            assertLocalArtifactPresent( artifact );
+        }
+        finally
+        {
+            sessionScope.exit();
+        }
     }
 }

maven-core/src/main/java/org/apache/maven/execution/infoproviders/DefaultPluginsMetadataInfoProvider.java

@@ -0,0 +1,137 @@
+package org.apache.maven.execution.infoproviders;
+
+/*
+ * 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.Objects;
+
+import javax.inject.Inject;
+import javax.inject.Named;
+import javax.inject.Provider;
+import javax.inject.Singleton;
+
+import org.apache.maven.artifact.repository.metadata.Metadata;
+import org.apache.maven.artifact.repository.metadata.Plugin;
+import org.apache.maven.artifact.repository.metadata.RepositoryMetadata;
+import org.apache.maven.execution.MavenSession;
+import org.apache.maven.project.MavenProject;
+import org.apache.maven.repository.internal.PluginsMetadataInfoProvider;
+import org.apache.maven.repository.legacy.metadata.ArtifactMetadata;
+import org.eclipse.aether.artifact.Artifact;
+
+import static java.util.Objects.requireNonNull;
+
+/**
+ * Default implementation of {@link PluginsMetadataInfoProvider}.
+ */
+@Named
+@Singleton
+public class DefaultPluginsMetadataInfoProvider
+    implements PluginsMetadataInfoProvider
+{
+    private final Provider<MavenSession> mavenSessionProvider;
+
+    @Inject
+    public DefaultPluginsMetadataInfoProvider( final Provider<MavenSession> mavenSessionProvider )
+    {
+        this.mavenSessionProvider = requireNonNull( mavenSessionProvider );
+    }
+
+    @Override
+    public PluginInfo getPluginInfo( final Artifact artifact )
+    {
+        MavenSession mavenSession = mavenSessionProvider.get();
+        if ( mavenSession != null )
+        {
+            MavenProject mavenProject = searchForProject( mavenSession, artifact );
+            if ( mavenProject != null && "maven-plugin".equals( mavenProject.getPackaging() ) )
+            {
+                Plugin plugin = searchForPluginGroupLevelRepositoryMetadata( mavenProject );
+
+                if ( plugin != null )
+                {
+                    return new PluginInfo()
+                    {
+                        @Override
+                        public String getPluginGroupId()
+                        {
+                            return artifact.getGroupId();
+                        }
+
+                        @Override
+                        public String getPluginArtifactId()
+                        {
+                            return artifact.getArtifactId();
+                        }
+
+                        @Override
+                        public String getPluginPrefix()
+                        {
+                            return plugin.getPrefix();
+                        }
+
+                        @Override
+                        public String getPluginName()
+                        {
+                            return plugin.getName();
+                        }
+                    };
+                }
+            }
+        }
+
+        return null;
+    }
+
+    private MavenProject searchForProject( MavenSession mavenSession, Artifact artifact )
+    {
+        for ( MavenProject mavenProject : mavenSession.getProjects() )
+        {
+            if ( mavenProject.getArtifact() != null
+                && Objects.equals( mavenProject.getGroupId(), artifact.getGroupId() )
+                && Objects.equals( mavenProject.getArtifactId(), artifact.getArtifactId() ) )
+            {
+                return mavenProject;
+            }
+        }
+        return null;
+    }
+
+    private Plugin searchForPluginGroupLevelRepositoryMetadata( MavenProject mavenProject )
+    {
+        org.apache.maven.artifact.Artifact projectArtifact = mavenProject.getArtifact();
+        for ( ArtifactMetadata artifactMetadata : projectArtifact.getMetadataList() )
+        {
+            if ( artifactMetadata instanceof RepositoryMetadata )
+            {
+                RepositoryMetadata repositoryMetadata = (RepositoryMetadata) artifactMetadata;
+                Metadata metadata = repositoryMetadata.getMetadata();
+
+                for ( Plugin plugin : metadata.getPlugins() )
+                {
+                    if ( Objects.equals( plugin.getArtifactId(), mavenProject.getArtifactId() ) )
+                    {
+                        return plugin;
+                    }
+                }
+            }
+        }
+        return null;
+    }
+}

maven-core/src/main/java/org/apache/maven/lifecycle/mapping/providers/MavenPluginLifecycleMappingProvider.java

@@ -74,13 +74,11 @@ public MavenPluginLifecycleMappingProvider()
     );
     lifecyclePhases.put(
         "install",
-        // TODO: MNG-6556: Do not upgrade to 3.0.0-M1 is does not install the plugin prefix metadata
-        new LifecyclePhase( "org.apache.maven.plugins:maven-install-plugin:2.5.2:install" )
+        new LifecyclePhase( "org.apache.maven.plugins:maven-install-plugin:3.0.0-M1:install" )
     );
     lifecyclePhases.put(
         "deploy",
-        // TODO: MNG-6556: Do not upgrade to 3.0.0-M1 is does not install the plugin prefix metadata
-        new LifecyclePhase( "org.apache.maven.plugins:maven-deploy-plugin:2.8.2:deploy" )
+        new LifecyclePhase( "org.apache.maven.plugins:maven-deploy-plugin:3.0.0-M1:deploy" )
     );
 
     Lifecycle lifecycle = new Lifecycle();