* AI: fixed wrong targeting for gain control abilities (#6340);

Mage.Sets/src/mage/cards/a/Annex.java

@@ -1,7 +1,5 @@
-
 package mage.cards.a;
 
-import java.util.UUID;
 import mage.abilities.Ability;
 import mage.abilities.common.SimpleStaticAbility;
 import mage.abilities.effects.common.AttachEffect;
@@ -16,17 +14,17 @@ import mage.constants.Zone;
 import mage.target.TargetPermanent;
 import mage.target.common.TargetLandPermanent;
 
+import java.util.UUID;
+
 /**
- *
  * @author andyfries
  */
 public final class Annex extends CardImpl {
 
     public Annex(UUID ownerId, CardSetInfo setInfo) {
-        super(ownerId,setInfo,new CardType[]{CardType.ENCHANTMENT},"{2}{U}{U}");
+        super(ownerId, setInfo, new CardType[]{CardType.ENCHANTMENT}, "{2}{U}{U}");
         this.subtype.add(SubType.AURA);
 
-
         // Enchant land
         TargetPermanent auraTarget = new TargetLandPermanent();
         this.getSpellAbility().addTarget(auraTarget);

Mage.Sets/src/mage/cards/c/ControlMagic.java

@@ -1,7 +1,5 @@
-
 package mage.cards.c;
 
-import java.util.UUID;
 import mage.abilities.Ability;
 import mage.abilities.common.SimpleStaticAbility;
 import mage.abilities.effects.common.AttachEffect;
@@ -10,30 +8,30 @@ import mage.abilities.keyword.EnchantAbility;
 import mage.cards.CardImpl;
 import mage.cards.CardSetInfo;
 import mage.constants.CardType;
-import mage.constants.SubType;
 import mage.constants.Outcome;
+import mage.constants.SubType;
 import mage.constants.Zone;
 import mage.target.TargetPermanent;
 import mage.target.common.TargetCreaturePermanent;
 
-/**
- *
- * @author KholdFuzion
+import java.util.UUID;
 
+/**
+ * @author KholdFuzion
  */
 public final class ControlMagic extends CardImpl {
 
     public ControlMagic(UUID ownerId, CardSetInfo setInfo) {
-        super(ownerId,setInfo,new CardType[]{CardType.ENCHANTMENT},"{2}{U}{U}");
+        super(ownerId, setInfo, new CardType[]{CardType.ENCHANTMENT}, "{2}{U}{U}");
         this.subtype.add(SubType.AURA);
 
-
         // Enchant creature
         TargetPermanent auraTarget = new TargetCreaturePermanent();
         this.getSpellAbility().addTarget(auraTarget);
         this.getSpellAbility().addEffect(new AttachEffect(Outcome.GainControl));
         Ability ability = new EnchantAbility(auraTarget.getTargetName());
         this.addAbility(ability);
+
         // You control enchanted creature.
         this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new ControlEnchantedEffect()));
     }

Mage.Sets/src/mage/cards/p/Preacher.java

@@ -38,7 +38,7 @@ public final class Preacher extends CardImpl {
         // You may choose not to untap Preacher during your untap step.
         this.addAbility(new SkipUntapOptionalAbility());
 
-        // {tap}: Gain control of target creature of an opponent's choice that they control for as long as Preacher remains tapped.
+        // {T}: Gain control of target creature of an opponent's choice that they control for as long as Preacher remains tapped.
         Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new PreacherEffect(), new TapSourceCost());
         ability.addTarget(new TargetOpponentsChoicePermanent(1, 1, new FilterControlledCreaturePermanent(), false));
         this.addAbility(ability);

Mage.Tests/src/test/java/org/mage/test/AI/basic/GainControlAITest.java

@@ -0,0 +1,75 @@
+package org.mage.test.AI.basic;
+
+import mage.constants.PhaseStep;
+import mage.constants.Zone;
+import org.junit.Test;
+import org.mage.test.serverside.base.CardTestPlayerBaseWithAIHelps;
+
+/**
+ * @author JayDi85
+ */
+public class GainControlAITest extends CardTestPlayerBaseWithAIHelps {
+
+    @Test
+    public void test_GainControl_Manual() {
+        // You control enchanted land.
+        addCard(Zone.HAND, playerA, "Annex", 1); // {2}{U}{U}
+        addCard(Zone.BATTLEFIELD, playerA, "Island", 4);
+        addCard(Zone.BATTLEFIELD, playerB, "Swamp", 1);
+
+        // take control
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Annex", "Swamp");
+
+        setStopAt(1, PhaseStep.END_TURN);
+        setStrictChooseMode(true);
+        execute();
+        assertAllCommandsUsed();
+
+        assertPermanentCount(playerA, "Island", 4);
+        assertPermanentCount(playerA, "Swamp", 1);
+        assertPermanentCount(playerB, "Swamp", 0);
+    }
+
+    @Test
+    public void test_GainControl_AI_Single() {
+        // You control enchanted land.
+        addCard(Zone.HAND, playerA, "Annex", 1); // {2}{U}{U}
+        addCard(Zone.BATTLEFIELD, playerA, "Island", 4);
+        addCard(Zone.BATTLEFIELD, playerB, "Swamp", 1);
+
+        // take control by AI
+        aiPlayPriority(1, PhaseStep.PRECOMBAT_MAIN, playerA);
+
+        setStopAt(1, PhaseStep.END_TURN);
+        setStrictChooseMode(true);
+        execute();
+        assertAllCommandsUsed();
+
+        assertPermanentCount(playerA, "Island", 4);
+        assertPermanentCount(playerA, "Swamp", 1);
+        assertPermanentCount(playerB, "Swamp", 0);
+    }
+
+    @Test
+    public void test_GainControl_AI_MostValuable() {
+        // You control enchanted land.
+        addCard(Zone.HAND, playerA, "Annex", 1); // {2}{U}{U}
+        addCard(Zone.BATTLEFIELD, playerA, "Island", 4);
+        addCard(Zone.BATTLEFIELD, playerB, "Swamp", 1);
+        addCard(Zone.BATTLEFIELD, playerB, "Badlands", 1);
+
+        // take control by AI (selects most valueable enemy card)
+        aiPlayPriority(1, PhaseStep.PRECOMBAT_MAIN, playerA);
+
+        setStopAt(1, PhaseStep.END_TURN);
+        setStrictChooseMode(true);
+        execute();
+        assertAllCommandsUsed();
+
+        assertPermanentCount(playerA, "Island", 4);
+        assertPermanentCount(playerA, "Swamp", 0);
+        assertPermanentCount(playerA, "Badlands", 1);
+        assertPermanentCount(playerB, "Swamp", 1);
+        assertPermanentCount(playerB, "Badlands", 0);
+    }
+}

Mage.Tests/src/test/java/org/mage/test/AI/basic/TargetControllerChangeTest.java

@@ -55,6 +55,6 @@ public class TargetControllerChangeTest extends CardTestPlayerBase {
         assertAllCommandsUsed();
 
         assertGraveyardCount(playerA, "Evangelize", 1);
-        assertPermanentCount(playerA, "Balduvian Bears", 1); // AI give smallest permanent to A as bad effect for B
+        assertPermanentCount(playerA, "Balduvian Bears", 1); // AI give smallest permanent to A as bad effect for target (target control change)
     }
 }

Mage/src/main/java/mage/constants/Outcome.java

@@ -17,7 +17,7 @@ public enum Outcome {
     PutCreatureInPlay(true),
     PutCardInPlay(true),
     PutLandInPlay(true),
-    GainControl(true),
+    GainControl(false),
     DrawCard(true),
     Discard(false),
     Sacrifice(false),
@@ -27,9 +27,9 @@ public enum Outcome {
     Protect(true),
     PutManaInPool(true),
     Regenerate(true),
-    PreventDamage(true),
-    PreventCast(false),
-    RedirectDamage(true),
+    PreventDamage(true), // TODO: check good or bad
+    PreventCast(false), // TODO: check good or bad
+    RedirectDamage(true), // TODO: check good or bad
     Tap(false),
     Transform(true),
     Untap(true),
@@ -40,9 +40,13 @@ public enum Outcome {
     Neutral(true),
     Removal(false),
     AIDontUseIt(false),
-    Vote(true);
+    Vote(true); // TODO: check good or bad
 
-    private final boolean good; // good or bad effect for targeting player (for AI usage)
+    // good or bad effect for TARGET, not targeting player (for AI usage)
+    // AI sorting targets by priorities (own or opponents) and selects most valueable or weakest
+    private final boolean good;
+
+    // no different between own or opponent targets (example: copy must choose from all permanents) // TODO: copy must choose most valueable from opponent too
     private boolean canTargetAll;
 
     Outcome(boolean good) {
@@ -63,7 +67,7 @@ public enum Outcome {
     }
 
     public static Outcome inverse(Outcome outcome) {
-        // inverse bad/good effect (as example, after controlling player change)
+        // inverse bad/good effect
         if (outcome.isGood()) {
             return Outcome.Detriment;
         } else {

Mage/src/main/java/mage/target/common/TargetOpponentsChoicePermanent.java

@@ -70,8 +70,8 @@ public class TargetOpponentsChoicePermanent extends TargetPermanent {
             return false;
         }
 
-        // opponent choose real targets (outcome must be inversed)
-        return super.chooseTarget(Outcome.inverse(outcome), opponentId, source, game);
+        // opponent choose real targets
+        return super.chooseTarget(outcome, opponentId, source, game);
     }
 
     @Override