From 258ec26fece03ab226f11f4fb75693d22fde5b32 Mon Sep 17 00:00:00 2001
From: Alex Vasile <48962821+Alex-Vasile@users.noreply.github.com>
Date: Fri, 26 Aug 2022 20:59:49 -0400
Subject: [PATCH] Added tests to try to capture sacrifice lands automatically
 going to the graveyard. For #8980.

---
 .../cost/sacrifice/SacrificeLandTest.java     | 96 +++++++++++++++++++
 1 file changed, 96 insertions(+)
 create mode 100644 Mage.Tests/src/test/java/org/mage/test/cards/cost/sacrifice/SacrificeLandTest.java

diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/cost/sacrifice/SacrificeLandTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/cost/sacrifice/SacrificeLandTest.java
new file mode 100644
index 0000000000..27244a31ea
--- /dev/null
+++ b/Mage.Tests/src/test/java/org/mage/test/cards/cost/sacrifice/SacrificeLandTest.java
@@ -0,0 +1,96 @@
+package org.mage.test.cards.cost.sacrifice;
+
+import mage.constants.PhaseStep;
+import mage.constants.Zone;
+import org.junit.Test;
+import org.mage.test.sba.PlaneswalkerRuleTest;
+import org.mage.test.serverside.base.CardTestPlayerBase;
+
+import java.util.Random;
+
+/**
+ * Reported bugs: https://github.com/magefree/mage/issues/8980
+ *      Cards like Soldevi Excavations, Heart of Yavimaya, and Lake of the Dead
+ *      will sometimes immediately go to the graveyard without asking the controller if they wish to sacrifice a land
+ *      even though they had one available
+ * @author Alex-Vasile
+ */
+public class SacrificeLandTest extends CardTestPlayerBase {
+
+    /**
+     * Perhaps it is a rollback issue with saving state
+     */
+    @Test
+    public void testRollback() {
+        // If Soldevi Excavations would enter the battlefield, sacrifice an untapped Island instead.
+        // If you do, put Soldevi Excavations onto the battlefield. If you don't, put it into its owner's graveyard.
+        String soldeviExcavations = "Soldevi Excavations";
+
+        addCard(Zone.HAND, playerA, soldeviExcavations);
+        addCard(Zone.BATTLEFIELD, playerA, "Island");
+
+        Random random = new Random();
+        boolean sacFirstLand = random.nextBoolean();
+        boolean sacSecondLand = random.nextBoolean();
+
+        playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, soldeviExcavations);
+        setChoice(playerA, sacFirstLand);
+
+        rollbackTurns(1, PhaseStep.PRECOMBAT_MAIN, playerA, 0);
+
+        rollbackAfterActionsStart();
+        playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, soldeviExcavations);
+        setChoice(playerA, sacSecondLand);
+        rollbackAfterActionsEnd();
+
+        setStopAt(1, PhaseStep.PRECOMBAT_MAIN);
+        execute();
+
+        assertPermanentCount(playerA, soldeviExcavations, sacSecondLand ? 1 : 0);
+        assertGraveyardCount(playerA, "Island", sacSecondLand ? 1 : 0);
+    }
+
+    /**
+     * Perhaps it will come up with playing multiple on seperate turns
+     */
+    @Test
+    public void testMultiple() {
+        // If Soldevi Excavations would enter the battlefield, sacrifice an untapped Island instead.
+        // If you do, put Soldevi Excavations onto the battlefield. If you don't, put it into its owner's graveyard.
+        String soldeviExcavations = "Soldevi Excavations";
+        // Sacrifice a Forest
+        String heartofYavimaya = "Heart of Yavimaya";
+        // Sacrifice a Swamp
+        String lakeOfTheDead = "Lake of the Dead";
+
+        addCard(Zone.HAND, playerA, soldeviExcavations);
+        addCard(Zone.BATTLEFIELD, playerA, "Island");
+
+        addCard(Zone.HAND, playerA, heartofYavimaya);
+        addCard(Zone.BATTLEFIELD, playerA, "Forest");
+
+        addCard(Zone.HAND, playerA, lakeOfTheDead);
+        addCard(Zone.BATTLEFIELD, playerA, "Swamp");
+
+        playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, soldeviExcavations);
+        setChoice(playerA, "Yes");
+
+        playLand(3, PhaseStep.PRECOMBAT_MAIN, playerA, heartofYavimaya);
+        setChoice(playerA, "Yes");
+
+        playLand(5, PhaseStep.PRECOMBAT_MAIN, playerA, lakeOfTheDead);
+        setChoice(playerA, "Yes");
+
+        setStopAt(5, PhaseStep.PRECOMBAT_MAIN);
+        execute();
+
+        assertPermanentCount(playerA, soldeviExcavations, 1);
+        assertGraveyardCount(playerA, "Island", 1);
+
+        assertPermanentCount(playerA, heartofYavimaya, 1);
+        assertGraveyardCount(playerA, "Forest", 1);
+
+        assertPermanentCount(playerA, lakeOfTheDead, 1);
+        assertGraveyardCount(playerA, "Swamp", 1);
+    }
+}