External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-20 02:30:08 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

Tests: fix choice notTarget wrongly not checking Filter

Browse source
This commit is contained in:
Susucre 2024-05-31 11:13:41 +02:00
parent 239f75fc8d
commit 5c02780cc4
2 changed files with 1 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Mage.Tests/src/test/java/org/mage/test/cards/single/mh3/BirthingRitualTest.java
View file

@ -3,7 +3,6 @@ package org.mage.test.cards.single.mh3;
import mage.constants.PhaseStep; import mage.constants.PhaseStep;
import mage.constants.Zone; import mage.constants.Zone;
import org.junit.Assert; import org.junit.Assert;
import org.junit.Ignore;
import org.junit.Test; import org.junit.Test;
import org.mage.test.player.TestPlayer; import org.mage.test.player.TestPlayer;
import org.mage.test.serverside.base.CardTestPlayerBase; import org.mage.test.serverside.base.CardTestPlayerBase;
@ -108,7 +107,6 @@ public class BirthingRitualTest extends CardTestPlayerBase {
assertHandCount(playerA, "Centaur Courser", 1); assertHandCount(playerA, "Centaur Courser", 1);
} }
@Ignore // TODO: something weird with unit test, will be fixed separately.
@Test @Test
public void test_Trigger_Sacrifice_MVRestriction() { public void test_Trigger_Sacrifice_MVRestriction() {
setStrictChooseMode(true); setStrictChooseMode(true);

2
Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java
View file

@ -4143,7 +4143,7 @@ public class TestPlayer implements Player {
continue; continue;
} }
if (hasObjectTargetNameOrAlias(card, targetName)) { if (hasObjectTargetNameOrAlias(card, targetName)) {
if (target.isNotTarget() || target.canTarget(card.getId(), source, game)) { if (target.canTarget(getId(), card.getId(), source, game)) {
target.add(card.getId(), game); target.add(card.getId(), game);
targetFound = true; targetFound = true;
break; break;