From e3f27cbc89d8290431b78d6304d27e980eb19b5e Mon Sep 17 00:00:00 2001
From: magenoxx <magenoxx@gmail>
Date: Sat, 7 Apr 2012 11:07:35 +0400
Subject: [PATCH] Fix for pass action in test framework. Updated assert
 information for assertPT.

---
 .../test/java/org/mage/test/player/TestPlayer.java |  2 +-
 .../base/impl/CardTestPlayerAPIImpl.java           | 14 ++++++++++----
 2 files changed, 11 insertions(+), 5 deletions(-)

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 44f111781b1..4f83f181caa 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
@@ -116,7 +116,7 @@ public class TestPlayer extends ComputerPlayer<TestPlayer> {
             }
         }
         pass();
-        return true;
+        return false;
     }
 
     @Override
diff --git a/Mage.Tests/src/test/java/org/mage/test/serverside/base/impl/CardTestPlayerAPIImpl.java b/Mage.Tests/src/test/java/org/mage/test/serverside/base/impl/CardTestPlayerAPIImpl.java
index b157bd396f2..b7b8b6b7f75 100644
--- a/Mage.Tests/src/test/java/org/mage/test/serverside/base/impl/CardTestPlayerAPIImpl.java
+++ b/Mage.Tests/src/test/java/org/mage/test/serverside/base/impl/CardTestPlayerAPIImpl.java
@@ -284,8 +284,11 @@ public abstract class CardTestPlayerAPIImpl extends MageTestPlayerBase implement
 			throws AssertionError {
 		int count = 0;
 		int fit = 0;
-		for (Permanent permanent : currentGame.getBattlefield().getAllActivePermanents(player.getId())) {
-			if (permanent.getName().equals(cardName)) {
+        int foundPower = 0;
+        int foundToughness = 0;
+        int found = 0;
+		for (Permanent permanent : currentGame.getBattlefield().getAllPermanents()) {
+			if (permanent.getName().equals(cardName) && permanent.getControllerId().equals(player.getId())) {
 				count++;
 				if (scope.equals(Filter.ComparisonScope.All)) {
 					Assert.assertEquals("Power is not the same (" + power + " vs. " + permanent.getPower().getValue() + ")",
@@ -295,8 +298,11 @@ public abstract class CardTestPlayerAPIImpl extends MageTestPlayerBase implement
 				} else if (scope.equals(Filter.ComparisonScope.Any)) {
 					if (power == permanent.getPower().getValue() && toughness == permanent.getToughness().getValue()) {
 						fit++;
-						break;
+                    	break;
 					}
+                    found++;
+                    foundPower = permanent.getPower().getValue();
+                    foundToughness = permanent.getToughness().getValue();
 				}
 			}
 		}
@@ -306,7 +312,7 @@ public abstract class CardTestPlayerAPIImpl extends MageTestPlayerBase implement
 
 		if (scope.equals(Filter.ComparisonScope.Any)) {
 			Assert.assertTrue("There is no such creature under player's control with specified power&toughness, player=" + player.getName() +
-					", cardName=" + cardName, fit > 0);
+					", cardName=" + cardName + " (found similar: " + found + ", one of them: power=" + foundPower + " toughness=" + foundToughness + ")", fit > 0);
 		}
 	}