From b2fcb8ea8cbfb322a5c2e39617919dd62b3c9f9f Mon Sep 17 00:00:00 2001 From: LevelX2 Date: Sun, 12 Jul 2015 23:16:02 +0200 Subject: [PATCH] Added some tests. --- .../cards/abilities/keywords/RetraceTest.java | 76 ++++++++++++------- .../continuous/ChiefOfTheFoundryTest.java | 59 ++++++++++++++ 2 files changed, 109 insertions(+), 26 deletions(-) create mode 100644 Mage.Tests/src/test/java/org/mage/test/cards/continuous/ChiefOfTheFoundryTest.java diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords/RetraceTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords/RetraceTest.java index 8a056cbecfd..190d8e44776 100644 --- a/Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords/RetraceTest.java +++ b/Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords/RetraceTest.java @@ -36,19 +36,16 @@ import org.mage.test.serverside.base.CardTestPlayerBase; * * @author LevelX2 */ - public class RetraceTest extends CardTestPlayerBase { /** - * 702.78. Retrace - * 702.78a Retrace appears on some instants and sorceries. It represents a static ability - * that functions while the card is in a player's graveyard. "Retrace" means "You may cast - * this card from your graveyard by discarding a land card as an additional cost to cast it." - * Casting a spell using its retrace ability follows the rules for paying additional costs - * in rules 601.2b and 601.2e-g. + * 702.78. Retrace 702.78a Retrace appears on some instants and sorceries. + * It represents a static ability that functions while the card is in a + * player's graveyard. "Retrace" means "You may cast this card from your + * graveyard by discarding a land card as an additional cost to cast it." + * Casting a spell using its retrace ability follows the rules for paying + * additional costs in rules 601.2b and 601.2e-g. */ - - @Test public void SimpleRetrace() { addCard(Zone.BATTLEFIELD, playerA, "Swamp", 1); @@ -60,16 +57,16 @@ public class RetraceTest extends CardTestPlayerBase { addCard(Zone.HAND, playerB, "Silvercoat Lion", 1); castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Raven's Crime", playerB); - + setStopAt(1, PhaseStep.BEGIN_COMBAT); execute(); - assertGraveyardCount(playerA,"Raven's Crime", 1); - assertGraveyardCount(playerA,"Swamp", 1); - - assertGraveyardCount(playerB,"Silvercoat Lion", 1); + assertGraveyardCount(playerA, "Raven's Crime", 1); + assertGraveyardCount(playerA, "Swamp", 1); + + assertGraveyardCount(playerB, "Silvercoat Lion", 1); } - + /** * Test that it does cost {B}{1} + land discard */ @@ -86,14 +83,14 @@ public class RetraceTest extends CardTestPlayerBase { addCard(Zone.HAND, playerB, "Silvercoat Lion", 1); castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Raven's Crime", playerB); - + setStopAt(1, PhaseStep.BEGIN_COMBAT); execute(); - assertGraveyardCount(playerA,"Raven's Crime", 1); - assertGraveyardCount(playerA,"Swamp", 0); // because not enough mana - - assertGraveyardCount(playerB,"Silvercoat Lion", 0); // because not enough mana + assertGraveyardCount(playerA, "Raven's Crime", 1); + assertGraveyardCount(playerA, "Swamp", 0); // because not enough mana + + assertGraveyardCount(playerB, "Silvercoat Lion", 0); // because not enough mana } /** @@ -112,14 +109,41 @@ public class RetraceTest extends CardTestPlayerBase { addCard(Zone.HAND, playerB, "Silvercoat Lion", 1); castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Raven's Crime", playerB); - + setStopAt(1, PhaseStep.BEGIN_COMBAT); execute(); - assertGraveyardCount(playerA,"Raven's Crime", 1); - assertGraveyardCount(playerA,"Swamp", 1); - - assertGraveyardCount(playerB,"Silvercoat Lion", 1); + assertGraveyardCount(playerA, "Raven's Crime", 1); + assertGraveyardCount(playerA, "Swamp", 1); + + assertGraveyardCount(playerB, "Silvercoat Lion", 1); + } + + /** + * I noticed the other day that I was not able to cast Worm Harvest from the + * graveyard. I'm not sure if this is an issue with all cards with the + * "retrace" ability but I figured it should be mentioned! + */ + @Test + public void RetraceCastFromGraveyard() { + addCard(Zone.BATTLEFIELD, playerA, "Swamp", 5); + + // Target player discards a card. + // Retrace + addCard(Zone.GRAVEYARD, playerA, "Worm Harvest"); + addCard(Zone.GRAVEYARD, playerA, "Swamp", 2); + addCard(Zone.HAND, playerA, "Mountain", 1); + + // Put a 1/1 black and green Worm creature token onto the battlefield for each land card in your graveyard. + // Retrace (You may cast this card from your graveyard by discarding a land card in addition to paying its other costs.) + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Worm Harvest"); + + setStopAt(1, PhaseStep.BEGIN_COMBAT); + execute(); + + assertPermanentCount(playerA, "Worm", 3); + + assertGraveyardCount(playerA, "Mountain", 1); + } - } diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/continuous/ChiefOfTheFoundryTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/continuous/ChiefOfTheFoundryTest.java new file mode 100644 index 00000000000..823961576c0 --- /dev/null +++ b/Mage.Tests/src/test/java/org/mage/test/cards/continuous/ChiefOfTheFoundryTest.java @@ -0,0 +1,59 @@ +/* + * 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.continuous; + +import mage.constants.PhaseStep; +import mage.constants.Zone; +import mage.filter.Filter; +import org.junit.Test; +import org.mage.test.serverside.base.CardTestPlayerBase; + +/** + * + * @author LevelX2 + */ +public class ChiefOfTheFoundryTest extends CardTestPlayerBase { + + /** + * Having two Chief of Foundry's out doesn't make them buff each other. + */ + @Test + public void testBoostOtherFoundry() { + addCard(Zone.BATTLEFIELD, playerA, "Swamp", 6); + addCard(Zone.HAND, playerA, "Chief of the Foundry"); + addCard(Zone.HAND, playerA, "Chief of the Foundry"); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Chief of the Foundry"); + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Chief of the Foundry"); + setStopAt(1, PhaseStep.BEGIN_COMBAT); + execute(); + + assertPowerToughness(playerA, "Chief of the Foundry", 3, 4, Filter.ComparisonScope.All); + } + +}