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

AI: added support of TargetCard targeting in hand, graveyard, exile zones. Added additional checks for tests.

Browse source
This commit is contained in:
Oleg Agafonov 2021-09-03 00:26:39 +04:00
parent 98b758e95e
commit 4d0761fa27
1 changed files with 21 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -2314,6 +2314,8 @@ public class TestPlayer implements Player {
return true;
}
Set<Zone> targetCardZonesChecked = new HashSet<>(); // control miss implementation
// player
if (target.getOriginalTarget() instanceof TargetPlayer
|| target.getOriginalTarget() instanceof TargetAnyTarget
@ -2397,9 +2399,12 @@ public class TestPlayer implements Player {
}
}
// card in hand
// card in hand (only own hand supports here)
// TODO: add not own hand too, example
if (target.getOriginalTarget() instanceof TargetCardInHand
|| target.getOriginalTarget() instanceof TargetDiscard) {
|| target.getOriginalTarget() instanceof TargetDiscard
|| (target.getOriginalTarget() instanceof TargetCard && target.getOriginalTarget().getZone() == Zone.HAND)) {
targetCardZonesChecked.add(Zone.HAND);
for (String targetDefinition : targets) {
checkTargetDefinitionMarksSupport(target, targetDefinition, "^");
String[] targetList = targetDefinition.split("\\^");
@ -2424,8 +2429,9 @@ public class TestPlayer implements Player {
// card in exile
if (target.getOriginalTarget() instanceof TargetCardInExile
|| target.getOriginalTarget() instanceof TargetPermanentOrSuspendedCard) {
|| target.getOriginalTarget() instanceof TargetPermanentOrSuspendedCard
|| (target.getOriginalTarget() instanceof TargetCard && target.getOriginalTarget().getZone() == Zone.EXILED)) {
targetCardZonesChecked.add(Zone.EXILED);
FilterCard filter = null;
if (target.getOriginalTarget().getFilter() instanceof FilterCard) {
filter = (FilterCard) target.getOriginalTarget().getFilter();
@ -2488,7 +2494,9 @@ public class TestPlayer implements Player {
if (target.getOriginalTarget() instanceof TargetCardInOpponentsGraveyard
|| target.getOriginalTarget() instanceof TargetCardInYourGraveyard
|| target.getOriginalTarget() instanceof TargetCardInGraveyard
|| target.getOriginalTarget() instanceof TargetCardInGraveyardOrBattlefield) {
|| target.getOriginalTarget() instanceof TargetCardInGraveyardOrBattlefield
|| (target.getOriginalTarget() instanceof TargetCard && target.getOriginalTarget().getZone() == Zone.GRAVEYARD)) {
targetCardZonesChecked.add(Zone.GRAVEYARD);
TargetCard targetFull = (TargetCard) target.getOriginalTarget();
List<UUID> needPlayers = game.getState().getPlayersInRange(getId(), game).toList();
@ -2560,6 +2568,14 @@ public class TestPlayer implements Player {
}
}
}
// uninplemented TargetCard's zone
if (target.getOriginalTarget() instanceof TargetCard && !targetCardZonesChecked.contains(target.getOriginalTarget().getZone())) {
Assert.fail("Found unimplemented TargetCard's zone or TargetCard's extented class: "
+ target.getOriginalTarget().getClass().getCanonicalName()
+ ", zone " + target.getOriginalTarget().getZone()
+ ", from " + (source == null ? "unknown source" : source.getSourceObject(game)));
}
}
// wrong target settings by addTarget