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

* Soulstinger - Fixed a bug that a -1/-1 counter was put on target even if Soulstinger had no counters on it.

Browse source
This commit is contained in:
LevelX2 2017-04-17 20:11:01 +02:00
parent cc6feacb2b
commit fb6bd25d85
2 changed files with 67 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
Mage.Sets/src/mage/cards/s/Soulstinger.java
View file

@ -33,8 +33,6 @@ import mage.abilities.Ability;
import mage.abilities.common.DiesTriggeredAbility;
import mage.abilities.common.EntersBattlefieldTriggeredAbility;
import mage.abilities.dynamicvalue.common.CountersSourceCount;
import mage.abilities.dynamicvalue.common.StaticValue;
import mage.abilities.effects.common.DrawCardSourceControllerEffect;
import mage.abilities.effects.common.counter.AddCountersTargetEffect;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
@ -52,21 +50,21 @@ public class Soulstinger extends CardImpl {
public Soulstinger(UUID ownerId, CardSetInfo setInfo) {
super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{3}{B}");
this.subtype.add("Scorpion");
this.subtype.add("Demon");
this.power = new MageInt(4);
this.toughness = new MageInt(5);
// When Soulstinger enters the battlefield, put two -1/-1 counter on target creature you control.
Ability ability = new EntersBattlefieldTriggeredAbility( new AddCountersTargetEffect(CounterType.M1M1.createInstance(2)));
Ability ability = new EntersBattlefieldTriggeredAbility(new AddCountersTargetEffect(CounterType.M1M1.createInstance(2)));
ability.addTarget(new TargetControlledCreaturePermanent());
this.addAbility(ability);
// When Soulstinger dies, you may put a -1/-1 counter on target creature for each -1/-1 counter on Soulstinger.
AddCountersTargetEffect effect =
new AddCountersTargetEffect(
CounterType.M1M1.createInstance(),
AddCountersTargetEffect effect
= new AddCountersTargetEffect(
CounterType.M1M1.createInstance(0),
new CountersSourceCount(CounterType.M1M1),
Outcome.Detriment);
effect.setText("you may put a -1/-1 counter on target creature for each -1/-1 counter on {this}");

62
Mage.Tests/src/test/java/org/mage/test/cards/abilities/oneshot/counter/AddingCountersToPermanentsTest.java
View file

@ -91,4 +91,66 @@ public class AddingCountersToPermanentsTest extends CardTestPlayerBase {
}
@Test
public void testSoulstingerNormal() {
addCard(Zone.BATTLEFIELD, playerA, "Swamp", 4);
// When Soulstinger enters the battlefield, put two -1/-1 counter on target creature you control.
// When Soulstinger dies, you may put a -1/-1 counter on target creature for each -1/-1 counter on Soulstinger.
addCard(Zone.HAND, playerA, "Soulstinger", 1); // Creature 4/5 {3}{B}
addCard(Zone.BATTLEFIELD, playerB, "Mountain", 5);
// Turn to Slag deals 5 damage to target creature. Destroy all Equipment attached to that creature.
addCard(Zone.HAND, playerB, "Turn to Slag", 1); // Sorcery {3}{R}{R}
addCard(Zone.BATTLEFIELD, playerB, "Pillarfield Ox", 1);
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Soulstinger");
addTarget(playerA, "Soulstinger");
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerB, "Turn to Slag", "Soulstinger");
setStopAt(2, PhaseStep.END_TURN);
execute();
assertGraveyardCount(playerA, "Soulstinger", 1);
assertGraveyardCount(playerB, "Turn to Slag", 1);
assertPowerToughness(playerB, "Pillarfield Ox", 0, 2);
}
/**
* Soulstinger died and gave a -1/-1 counter to an opponent's creature.
* Soulstinger had no -1/-1 counters on it, but the opponent's creature did,
* so maybe checking quantity of counters on the wrong creature?
*/
@Test
public void testSoulstinger() {
addCard(Zone.BATTLEFIELD, playerA, "Swamp", 4);
// When Soulstinger enters the battlefield, put two -1/-1 counter on target creature you control.
// When Soulstinger dies, you may put a -1/-1 counter on target creature for each -1/-1 counter on Soulstinger.
addCard(Zone.HAND, playerA, "Soulstinger", 1); // Creature 4/5 {3}{B}
addCard(Zone.BATTLEFIELD, playerA, "Silvercoat Lion", 1);
addCard(Zone.BATTLEFIELD, playerB, "Mountain", 5);
// Turn to Slag deals 5 damage to target creature. Destroy all Equipment attached to that creature.
addCard(Zone.HAND, playerB, "Turn to Slag", 1); // Sorcery {3}{R}{R}
addCard(Zone.BATTLEFIELD, playerB, "Silvercoat Lion", 1);
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Soulstinger");
addTarget(playerA, "Silvercoat Lion");
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerB, "Turn to Slag", "Soulstinger");
setStopAt(2, PhaseStep.END_TURN);
execute();
assertGraveyardCount(playerA, "Silvercoat Lion", 1);
assertGraveyardCount(playerA, "Soulstinger", 1);
assertGraveyardCount(playerB, "Turn to Slag", 1);
assertPowerToughness(playerB, "Silvercoat Lion", 2, 2);
}
}