From 473ecc329207ddb6a815d2ab27f3339d6bea02a3 Mon Sep 17 00:00:00 2001 From: Alex Vasile <48962821+Alex-Vasile@users.noreply.github.com> Date: Thu, 25 Aug 2022 14:50:30 -0400 Subject: [PATCH] [SNC] Fix Unlucky Witness not letting you play cards. Closes #9413. --- .../src/mage/cards/u/UnluckyWitness.java | 2 +- .../cards/single/snc/UnluckyWitnessTest.java | 71 +++++++++++++++++++ 2 files changed, 72 insertions(+), 1 deletion(-) create mode 100644 Mage.Tests/src/test/java/org/mage/test/cards/single/snc/UnluckyWitnessTest.java diff --git a/Mage.Sets/src/mage/cards/u/UnluckyWitness.java b/Mage.Sets/src/mage/cards/u/UnluckyWitness.java index 9b1baa9a83..5e984e60aa 100644 --- a/Mage.Sets/src/mage/cards/u/UnluckyWitness.java +++ b/Mage.Sets/src/mage/cards/u/UnluckyWitness.java @@ -34,7 +34,7 @@ public final class UnluckyWitness extends CardImpl { this.toughness = new MageInt(1); // When Unlucky Witness dies, exile the top two cards of your library. Until your next end step, you may play one of those cards. - this.addAbility(new DiesSourceTriggeredAbility(new UnluckyWitnessEffect())); + this.addAbility(new DiesSourceTriggeredAbility(new UnluckyWitnessEffect()), new UnluckyWitnessWatcher()); } private UnluckyWitness(final UnluckyWitness card) { diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/single/snc/UnluckyWitnessTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/single/snc/UnluckyWitnessTest.java new file mode 100644 index 0000000000..1cdaf5f3f1 --- /dev/null +++ b/Mage.Tests/src/test/java/org/mage/test/cards/single/snc/UnluckyWitnessTest.java @@ -0,0 +1,71 @@ +package org.mage.test.cards.single.snc; + +import mage.constants.PhaseStep; +import mage.constants.Zone; +import org.junit.Test; +import org.mage.test.serverside.base.CardTestPlayerBase; + +/** + * {@link mage.cards.u.UnluckyWitness Unlucky Witness} + * When Unlucky Witness dies, exile the top two cards of your library. + * Until your next end step, you may play one of those cards. + * + * @author Alex-Vasile + */ +public class UnluckyWitnessTest extends CardTestPlayerBase { + + private static final String unluckyWitness = "Unlucky Witness"; + + /** + * Reported bug: https://github.com/magefree/mage/issues/9413 + * You currently can't play either of the two exiled cards + */ + @Test + public void canPlayExiledCard() { + String murder = "Murder"; + addCard(Zone.HAND, playerA, murder); + addCard(Zone.BATTLEFIELD, playerA, unluckyWitness); + addCard(Zone.BATTLEFIELD, playerA, "Swamp", 3); + + addCard(Zone.LIBRARY, playerA, "Exotic Orchard", 2); + + skipInitShuffling(); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, murder); + + playLand(1, PhaseStep.POSTCOMBAT_MAIN, playerA, "Exotic Orchard"); + + execute(); + assertHandCount(playerA, 0); + assertExileCount(playerA, 1); + assertExileCount(playerA, "Exotic Orchard", 1); + assertGraveyardCount(playerA, unluckyWitness, 1); + } + + /** + * Same as the test above, but make sure that spell can also be cast + */ + @Test + public void canCastExiledCard() { + String murder = "Murder"; + addCard(Zone.HAND, playerA, murder); + addCard(Zone.BATTLEFIELD, playerA, unluckyWitness); + addCard(Zone.BATTLEFIELD, playerA, "Swamp", 4); + + addCard(Zone.LIBRARY, playerA, "Sol Ring", 2); + + skipInitShuffling(); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, murder); + + castSpell(1, PhaseStep.POSTCOMBAT_MAIN, playerA, "Sol Ring"); + + execute(); + + assertGraveyardCount(playerA, unluckyWitness, 1); + assertHandCount(playerA, 0); + assertExileCount(playerA, 1); + assertPermanentCount(playerA, "Sol Ring", 1); + assertExileCount(playerA, "Sol Ring", 1); + } +}