From 151713138ba2b8b37a0f40cccee6a99dc004c34d Mon Sep 17 00:00:00 2001
From: Igor Yova <iyova@playtika.com>
Date: Wed, 10 Apr 2024 16:04:35 +0200
Subject: [PATCH] Revert "WIP: Add functionality marge for build extensions
 section (#36)" (#53)

This reverts commit cb4810a3c93925c56c05e6602352d64ddb2e0f4c.
---
 .../mixin-example/mixin1/pom.xml              |  9 ---
 .../mixin-maven-plugin/mixin-example/pom.xml  |  2 +-
 .../com/playtika/maven/plugins/Mixin.java     |  9 +--
 .../maven/plugins/MixinModelMerger.java       | 14 +---
 .../com/playtika/maven/plugins/Mixins.java    |  5 --
 .../MixinModelMergeBuildExtensions.java       | 75 -------------------
 6 files changed, 6 insertions(+), 108 deletions(-)
 delete mode 100755 mixin-maven-plugin/src/test/java/com/playtika/maven/plugins/MixinModelMergeBuildExtensions.java

diff --git a/examples/mixin-maven-plugin/mixin-example/mixin1/pom.xml b/examples/mixin-maven-plugin/mixin-example/mixin1/pom.xml
index f641662..831ac56 100755
--- a/examples/mixin-maven-plugin/mixin-example/mixin1/pom.xml
+++ b/examples/mixin-maven-plugin/mixin-example/mixin1/pom.xml
@@ -13,15 +13,6 @@
         <message>HELLO DEFAULT MESSAGE FROM MIXIN</message>
     </properties>
     <build>
-        <extensions>
-            <extension>
-                <groupId>org.apache.maven.extensions</groupId>
-                <artifactId>maven-build-cache-extension</artifactId>
-                <version>1.0.1</version>
-            </extension>
-        </extensions>
-
-
         <pluginManagement>
             <plugins>
                 <!-- This is a simple execution of antrun during initialize phase to echo a message from a property.
diff --git a/examples/mixin-maven-plugin/mixin-example/pom.xml b/examples/mixin-maven-plugin/mixin-example/pom.xml
index 59ab088..26e0046 100755
--- a/examples/mixin-maven-plugin/mixin-example/pom.xml
+++ b/examples/mixin-maven-plugin/mixin-example/pom.xml
@@ -7,7 +7,7 @@
     <name>${project.groupId}:${project.artifactId}</name>
 
     <properties>
-        <version.com.playtika.maven.plugins>1.0.0-SNAPSHOT</version.com.playtika.maven.plugins>
+        <version.com.playtika.maven.plugins>1.0.1</version.com.playtika.maven.plugins>
     </properties>
     <modules>
         <module>mixin1</module>
diff --git a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixin.java b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixin.java
index 6cd36ec..9d61805 100755
--- a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixin.java
+++ b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixin.java
@@ -19,7 +19,6 @@ public class Mixin {
     private Boolean mergeScm;
     private Boolean mergeRepositories;
     private Boolean mergePluginRepositories;
-    private Boolean mergeBuildExtensions;
     private Boolean recurse;
     private Boolean activateProfiles;
     private Mixins mixins;
@@ -114,10 +113,6 @@ public boolean isMergePluginRepositories() {
         return mergePluginRepositories != null ? mergePluginRepositories : mixins.isMergePluginRepositories();
     }
 
-    public boolean isMergeBuildExtensions() {
-        return mergeBuildExtensions != null ? mergeBuildExtensions : mixins.isMergeBuildExtensions();
-    }
-
     public void setMergePluginRepositories(Boolean mergePluginRepositories) {
         this.mergePluginRepositories = mergePluginRepositories;
     }
@@ -173,13 +168,11 @@ public void merge(Model mixinModel, MavenProject mavenProject, MavenSession mave
         if (isMergePluginRepositories()) {
             mixinModelMerger.mergePluginRepositories(mavenProject.getModel(), mixinModel);
         }
-        if (isMergeBuildExtensions()) {
-            mixinModelMerger.mergeBuildExtensions(mavenProject.getModel(), mixinModel);
-        }
     }
 
     @Override
     public String toString() {
         return getKey();
     }
+
 }
diff --git a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/MixinModelMerger.java b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/MixinModelMerger.java
index 0298261..bc46e1c 100755
--- a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/MixinModelMerger.java
+++ b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/MixinModelMerger.java
@@ -1,9 +1,5 @@
 package com.playtika.maven.plugins;
 
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-
 import org.apache.maven.model.Build;
 import org.apache.maven.model.DistributionManagement;
 import org.apache.maven.model.Model;
@@ -17,6 +13,10 @@
 import org.apache.maven.model.merge.MavenModelMerger;
 import org.codehaus.plexus.component.annotations.Component;
 
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
 /**
  * MixinModelMerger
  * Enables pluginManagement and properties import from different pom files. Uses protected methods of MavenModelMerger, and therefore inherits from it
@@ -187,10 +187,4 @@ public void mergePluginRepositories(Model targetModel, Model sourceModel) {
         super.mergeModelBase_PluginRepositories(targetModel, sourceModel, false, context);
     }
 
-    public void mergeBuildExtensions(Model targetModel, Model sourceModel) {
-        Map<Object, Object> context = new HashMap<Object, Object>();
-        Build targetBuild = targetModel.getBuild();
-        Build sourceBuild = sourceModel.getBuild();
-        super.mergeBuild_Extensions(targetBuild, sourceBuild, false, context);
-    }
 }
diff --git a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixins.java b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixins.java
index 793b375..8187bcf 100755
--- a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixins.java
+++ b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/Mixins.java
@@ -7,7 +7,6 @@ public class Mixins {
     private List<Mixin> mixins = new ArrayList<Mixin>();
     private boolean mergePlugins = true;
     private boolean mergePluginManagement = true;
-    private boolean mergeBuildExtensions = true;
     private boolean mergeProperties = true;
     private boolean mergeReporting = true;
     private boolean mergeDistributionManagement = true;
@@ -112,10 +111,6 @@ public boolean isMergePluginRepositories() {
         return mergePluginRepositories;
     }
 
-    public boolean isMergeBuildExtensions() {
-        return mergeBuildExtensions;
-    }
-
     public void setMergePluginRepositories(boolean mergePluginRepositories) {
         this.mergePluginRepositories = mergePluginRepositories;
     }
diff --git a/mixin-maven-plugin/src/test/java/com/playtika/maven/plugins/MixinModelMergeBuildExtensions.java b/mixin-maven-plugin/src/test/java/com/playtika/maven/plugins/MixinModelMergeBuildExtensions.java
deleted file mode 100755
index db4fa8b..0000000
--- a/mixin-maven-plugin/src/test/java/com/playtika/maven/plugins/MixinModelMergeBuildExtensions.java
+++ /dev/null
@@ -1,75 +0,0 @@
-package com.playtika.maven.plugins;
-
-import java.util.List;
-
-import org.apache.maven.model.Build;
-import org.apache.maven.model.Extension;
-import org.apache.maven.model.Model;
-import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Test;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertNotNull;
-
-
-public class MixinModelMergeBuildExtensions {
-
-    private MixinModelMerger merger;
-    private Model buildExtensions, emptyBuildExtensions;
-
-    @BeforeEach
-    public void setup() {
-        merger = new MixinModelMerger();
-        buildExtensions = new Model();
-        buildExtensions.setBuild(new Build());
-        Extension extension = new Extension();
-        extension.setGroupId("com.test");
-        extension.setArtifactId("test-magre-extension");
-        extension.setVersion("1.0.0");
-        buildExtensions.getBuild().getExtensions().add(extension);
-
-        emptyBuildExtensions = new Model();
-        emptyBuildExtensions.setBuild(new Build());
-    }
-
-    @Test
-    public void testEmptyTarget() {
-        List<Extension> origExtensions = buildExtensions.getBuild().getExtensions();
-        assertNotNull(origExtensions);
-        assertEquals(origExtensions.size(), 1);
-        merger.mergeBuildExtensions(emptyBuildExtensions, buildExtensions);
-        assertEquals(origExtensions, buildExtensions.getBuild().getExtensions());
-    }
-
-    @Test
-    public void testEmptySrc() {
-        merger.mergeScm(buildExtensions, emptyBuildExtensions);
-        assertNotNull(emptyBuildExtensions.getBuild().getExtensions());
-        assertEquals(emptyBuildExtensions.getBuild().getExtensions().size(), 0);
-
-        assertNotNull(buildExtensions.getBuild().getExtensions());
-        assertEquals(buildExtensions.getBuild().getExtensions().size(), 1);
-    }
-
-    @Test
-    public void testMageFromSrc() {
-        emptyBuildExtensions = new Model();
-        emptyBuildExtensions.setBuild(new Build());
-        Extension extension = new Extension();
-        extension.setGroupId("com.test2");
-        extension.setArtifactId("test-magre-extension-v2");
-        extension.setVersion("2.0.0");
-        emptyBuildExtensions.getBuild().getExtensions().add(extension);
-
-        assertNotNull(buildExtensions.getBuild().getExtensions());
-        assertEquals(1, buildExtensions.getBuild().getExtensions().size());
-
-        assertNotNull(emptyBuildExtensions.getBuild().getExtensions());
-        assertEquals(1, emptyBuildExtensions.getBuild().getExtensions().size());
-
-        merger.mergeBuildExtensions(buildExtensions, emptyBuildExtensions);
-
-        assertNotNull(buildExtensions.getBuild().getExtensions());
-        assertEquals(2, buildExtensions.getBuild().getExtensions().size());
-    }
-}