diff --git a/Mage.Sets/src/mage/sets/conflux/Banefire.java b/Mage.Sets/src/mage/sets/conflux/Banefire.java index 8e504b49679..ef147d081d2 100644 --- a/Mage.Sets/src/mage/sets/conflux/Banefire.java +++ b/Mage.Sets/src/mage/sets/conflux/Banefire.java @@ -57,13 +57,11 @@ public class Banefire extends CardImpl { super(ownerId, 58, "Banefire", Rarity.RARE, new CardType[]{CardType.SORCERY}, "{X}{R}"); this.expansionSetCode = "CON"; - // Banefire deals X damage to target creature or player. this.getSpellAbility().addEffect(new BaneFireEffect()); this.getSpellAbility().addTarget(new TargetCreatureOrPlayer()); // If X is 5 or more, Banefire can't be countered by spells or abilities and the damage can't be prevented. this.addAbility(new SimpleStaticAbility(Zone.STACK, new BanefireCantCounterEffect())); - } public Banefire(final Banefire card) { @@ -118,7 +116,7 @@ class BaneFireEffect extends OneShotEffect { Player targetPlayer = game.getPlayer(source.getFirstTarget()); Permanent targetCreature = game.getPermanent(source.getFirstTarget()); int damage = source.getManaCostsToPay().getX(); - boolean preventable = damage >= 5; + boolean preventable = damage < 5; if (targetPlayer != null) { targetPlayer.damage(damage, source.getSourceId(), game, false, preventable); return true; diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/prevention/SwansOfBrynArgollTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/prevention/SwansOfBrynArgollTest.java index 4ef1adf1a67..f66f70aeb56 100644 --- a/Mage.Tests/src/test/java/org/mage/test/cards/prevention/SwansOfBrynArgollTest.java +++ b/Mage.Tests/src/test/java/org/mage/test/cards/prevention/SwansOfBrynArgollTest.java @@ -64,4 +64,30 @@ public class SwansOfBrynArgollTest extends CardTestPlayerBase{ assertHandCount(playerA, 0); } + /** + * Since you can't prevent damage that Combust deals, it should be able to kill Swans of Bryn Argoll. + */ + @Test + public void testAgainstBanefire() { + // 4/3 Flying + // If a source would deal damage to Swans of Bryn Argoll, prevent that damage. The source's controller draws cards equal to the damage prevented this way. + addCard(Zone.BATTLEFIELD, playerA, "Swans of Bryn Argoll"); + + // Banefire deals X damage to target creature or player. + // If X is 5 or more, Banefire can't be countered by spells or abilities and the damage can't be prevented. + addCard(Zone.HAND, playerB, "Banefire", 1); + addCard(Zone.BATTLEFIELD, playerB, "Mountain", 8); + + castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerB, "Banefire", "Swans of Bryn Argoll"); + setChoice(playerB, "X=7"); + + setStopAt(2, PhaseStep.BEGIN_COMBAT); + execute(); + + assertGraveyardCount(playerB, "Banefire", 1); + assertPermanentCount(playerA, "Swans of Bryn Argoll", 0); + assertGraveyardCount(playerA, "Swans of Bryn Argoll", 1); + assertHandCount(playerA, 0); + + } } \ No newline at end of file