From a66e64f595f56931fd803256659d0cafebacc968 Mon Sep 17 00:00:00 2001 From: drmDev Date: Thu, 24 Mar 2016 06:17:47 -0400 Subject: [PATCH 1/2] Mulldrifter tests for reported bug --- .../test/cards/single/MulldrifterTest.java | 90 +++++++++++++++++++ 1 file changed, 90 insertions(+) create mode 100644 Mage.Tests/src/test/java/org/mage/test/cards/single/MulldrifterTest.java diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/single/MulldrifterTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/single/MulldrifterTest.java new file mode 100644 index 00000000000..7a398f9ccaa --- /dev/null +++ b/Mage.Tests/src/test/java/org/mage/test/cards/single/MulldrifterTest.java @@ -0,0 +1,90 @@ +/* + * To change this license header, choose License Headers in Project Properties. + * To change this template file, choose Tools | Templates + * and open the template in the editor. + */ +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 escplan9 (Derek Monturo - dmontur1 at gmail dot com) + */ +public class MulldrifterTest extends CardTestPlayerBase { + + /** + * Reported bug: Muldrifter only draws 1 card. And only once during a turn if you haven't already drawn a card. + * Example, If it is my turn and I play Muldrifter, no card is drawn for trigger. + * + * If it is not my turn and I play Ghostly Flicker targeting Eternal Witness and Muldrifter, when Muldrifter enters play, only 1 card is drawn. + * + * If I repeat the same thing in the same turn, the second time Muldrifter enters the battlefield, no cards are drawn. + */ + @Test + public void testMulldrifterNotEvoked() { + + // {4}{U} When Mulldrifter enters the battlefield, draw two cards. + addCard(Zone.HAND, playerA, "Mulldrifter"); // 2/2 + addCard(Zone.BATTLEFIELD, playerA, "Plains", 2); + addCard(Zone.BATTLEFIELD, playerA, "Island", 3); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Mulldrifter"); + setChoice(playerA, "No"); // cast regularly, not evoked + setStopAt(1, PhaseStep.BEGIN_COMBAT); + execute(); + + assertPermanentCount(playerA, "Mulldrifter", 1); + assertHandCount(playerA, 2); // should have drawn 2 cards + } + + /** + * + */ + @Test + public void testMulldrifterEvoked() { + + // {4}{U} When Mulldrifter enters the battlefield, draw two cards. Evoke {2}{U} + addCard(Zone.HAND, playerA, "Mulldrifter"); // 2/2 + addCard(Zone.BATTLEFIELD, playerA, "Island", 3); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Mulldrifter"); + setChoice(playerA, "Yes"); // only paid evoke cost + setStopAt(1, PhaseStep.BEGIN_COMBAT); + execute(); + + assertPermanentCount(playerA, "Mulldrifter", 0); + assertGraveyardCount(playerA, "Mulldrifter", 1); + assertHandCount(playerA, 2); // should have drawn 2 cards + } + + /** + * + */ + @Test + public void testMulldrifterFlickered() { + + // {4}{U} When Mulldrifter enters the battlefield, draw two cards. Evoke {2}{U} + addCard(Zone.BATTLEFIELD, playerA, "Mulldrifter"); // 2/2 + addCard(Zone.BATTLEFIELD, playerA, "Merfolk Looter"); // 1/1 + addCard(Zone.BATTLEFIELD, playerA, "Island", 5); + + // Ghostly Flicker {2}{U} Instant + // Exile two target artifacts, creatures, and/or lands you control, then return those cards to the battlefield under your control. + addCard(Zone.HAND, playerA, "Ghostly Flicker"); + + castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Ghostly Flicker"); + addTarget(playerA, "Mulldrifter^Merfolk Looter"); + setChoice(playerA, "No"); // do not pay evoke cost + setStopAt(1, PhaseStep.BEGIN_COMBAT); + execute(); + + assertPermanentCount(playerA, "Mulldrifter", 1); + assertPermanentCount(playerA, "Merfolk Looter", 1); + assertGraveyardCount(playerA, "Ghostly Flicker", 1); + assertHandCount(playerA, 2); // should have drawn 2 cards + } +} \ No newline at end of file From b0228a66efdc33ece770fb347eaa279ce227b83d Mon Sep 17 00:00:00 2001 From: drmDev Date: Thu, 24 Mar 2016 07:19:27 -0400 Subject: [PATCH 2/2] Abyssal Nocturnus card implementation --- .../mage/sets/guildpact/AbyssalNocturnus.java | 74 +++++++++++++++++++ 1 file changed, 74 insertions(+) create mode 100644 Mage.Sets/src/mage/sets/guildpact/AbyssalNocturnus.java diff --git a/Mage.Sets/src/mage/sets/guildpact/AbyssalNocturnus.java b/Mage.Sets/src/mage/sets/guildpact/AbyssalNocturnus.java new file mode 100644 index 00000000000..1d3bb8a1673 --- /dev/null +++ b/Mage.Sets/src/mage/sets/guildpact/AbyssalNocturnus.java @@ -0,0 +1,74 @@ +/* + * 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 mage.sets.guildpact; + +import java.util.UUID; +import mage.MageInt; +import mage.abilities.Ability; +import mage.abilities.common.DiscardsACardOpponentTriggeredAbility; +import mage.abilities.effects.Effect; +import mage.abilities.effects.common.continuous.BoostSourceEffect; +import mage.abilities.effects.common.continuous.GainAbilitySourceEffect; +import mage.abilities.keyword.FearAbility; +import mage.cards.CardImpl; +import mage.constants.CardType; +import mage.constants.Duration; +import mage.constants.Rarity; + +/** + * + * @author escplan9 (Derek Monturo - dmontur1 at gmail dot com) + */ +public class AbyssalNocturnus extends CardImpl { + + public AbyssalNocturnus(UUID ownerId) { + super(ownerId, 43, "Abyssal Nocturnus", Rarity.RARE, new CardType[]{CardType.CREATURE}, "{1}{B}{B}"); + this.expansionSetCode = "GPT"; + this.subtype.add("Horror"); + this.power = new MageInt(2); + this.toughness = new MageInt(2); + + // Whenever an opponent discards a card, Abyssal Nocturnus gets +2/+2 and gains fear until end of turn. + Effect boostEffect = new BoostSourceEffect(2, 2, Duration.EndOfTurn); + boostEffect.setText("{this} gets +2/+2"); + Ability ability = new DiscardsACardOpponentTriggeredAbility(boostEffect, false); // not optional + Effect fearEffect = new GainAbilitySourceEffect(FearAbility.getInstance(), Duration.EndOfTurn); + fearEffect.setText("and gains fear until end of turn"); + ability.addEffect(fearEffect); + this.addAbility(ability); + } + + public AbyssalNocturnus(final AbyssalNocturnus card) { + super(card); + } + + @Override + public AbyssalNocturnus copy() { + return new AbyssalNocturnus(this); + } +}