Failure/foul-magics
1
0
Fork 0
forked from External/mage
Code Pull requests Releases Packages Activity Actions

fix siege defeated trigger to be optional

Browse source
This commit is contained in:
xenohedron 2023-10-11 19:56:59 -04:00
parent 347ea5b9fe
commit fa49ffd27f
3 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Mage.Tests/src/test/java/org/mage/test/cards/battle/BattleDuelTest.java
View file

@ -121,6 +121,7 @@ public class BattleDuelTest extends BattleBaseTest {
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, belenon);
castSpell(1, PhaseStep.POSTCOMBAT_MAIN, playerA, impact, belenon);
setChoice(playerA, true); // yes to cast it transformed
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
@ -165,6 +166,7 @@ public class BattleDuelTest extends BattleBaseTest {
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Invasion of Dominaria");
castSpell(1, PhaseStep.POSTCOMBAT_MAIN, playerA, impact, "Invasion of Dominaria");
setChoice(playerA, true); // yes to cast it transformed
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);

2
Mage.Tests/src/test/java/org/mage/test/cards/single/mom/InvasionOfAlaraTest.java
View file

@ -61,7 +61,7 @@ public class InvasionOfAlaraTest extends CardTestPlayerBase {
checkHandCardCount("Divination in hand", 1, PhaseStep.BEGIN_COMBAT, playerA, divination, 1);
activateAbility(1, PhaseStep.POSTCOMBAT_MAIN, playerA, "Sacrifice", invasion); // remove all counters
//setChoice(playerA, true); // yes to cast
setChoice(playerA, true); // yes to cast
addTarget(playerA, playerA); // to draw 2 cards (meteor and puma)
addTarget(playerA, golems); // to destroy (last effect)
// now resolve

2
Mage/src/main/java/mage/abilities/common/SiegeAbility.java
View file

@ -44,7 +44,7 @@ public class SiegeAbility extends StaticAbility {
class SiegeDefeatedTriggeredAbility extends TriggeredAbilityImpl {
SiegeDefeatedTriggeredAbility() {
super(Zone.BATTLEFIELD, new SiegeDefeatedEffect());
super(Zone.BATTLEFIELD, new SiegeDefeatedEffect(), true);
this.setRuleVisible(false);
}