github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2025-04-11 09:11:12 -09:00
Code Issues Projects Releases Packages Wiki Activity

* Fixed a bug that P/T set of target animated lands (e.g. by Awaken) were set in the wrong sublayer so that the P/T of self animated lands (e.g. Shambling Vent) were always overwritten desite their ability time stamp.

Browse source
This commit is contained in:
LevelX2 2015-12-16 12:36:09 +01:00
parent 70833c6938
commit aa07fcecd5
2 changed files with 64 additions and 2 deletions
Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords
AwakenTest.java
Mage/src/main/java/mage/abilities/effects/common/continuous
BecomesCreatureTargetEffect.java

62
Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords/AwakenTest.java
View file

@ -58,4 +58,66 @@ public class AwakenTest extends CardTestPlayerBase {
assertGraveyardCount(playerB, "Silvercoat Lion", 1);
}
/**
* Used Awaken on Scatter to the Winds on an unanimated Shambling Vent.
* Animated vent and it was still a 3/3 with lifelink when it should've been
* 5/6.
*/
@Test
public void testShamblingVentAndAnimation() {
addCard(Zone.BATTLEFIELD, playerA, "Island", 5);
addCard(Zone.BATTLEFIELD, playerA, "Chromatic Lantern", 1);
// Shambling Vent enters the battlefield tapped.
// {T}: Add {W} or {B} to your mana pool.
// {1}{W}{B}: Shambling Vent becomes a 2/3 white and black Elemental creature with lifelink until end of turn. It's still a land.
addCard(Zone.BATTLEFIELD, playerA, "Shambling Vent", 1);
// Counter target spell.
// Awaken 3-{4}{U}{U}
addCard(Zone.HAND, playerA, "Scatter to the Winds", 1);
addCard(Zone.BATTLEFIELD, playerB, "Plains", 2);
addCard(Zone.HAND, playerB, "Silvercoat Lion", 1);
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerB, "Silvercoat Lion");
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerA, "Scatter to the Winds with awaken", "Silvercoat Lion");
addTarget(playerA, "Shambling Vent");
activateAbility(3, PhaseStep.PRECOMBAT_MAIN, playerA, "{1}{W}{B}");
setStopAt(3, PhaseStep.BEGIN_COMBAT);
execute();
assertGraveyardCount(playerA, "Scatter to the Winds", 1);
assertPowerToughness(playerA, "Shambling Vent", 5, 6);
assertGraveyardCount(playerB, "Silvercoat Lion", 1);
}
@Test
public void testShamblingVent() {
addCard(Zone.BATTLEFIELD, playerA, "Island", 5);
addCard(Zone.BATTLEFIELD, playerA, "Chromatic Lantern", 1);
// Shambling Vent enters the battlefield tapped.
// {T}: Add {W} or {B} to your mana pool.
// {1}{W}{B}: Shambling Vent becomes a 2/3 white and black Elemental creature with lifelink until end of turn. It's still a land.
addCard(Zone.BATTLEFIELD, playerA, "Shambling Vent", 1);
// Counter target spell.
// Awaken 3-{4}{U}{U}
addCard(Zone.HAND, playerA, "Scatter to the Winds", 1);
addCard(Zone.BATTLEFIELD, playerB, "Plains", 2);
addCard(Zone.HAND, playerB, "Silvercoat Lion", 1);
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerB, "Silvercoat Lion");
castSpell(2, PhaseStep.PRECOMBAT_MAIN, playerA, "Scatter to the Winds with awaken", "Silvercoat Lion");
addTarget(playerA, "Shambling Vent");
setStopAt(2, PhaseStep.BEGIN_COMBAT);
execute();
assertGraveyardCount(playerA, "Scatter to the Winds", 1);
assertPowerToughness(playerA, "Shambling Vent", 3, 3);
assertGraveyardCount(playerB, "Silvercoat Lion", 1);
}
}

4
Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesCreatureTargetEffect.java
View file

@ -130,13 +130,13 @@ public class BecomesCreatureTargetEffect extends ContinuousEffectImpl {
if (sublayer == SubLayer.NA) {
if (token.getAbilities().size() > 0) {
for (Ability ability : token.getAbilities()) {
permanent.addAbility(ability, game);
permanent.addAbility(ability, source.getSourceId(), game);
}
}
}
break;
case PTChangingEffects_7:
if (sublayer == SubLayer.SetPT_7b) {
if (sublayer == SubLayer.CharacteristicDefining_7a) {
permanent.getToughness().setValue(token.getToughness().getValue());
permanent.getPower().setValue(token.getPower().getValue());
}