Failure/foul-magics
1
0
Fork 0
forked from External/mage
Code Pull requests Releases Packages Activity Actions

fix LoseLifeControllerAttachedEffect (#11680)

Browse source 
add test for Pooling Venom
This commit is contained in:
xenohedron 2024-01-19 18:32:01 -05:00 • committed by GitHub
parent dafce47cd6
commit 8cef411c70
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 67 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

63
Mage.Tests/src/test/java/org/mage/test/cards/single/fut/PoolingVenomTest.java Normal file
View file

@ -0,0 +1,63 @@
package org.mage.test.cards.single.fut;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
/**
* @author xenohedron
*/
public class PoolingVenomTest extends CardTestPlayerBase {
private static final String venom = "Pooling Venom";
// Whenever enchanted land becomes tapped, its controller loses 2 life.
private static final String island = "Island";
private static final String swamp = "Swamp";
private static final String addU = "{T}: Add {U}";
private static final String addB = "{T}: Add {B}";
@Test
public void testOppsLand() {
addCard(Zone.BATTLEFIELD, playerB, island);
addCard(Zone.BATTLEFIELD, playerA, swamp, 2);
addCard(Zone.HAND, playerA, venom);
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, venom, island);
activateAbility(1, PhaseStep.POSTCOMBAT_MAIN, playerB, addU);
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertPermanentCount(playerA, venom, 1);
assertTapped(island, true);
assertLife(playerA, 20);
assertLife(playerB, 18);
}
@Test
public void testOwnLand() {
addCard(Zone.BATTLEFIELD, playerA, island);
addCard(Zone.BATTLEFIELD, playerA, swamp, 2);
addCard(Zone.HAND, playerA, venom);
activateManaAbility(1, PhaseStep.PRECOMBAT_MAIN, playerA, addB);
activateManaAbility(1, PhaseStep.PRECOMBAT_MAIN, playerA, addB);
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, venom, island);
activateManaAbility(1, PhaseStep.POSTCOMBAT_MAIN, playerA, addU);
setStrictChooseMode(true);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertPermanentCount(playerA, venom, 1);
assertTapped(island, true);
assertLife(playerA, 18);
assertLife(playerB, 20);
}
}

5
Mage/src/main/java/mage/abilities/effects/common/LoseLifeControllerAttachedEffect.java
View file

@ -44,7 +44,10 @@ public class LoseLifeControllerAttachedEffect extends OneShotEffect {
Permanent attachedTo = (Permanent) game.getLastKnownInformation(attachment.getAttachedTo(),
Zone.BATTLEFIELD, attachment.getAttachedToZoneChangeCounter());
if (attachedTo == null) {
return false;
attachedTo = game.getPermanent(attachment.getAttachedTo());
if (attachedTo == null) {
return false;
}
}
Player player = game.getPlayer(attachedTo.getControllerId());
if (player == null) {