github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-12-24 11:50:45 +00:00
Code Issues Projects Releases Packages Wiki Activity

Play card without mana cost - fixed that AI and other players can play cards without additional cost in some use cases (#6760);

Browse source
This commit is contained in:
Oleg Agafonov 2020-07-06 21:06:49 +04:00
parent b679087284
commit 802aca942a
2 changed files with 38 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
Mage.Tests/src/test/java/org/mage/test/cards/asthough/PlayFromNonHandZoneTest.java
View file

@ -3,12 +3,12 @@ package org.mage.test.cards.asthough;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
import org.mage.test.serverside.base.CardTestPlayerBaseWithAIHelps;
/**
* @author LevelX2
*/
public class PlayFromNonHandZoneTest extends CardTestPlayerBase {
public class PlayFromNonHandZoneTest extends CardTestPlayerBaseWithAIHelps {
@Test
public void testWorldheartPhoenixNormal() {
@ -148,6 +148,39 @@ public class PlayFromNonHandZoneTest extends CardTestPlayerBase {
assertGraveyardCount(playerA, "Cathartic Reunion", 1);
}
@Test
public void testNarsetEnlightenedMasterAdditionalCost_AI() {
skipInitShuffling();
removeAllCardsFromLibrary(playerA);
// First strike
// Hexproof
// Whenever Narset, Enlightented Master attacks, exile the top four cards of your library. Until end of turn, you may cast noncreature cards exiled with Narset this turn without paying their mana costs.
addCard(Zone.BATTLEFIELD, playerA, "Narset, Enlightened Master", 1);
//
// {1}{R}
// As an additional cost to cast this spell, discard two cards.
addCard(Zone.LIBRARY, playerA, "Swamp", 3); // 3 cards for draw effect
addCard(Zone.LIBRARY, playerA, "Cathartic Reunion", 1); // exile from lib
addCard(Zone.LIBRARY, playerA, "Plains", 3); // exile from lib
addCard(Zone.HAND, playerA, "Swamp", 1);
addCard(Zone.HAND, playerA, "Forest", 1);
attack(1, playerA, "Narset, Enlightened Master");
// AI simulation to cast exiled card (possible bug: ai real cast uses cost payed status from one of the simulation and don't pay)
aiPlayPriority(1, PhaseStep.POSTCOMBAT_MAIN, playerA);
setStrictChooseMode(true);
setStopAt(2, PhaseStep.END_TURN);
execute();
assertAllCommandsUsed();
assertGraveyardCount(playerA, "Swamp", 1);
assertGraveyardCount(playerA, "Forest", 1);
assertGraveyardCount(playerA, "Cathartic Reunion", 1);
}
/**
* Kess, Dissident Mage and Fire/Ice - When you cast fire/ice from your
* graverayr with kess it doesn't exile it self

5
Mage/src/main/java/mage/players/PlayerImpl.java
View file

@ -1075,9 +1075,10 @@ public abstract class PlayerImpl implements Player, Serializable {
@Override
public void setCastSourceIdWithAlternateMana(UUID sourceId, ManaCosts<ManaCost> manaCosts, Costs<Cost> costs) {
// cost must be copied for data consistence between game simulations
castSourceIdWithAlternateMana.add(sourceId);
castSourceIdManaCosts.put(sourceId, manaCosts);
castSourceIdCosts.put(sourceId, costs);
castSourceIdManaCosts.put(sourceId, manaCosts != null ? manaCosts.copy() : null);
castSourceIdCosts.put(sourceId, costs != null ? costs.copy() : null);
}
@Override