Open the way - fix maximum X costs

Mage.Sets/src/mage/cards/o/OpenTheWay.java

@@ -76,7 +76,8 @@ public OpenTheWayEffect copy() {
     public boolean apply(Game game, Ability source) {
         Player player = game.getPlayer(source.getControllerId());
         int xValue = CardUtil.getSourceCostsTag(game, source, "X", 0);
-        if (player == null || xValue < 1) {
+        int playerCount = game.getPlayers().size();
+        if (player == null || xValue < 1 || xValue > playerCount) {
             return false;
         }
         Cards toReveal = new CardsImpl();

Mage.Tests/src/test/java/org/mage/test/cards/cost/variable/OpenTheWayTest.java

@@ -0,0 +1,47 @@
+package org.mage.test.cards.cost.variable;
+
+import mage.constants.PhaseStep;
+import mage.constants.Zone;
+import org.junit.Test;
+import org.mage.test.serverside.base.CardTestPlayerBase;
+
+public class OpenTheWayTest extends CardTestPlayerBase {
+
+    @Test
+    public void testCardWithValidValue(){
+        addCard(Zone.BATTLEFIELD, playerA, "Forest", 10);
+        addCard(Zone.HAND, playerA, "Open the Way");
+        playerA.getLibrary().clear();
+        addCard(Zone.LIBRARY, playerA, "Forest", 10);
+
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Open the Way");
+        setChoice(playerA, "X=2");
+
+        setStrictChooseMode(true);
+
+        setStopAt(1, PhaseStep.POSTCOMBAT_MAIN);
+        execute();
+
+        assertPermanentCount(playerA, "Forest", 12);
+
+    }
+
+    @Test
+    public void testCardWithInvalidVAlue(){
+        addCard(Zone.BATTLEFIELD, playerA, "Forest", 10);
+        addCard(Zone.HAND, playerA, "Open the Way");
+        playerA.getLibrary().clear();
+        addCard(Zone.LIBRARY, playerA, "Forest", 10);
+
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Open the Way");
+        setChoice(playerA, "X=5");
+
+        setStrictChooseMode(true);
+
+        setStopAt(1, PhaseStep.POSTCOMBAT_MAIN);
+        execute();
+
+        assertPermanentCount(playerA, "Forest", 10);
+
+    }
+}