From 5c02780cc42cd3d78e6660e02b5e42aafc91609c Mon Sep 17 00:00:00 2001 From: Susucre <34709007+Susucre@users.noreply.github.com> Date: Fri, 31 May 2024 11:13:41 +0200 Subject: [PATCH] Tests: fix choice notTarget wrongly not checking Filter --- .../java/org/mage/test/cards/single/mh3/BirthingRitualTest.java | 2 -- Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/single/mh3/BirthingRitualTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/single/mh3/BirthingRitualTest.java index 00e9662209e..5c0db02c69c 100644 --- a/Mage.Tests/src/test/java/org/mage/test/cards/single/mh3/BirthingRitualTest.java +++ b/Mage.Tests/src/test/java/org/mage/test/cards/single/mh3/BirthingRitualTest.java @@ -3,7 +3,6 @@ package org.mage.test.cards.single.mh3; import mage.constants.PhaseStep; import mage.constants.Zone; import org.junit.Assert; -import org.junit.Ignore; import org.junit.Test; import org.mage.test.player.TestPlayer; import org.mage.test.serverside.base.CardTestPlayerBase; @@ -108,7 +107,6 @@ public class BirthingRitualTest extends CardTestPlayerBase { assertHandCount(playerA, "Centaur Courser", 1); } - @Ignore // TODO: something weird with unit test, will be fixed separately. @Test public void test_Trigger_Sacrifice_MVRestriction() { setStrictChooseMode(true); diff --git a/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java b/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java index 6c6543cfe1e..f700adf8aae 100644 --- a/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java +++ b/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java @@ -4143,7 +4143,7 @@ public class TestPlayer implements Player { continue; } 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); targetFound = true; break;