From abcc34432e4939061a3c93643f987c3172ef60df Mon Sep 17 00:00:00 2001 From: drmDev Date: Wed, 26 Apr 2017 20:20:26 -0400 Subject: [PATCH] added comments noting tests currently failing and their associated issues --- .../cards/continuous/LandTypeChangingEffectsTest.java | 6 ++++++ .../mage/test/cards/cost/modification/FluctuatorTest.java | 8 ++++---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/continuous/LandTypeChangingEffectsTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/continuous/LandTypeChangingEffectsTest.java index e297a91159..091484c112 100644 --- a/Mage.Tests/src/test/java/org/mage/test/cards/continuous/LandTypeChangingEffectsTest.java +++ b/Mage.Tests/src/test/java/org/mage/test/cards/continuous/LandTypeChangingEffectsTest.java @@ -135,6 +135,9 @@ public class LandTypeChangingEffectsTest extends CardTestPlayerBase { String bloodmoon = "Blood Moon"; String canopyvista = "Canopy Vista"; + /* + NOTE: this test is currently failing due to bug in code. See issue #3072 + */ @Test public void testBloodMoonBeforeUrborg() { // Blood Moon 2R @@ -164,6 +167,9 @@ public class LandTypeChangingEffectsTest extends CardTestPlayerBase { Assert.assertTrue("The mana the land can produce should be [{R}] but it's " + playerB.getManaAvailable(currentGame).toString(), playerB.getManaAvailable(currentGame).toString().equals("[{R}]")); } + /* + NOTE: this test is currently failing due to bug in code. See issue #3072 + */ @Test public void testBloodMoonAfterUrborg() { // Blood Moon 2R diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/cost/modification/FluctuatorTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/cost/modification/FluctuatorTest.java index 98f4100f9a..eec83a8069 100644 --- a/Mage.Tests/src/test/java/org/mage/test/cards/cost/modification/FluctuatorTest.java +++ b/Mage.Tests/src/test/java/org/mage/test/cards/cost/modification/FluctuatorTest.java @@ -38,8 +38,8 @@ import org.mage.test.serverside.base.CardTestPlayerBase; public class FluctuatorTest extends CardTestPlayerBase { /** - * NOTE: As of 4/19/2017 this test is failing due to a bug in code. See - * issue #3148 + * NOTE: As of 4/19/2017 this test is failing due to a bug in code. + * See issue #3148 * * Fluctuator makes 'Akroma's Vengeance' cyclic cost reduced to {1} Test it * with one Plains on battlefield. @@ -93,8 +93,8 @@ public class FluctuatorTest extends CardTestPlayerBase { } /** - * NOTE: As of 4/19/2017 this test is failing due to a bug in code. See - * issue #3148 + * NOTE: As of 4/19/2017 this test is failing due to a bug in code. + * See issue #3148 * * Test 2 Fluctuators reduce cycling cost up to 4. */