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

Fix [NEO] Kairi, the Swirling Sky buggy targetting

Browse source
This commit is contained in:
Susucre 2024-04-05 13:23:51 +02:00
parent 896423ad40
commit 8a6b39c912
2 changed files with 99 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

18
Mage.Sets/src/mage/cards/k/KairiTheSwirlingSky.java
View file

@ -95,13 +95,17 @@ class KairiTheSwirlingSkyTarget extends TargetPermanent {
if (permanent == null) {
return false;
}
return permanent.getManaValue()
+ this.getTargets()
.stream()
.map(game::getPermanent)
.filter(Objects::nonNull)
.mapToInt(MageObject::getManaValue)
.sum() <= 6;
int added = 0; // We need to prevent the target to be counted twice on revalidation.
if (!this.getTargets().contains(id)) {
added = permanent.getManaValue();// fresh target, adding its MV
}
return added +
this.getTargets()
.stream()
.map(game::getPermanent)
.filter(Objects::nonNull)
.mapToInt(MageObject::getManaValue)
.sum() <= 6;
}
}

88
Mage.Tests/src/test/java/org/mage/test/cards/single/neo/KairiTheSwirlingSkyTest.java Normal file
View file

@ -0,0 +1,88 @@
package org.mage.test.cards.single.neo;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Assert;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
/**
* @author Susucr
*/
public class KairiTheSwirlingSkyTest extends CardTestPlayerBase {
/**
* {@link mage.cards.k.KairiTheSwirlingSky Kairi, the Swirling Sky} {4}{U}{U}
* Legendary Creature — Dragon Spirit
* Flying, ward {3}
* When Kairi, the Swirling Sky dies, choose one —
* • Return any number of target nonland permanents with total mana value 6 or less to their owners’ hands.
* • Mill six cards, then return up to two instant and/or sorcery cards from your graveyard to your hand.
* 6/6
*/
private static final String kairi = "Kairi, the Swirling Sky";
@Test
public void test_Return_MV6() {
setStrictChooseMode(true);
addCard(Zone.BATTLEFIELD, playerA, kairi);
addCard(Zone.BATTLEFIELD, playerA, "Colossal Dreadmaw");
addCard(Zone.HAND, playerA, "Doom Blade");
addCard(Zone.BATTLEFIELD, playerA, "Swamp", 2);
setModeChoice(playerA, "1");
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Doom Blade", kairi);
addTarget(playerA, "Colossal Dreadmaw"); // returning dreadmaw
setStopAt(1, PhaseStep.END_TURN);
execute();
assertHandCount(playerA, "Colossal Dreadmaw", 1);
}
@Test
public void test_Return_MV4_MV0() {
setStrictChooseMode(true);
addCard(Zone.BATTLEFIELD, playerA, kairi);
addCard(Zone.BATTLEFIELD, playerA, "Memnite");
addCard(Zone.BATTLEFIELD, playerA, "Abbey Griffin");
addCard(Zone.HAND, playerA, "Doom Blade");
addCard(Zone.BATTLEFIELD, playerA, "Swamp", 2);
setModeChoice(playerA, "1");
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Doom Blade", kairi);
addTarget(playerA, "Abbey Griffin^Memnite"); // returning both Griffin & Memnite
setStopAt(1, PhaseStep.END_TURN);
execute();
assertHandCount(playerA, "Abbey Griffin", 1);
assertHandCount(playerA, "Memnite", 1);
}
@Test
public void test_Cant_Return_MV7() {
setStrictChooseMode(true);
addCard(Zone.BATTLEFIELD, playerA, kairi);
addCard(Zone.BATTLEFIELD, playerA, "Axebane Stag");
addCard(Zone.HAND, playerA, "Doom Blade");
addCard(Zone.BATTLEFIELD, playerA, "Swamp", 2);
setModeChoice(playerA, "1"); // choose mode #1
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Doom Blade", kairi);
addTarget(playerA, "Axebane Stag"); // returning Axebane Stag is an invalid target
setStopAt(1, PhaseStep.END_TURN);
String foundError = "";
try {
execute();
} catch (AssertionError e) {
foundError = e.getMessage();
}
Assert.assertEquals("PlayerA - Targets list was setup by addTarget with [Axebane Stag], but not used", foundError.split("\n")[0]);
}
}