diff --git a/Mage.Sets/src/mage/cards/j/JacesMindseeker.java b/Mage.Sets/src/mage/cards/j/JacesMindseeker.java index c8ac2cee78..e29c9725b1 100644 --- a/Mage.Sets/src/mage/cards/j/JacesMindseeker.java +++ b/Mage.Sets/src/mage/cards/j/JacesMindseeker.java @@ -27,6 +27,7 @@ */ package mage.cards.j; +import java.util.HashSet; import java.util.Set; import java.util.UUID; import mage.MageInt; @@ -56,7 +57,7 @@ import mage.target.common.TargetOpponent; public class JacesMindseeker extends CardImpl { public JacesMindseeker(UUID ownerId, CardSetInfo setInfo) { - super(ownerId,setInfo,new CardType[]{CardType.CREATURE},"{4}{U}{U}"); + super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{4}{U}{U}"); this.subtype.add("Fish"); this.subtype.add("Illusion"); @@ -106,7 +107,9 @@ class JaceMindseekerEffect extends OneShotEffect { Player targetOpponent = game.getPlayer(targetPointer.getFirst(game, source)); if (targetOpponent != null) { Set allCards = targetOpponent.getLibrary().getTopCards(game, 5); - targetOpponent.moveCards(allCards, Zone.GRAVEYARD, source, game); + Set toMove = new HashSet<>(); + toMove.addAll(allCards); + targetOpponent.moveCards(toMove, Zone.GRAVEYARD, source, game); for (Card card : allCards) { if (filter.match(card, game)) { Zone zone = game.getState().getZone(card.getId()); @@ -117,25 +120,25 @@ class JaceMindseekerEffect extends OneShotEffect { } } } - } - // cast an instant or sorcery for free - if (cardsToCast.size() > 0) { - Player controller = game.getPlayer(source.getControllerId()); - if (controller != null) { - TargetCard target = new TargetCard(Zone.GRAVEYARD, filter); // zone should be ignored here - target.setNotTarget(true); - if (controller.chooseUse(outcome, "Cast an instant or sorcery card from among them for free?", source, game) - && controller.choose(outcome, cardsToCast, target, game)) { - Card card = cardsToCast.get(target.getFirstTarget(), game); - if (card != null) { - controller.cast(card.getSpellAbility(), game, true); + // cast an instant or sorcery for free + if (cardsToCast.size() > 0) { + Player controller = game.getPlayer(source.getControllerId()); + if (controller != null) { + TargetCard target = new TargetCard(Zone.GRAVEYARD, filter); // zone should be ignored here + target.setNotTarget(true); + if (controller.chooseUse(outcome, "Cast an instant or sorcery card from among them for free?", source, game) + && controller.choose(outcome, cardsToCast, target, game)) { + Card card = cardsToCast.get(target.getFirstTarget(), game); + if (card != null) { + controller.cast(card.getSpellAbility(), game, true); + } } } + } - + return true; } - return false; } } diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/single/JacesMindseekerTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/single/JacesMindseekerTest.java new file mode 100644 index 0000000000..582bd52d95 --- /dev/null +++ b/Mage.Tests/src/test/java/org/mage/test/cards/single/JacesMindseekerTest.java @@ -0,0 +1,79 @@ +/* + * Copyright 2010 BetaSteward_at_googlemail.com. All rights reserved. + * + * Redistribution and use in source and binary forms, with or without modification, are + * permitted provided that the following conditions are met: + * + * 1. Redistributions of source code must retain the above copyright notice, this list of + * conditions and the following disclaimer. + * + * 2. Redistributions in binary form must reproduce the above copyright notice, this list + * of conditions and the following disclaimer in the documentation and/or other materials + * provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY BetaSteward_at_googlemail.com ``AS IS'' AND ANY EXPRESS OR IMPLIED + * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND + * FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL BetaSteward_at_googlemail.com OR + * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR + * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON + * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF + * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + * + * The views and conclusions contained in the software and documentation are those of the + * authors and should not be interpreted as representing official policies, either expressed + * or implied, of BetaSteward_at_googlemail.com. + */ +package org.mage.test.cards.single; + +import mage.constants.PhaseStep; +import mage.constants.Zone; +import org.junit.Test; +import org.mage.test.serverside.base.CardTestPlayerBase; + +/** + * + * @author LevelX2 + */ +public class JacesMindseekerTest extends CardTestPlayerBase { + + /** + * Jace's Mindseeker trigger ability is not working properly. It doesn't + * allow me to cast an instant or sorcery if there is one among the 5 cards + * put into the graveyard. I think the problem is that when the cards are + * put into the graveyard, the cards can't be cast anymore. What if the + * cards are revealed first before they are put into the graveyard? That + * doesn't follow the sequence on the card, but it might solve the bug. + */ + @Test + public void testJacesMindseeker() { + addCard(Zone.BATTLEFIELD, playerA, "Island", 6); + // Flying + // When Jace's Mindseeker enters the battlefield, target opponent puts the top five cards of his or her library into his or her graveyard. + // You may cast an instant or sorcery card from among them without paying its mana cost. + addCard(Zone.HAND, playerA, "Jace's Mindseeker", 1); // Creature 4/4 {4}{U}{U} + + addCard(Zone.LIBRARY, playerB, "Silvercoat Lion", 2); + addCard(Zone.LIBRARY, playerB, "Lightning Bolt", 1); + addCard(Zone.LIBRARY, playerB, "Silvercoat Lion", 2); + skipInitShuffling(); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Jace's Mindseeker"); + addTarget(playerA, playerB); + setChoice(playerA, "Yes"); + setChoice(playerA, "Lightning Bolt"); + addTarget(playerA, playerB); + + setStopAt(1, PhaseStep.BEGIN_COMBAT); + execute(); + + assertPermanentCount(playerA, "Jace's Mindseeker", 1); + assertGraveyardCount(playerB, "Lightning Bolt", 1); + assertGraveyardCount(playerB, "Silvercoat Lion", 4); + + assertLife(playerA, 20); + assertLife(playerB, 17); + + } +}