From 368faa37dd7718351c9e77bbbaefde54419b83d3 Mon Sep 17 00:00:00 2001
From: LevelX2 <ludwig.hirth@online.de>
Date: Mon, 29 Jun 2020 18:02:18 +0200
Subject: [PATCH] * Added test for Dream Leash.

---
 .../cards/targets/TargetRestrictionsTest.java | 89 +++++++++++++++++++
 .../mage/abilities/keyword/EmergeAbility.java |  2 +-
 2 files changed, 90 insertions(+), 1 deletion(-)
 create mode 100644 Mage.Tests/src/test/java/org/mage/test/cards/targets/TargetRestrictionsTest.java

diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/targets/TargetRestrictionsTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/targets/TargetRestrictionsTest.java
new file mode 100644
index 0000000000..e7bbd5b477
--- /dev/null
+++ b/Mage.Tests/src/test/java/org/mage/test/cards/targets/TargetRestrictionsTest.java
@@ -0,0 +1,89 @@
+/*
+ * To change this license header, choose License Headers in Project Properties.
+ * To change this template file, choose Tools | Templates
+ * and open the template in the editor.
+ */
+package org.mage.test.cards.targets;
+
+import mage.constants.PhaseStep;
+import mage.constants.Zone;
+import org.junit.Test;
+import org.mage.test.serverside.base.CardTestPlayerBase;
+
+/**
+ *
+ * @author LevelX2
+ */
+
+public class TargetRestrictionsTest extends CardTestPlayerBase {
+
+    @Test
+    public void testDreamLeashWorks() {
+        addCard(Zone.BATTLEFIELD, playerA, "Island", 6);
+        // Enchant permanent
+        // You can't choose an untapped permanent as Dream Leash's target as you cast Dream Leash.
+        // You control enchanted permanent.        
+        addCard(Zone.HAND, playerA, "Dream Leash"); // Enchantment {3}{U}{U}
+        
+        // Tap target creature. It doesn't untap during its controller's next untap step.
+        addCard(Zone.HAND, playerA, "Take into Custody"); // Instant {U}
+        
+        addCard(Zone.BATTLEFIELD, playerB, "Sejiri Merfolk");
+
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Take into Custody", "Sejiri Merfolk");
+        
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Dream Leash", "Sejiri Merfolk");
+
+        setStrictChooseMode(true);
+        setStopAt(3, PhaseStep.BEGIN_COMBAT);
+        execute();
+        
+        assertAllCommandsUsed();
+        
+        assertPermanentCount(playerA, "Dream Leash", 1);
+        assertPermanentCount(playerA, "Sejiri Merfolk", 1);
+        
+        assertLife(playerA, 20);
+        assertLife(playerB, 20);
+    }
+    
+    @Test
+    public void testDreamLeashUntappingAsResponseToCast() {
+        addCard(Zone.BATTLEFIELD, playerA, "Island", 6);
+        // Enchant permanent
+        // You can't choose an untapped permanent as Dream Leash's target as you cast Dream Leash.
+        // You control enchanted permanent.        
+        addCard(Zone.HAND, playerA, "Dream Leash"); // Enchantment {3}{U}{U}
+        
+        // Tap target creature. It doesn't untap during its controller's next untap step.
+        addCard(Zone.HAND, playerA, "Take into Custody"); // Instant {U}
+        
+        // Untap target creature. It gets +1/+3 until end of turn.
+        addCard(Zone.HAND, playerB, "Ornamental Courage"); // Instant {G}       
+
+        addCard(Zone.BATTLEFIELD, playerB, "Forest");
+        addCard(Zone.BATTLEFIELD, playerB, "Sejiri Merfolk");
+
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Take into Custody", "Sejiri Merfolk");
+        
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Dream Leash", "Sejiri Merfolk");
+        
+        castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerB, "Ornamental Courage", "Sejiri Merfolk", "Dream Leash");
+
+        setStrictChooseMode(true);
+        setStopAt(3, PhaseStep.BEGIN_COMBAT);
+        execute();
+        
+        assertAllCommandsUsed();
+        
+        
+        assertGraveyardCount(playerA, "Take into Custody", 1);
+        assertGraveyardCount(playerB, "Ornamental Courage", 1);
+        assertPermanentCount(playerA, "Dream Leash", 1);
+        assertPermanentCount(playerA, "Sejiri Merfolk", 1);
+        assertPowerToughness(playerA, "Sejiri Merfolk", 2, 1);
+        
+        assertLife(playerA, 20);
+        assertLife(playerB, 20);
+    }
+}
\ No newline at end of file
diff --git a/Mage/src/main/java/mage/abilities/keyword/EmergeAbility.java b/Mage/src/main/java/mage/abilities/keyword/EmergeAbility.java
index 8045bc39bb..0aa1eddbd6 100644
--- a/Mage/src/main/java/mage/abilities/keyword/EmergeAbility.java
+++ b/Mage/src/main/java/mage/abilities/keyword/EmergeAbility.java
@@ -29,7 +29,7 @@ public class EmergeAbility extends SpellAbility {
     public EmergeAbility(Card card, ManaCosts<ManaCost> emergeCost) {
         super(card.getSpellAbility());
         this.emergeCost = emergeCost.copy();
-        this.newId(); // Why is this neccessary? will create new id anyway or not?
+        this.newId(); // Set newId because cards spell ability is copied and needs own id
         this.setCardName(card.getName() + " with emerge");
         zone = Zone.HAND;
         spellAbilityType = SpellAbilityType.BASE_ALTERNATE;