Merge pull request #2944 from Alexsandr0x/sakashima-bug

SpellStutter Sprite - Sakashima Impostor Bug
This commit is contained in:
Derek M 2017-03-07 16:10:06 -05:00 committed by GitHub
commit 2ff68c2fb3
2 changed files with 30 additions and 1 deletions

View file

@ -126,7 +126,8 @@ class SpellstutterSpriteCounterTargetEffect extends OneShotEffect {
*/
int numberFaeries = game.getState().getBattlefield().countAll(SpellstutterSprite.filter, source.getControllerId(), game);
StackObject stackObject = game.getStack().getStackObject(source.getFirstTarget());
if (stackObject.getConvertedManaCost() <= numberFaeries) {
// If do'nt have any spell targeted
if (stackObject != null && stackObject.getConvertedManaCost() <= numberFaeries) {
if (game.getStack().counter(source.getFirstTarget(), source.getSourceId(), game)) {
return true;
}

View file

@ -0,0 +1,28 @@
package org.mage.test.cards.abilities.other;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
/**
* Created by alexsandro on 06/03/17.
*/
public class SakashimaTheImpostorTest extends CardTestPlayerBase {
@Test
public void copySpellStutterTest() {
addCard(Zone.BATTLEFIELD, playerA, "Spellstutter Sprite", 1);
addCard(Zone.BATTLEFIELD, playerB, "Island", 4);
addCard(Zone.HAND, playerB, "Sakashima the Impostor", 4);
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerB, "Sakashima the Impostor");
setChoice(playerB, "Spellstutter Sprite");
setStopAt(4, PhaseStep.END_TURN);
execute();
assertPermanentCount(playerA, "Spellstutter Sprite", 1);
assertPowerToughness(playerB, "Sakashima the Impostor", 1, 1);
}
}