* AI: fixed that computer can broke some non mana pays (Echo, Escalate, Recover, Tap source unless pay, Slow Motion, #8182);

This commit is contained in:
Oleg Agafonov 2021-09-03 02:11:21 +04:00
parent 4d0761fa27
commit 421cac0ce3
7 changed files with 112 additions and 6 deletions

View file

@ -1,4 +1,3 @@
package mage.cards.g; package mage.cards.g;
import java.util.UUID; import java.util.UUID;

View file

@ -69,7 +69,7 @@ class SacrificeEquipedUnlessPaysEffect extends OneShotEffect {
public SacrificeEquipedUnlessPaysEffect(final SacrificeEquipedUnlessPaysEffect effect) { public SacrificeEquipedUnlessPaysEffect(final SacrificeEquipedUnlessPaysEffect effect) {
super(effect); super(effect);
this.cost = effect.cost; this.cost = effect.cost.copy();
} }
@Override @Override

View file

@ -0,0 +1,107 @@
package org.mage.test.cards.single.soi;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
/**
* @author JayDi85
*/
public class GameTrailTest extends CardTestPlayerBase {
@Test
public void test_Reveal() {
// As Game Trail enters the battlefield, you may reveal a Mountain or Forest card from your hand. If you don't, Game Trail enters the battlefield tapped.
// {T}: Add {R} or {G}.
addCard(Zone.HAND, playerA, "Game Trail", 1); // land
addCard(Zone.HAND, playerA, "Forest", 1);
addCard(Zone.HAND, playerA, "Mountain", 1);
// play with reveal
playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Game Trail");
setChoice(playerA, true); // reveal
setChoice(playerA, "Forest"); // reveal forest
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertAllCommandsUsed();
assertTapped("Game Trail", false);
}
@Test
public void test_RevealWithoutCards() {
// As Game Trail enters the battlefield, you may reveal a Mountain or Forest card from your hand. If you don't, Game Trail enters the battlefield tapped.
// {T}: Add {R} or {G}.
addCard(Zone.HAND, playerA, "Game Trail", 1); // land
addCard(Zone.HAND, playerA, "Swamp", 1);
// play with reveal
playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Game Trail");
// no reveal choices
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertAllCommandsUsed();
assertTapped("Game Trail", true);
}
@Test
public void test_NotReveal() {
// As Game Trail enters the battlefield, you may reveal a Mountain or Forest card from your hand. If you don't, Game Trail enters the battlefield tapped.
// {T}: Add {R} or {G}.
addCard(Zone.HAND, playerA, "Game Trail", 1); // land
addCard(Zone.HAND, playerA, "Forest", 1);
addCard(Zone.HAND, playerA, "Mountain", 1);
// play with reveal
playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Game Trail");
setChoice(playerA, false); // no reveal
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertAllCommandsUsed();
assertTapped("Game Trail", true);
}
@Test
public void test_RevealAfterCultivate() {
removeAllCardsFromLibrary(playerA);
// As Game Trail enters the battlefield, you may reveal a Mountain or Forest card from your hand. If you don't, Game Trail enters the battlefield tapped.
// {T}: Add {R} or {G}.
addCard(Zone.HAND, playerA, "Game Trail", 1); // land
addCard(Zone.LIBRARY, playerA, "Forest", 1);
addCard(Zone.LIBRARY, playerA, "Island", 1);
addCard(Zone.LIBRARY, playerA, "Swamp", 1);
//
// Search your library for up to two basic land cards, reveal those cards, and put one onto the battlefield tapped
// and the other into your hand. Then shuffle your library.
addCard(Zone.HAND, playerA, "Cultivate", 1); // {2}{G}
addCard(Zone.BATTLEFIELD, playerA, "Forest", 3);
// put forest by cultivate
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Cultivate");
addTarget(playerA, "Forest^Swamp");
setChoice(playerA, "Swamp"); // put tapped
waitStackResolved(1, PhaseStep.PRECOMBAT_MAIN);
// play game trail
playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Game Trail");
setChoice(playerA, true); // reveal
setChoice(playerA, "Forest");
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertAllCommandsUsed();
assertTapped("Game Trail", false);
}
}

View file

@ -26,7 +26,7 @@ public class TapSourceUnlessPaysEffect extends OneShotEffect {
public TapSourceUnlessPaysEffect(final TapSourceUnlessPaysEffect effect) { public TapSourceUnlessPaysEffect(final TapSourceUnlessPaysEffect effect) {
super(effect); super(effect);
this.cost = effect.cost; this.cost = effect.cost.copy();
} }
@Override @Override

View file

@ -34,7 +34,7 @@ public class EchoEffect extends OneShotEffect {
public EchoEffect(final EchoEffect effect) { public EchoEffect(final EchoEffect effect) {
super(effect); super(effect);
this.cost = effect.cost; this.cost = effect.cost == null ? null : effect.cost.copy();
this.amount = effect.amount; this.amount = effect.amount;
} }

View file

@ -45,7 +45,7 @@ class EscalateEffect extends CostModificationEffectImpl {
EscalateEffect(final EscalateEffect effect) { EscalateEffect(final EscalateEffect effect) {
super(effect); super(effect);
this.cost = effect.cost; this.cost = effect.cost.copy();
} }
@Override @Override

View file

@ -76,7 +76,7 @@ class RecoverEffect extends OneShotEffect {
public RecoverEffect(final RecoverEffect effect) { public RecoverEffect(final RecoverEffect effect) {
super(effect); super(effect);
this.cost = effect.cost; this.cost = effect.cost.copy();
} }
@Override @Override