From 59571d0a03b5dee9b1d589ed899d684eef08a1b0 Mon Sep 17 00:00:00 2001
From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com>
Date: Mon, 23 Aug 2021 20:51:59 +0000
Subject: [PATCH 1/3] Bump jsoup from 1.13.1 to 1.14.2 in /Mage.Client

Bumps [jsoup](https://github.com/jhy/jsoup) from 1.13.1 to 1.14.2.
- [Release notes](https://github.com/jhy/jsoup/releases)
- [Changelog](https://github.com/jhy/jsoup/blob/master/CHANGES)
- [Commits](https://github.com/jhy/jsoup/compare/jsoup-1.13.1...jsoup-1.14.2)

---
updated-dependencies:
- dependency-name: org.jsoup:jsoup
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
---
 Mage.Client/pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Mage.Client/pom.xml b/Mage.Client/pom.xml
index fc3c45e7172..79867c89069 100644
--- a/Mage.Client/pom.xml
+++ b/Mage.Client/pom.xml
@@ -110,7 +110,7 @@
         <dependency>
             <groupId>org.jsoup</groupId>
             <artifactId>jsoup</artifactId>
-            <version>1.13.1</version>
+            <version>1.14.2</version>
         </dependency>
         <dependency>
             <artifactId>truevfs-profile-base</artifactId>

From d481172eb4ce193e84156a88339dfdddb2865af9 Mon Sep 17 00:00:00 2001
From: Oleg Agafonov <jaydi85@gmail.com>
Date: Fri, 27 Aug 2021 20:51:27 +0400
Subject: [PATCH 2/3] Code: added test coverage reports by JaCoCo lib (generate
 by command like "mvn install -Djacoco.skip=false"); Code: added test coverage
 support for Sonar reports;

---
 .gitignore          |  1 +
 Mage.Tests/pom.xml  |  3 ---
 Mage.Verify/pom.xml |  3 ---
 pom.xml             | 52 +++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 53 insertions(+), 6 deletions(-)

diff --git a/.gitignore b/.gitignore
index 403ce01bda1..fa8fced8097 100644
--- a/.gitignore
+++ b/.gitignore
@@ -15,6 +15,7 @@ Mage.Client/target
 Mage.Common/target
 
 # Mage.Plugins
+Mage.Plugins/target
 Mage.Plugins/Mage.Card.Plugin/target
 Mage.Plugins/Mage.Counter.Plugin/target
 Mage.Plugins/Mage.Theme.Plugin/target
diff --git a/Mage.Tests/pom.xml b/Mage.Tests/pom.xml
index c0f30839981..55f495d7659 100644
--- a/Mage.Tests/pom.xml
+++ b/Mage.Tests/pom.xml
@@ -81,9 +81,6 @@
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-surefire-plugin</artifactId>
-                    <configuration>
-                        <argLine>-Dfile.encoding=UTF-8</argLine>
-                    </configuration>
                 </plugin>
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
diff --git a/Mage.Verify/pom.xml b/Mage.Verify/pom.xml
index fdab7c4bd97..91263f642d1 100644
--- a/Mage.Verify/pom.xml
+++ b/Mage.Verify/pom.xml
@@ -64,9 +64,6 @@
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-surefire-plugin</artifactId>
-                    <configuration>
-                        <argLine>-Dfile.encoding=UTF-8</argLine>
-                    </configuration>
                 </plugin>
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
diff --git a/pom.xml b/pom.xml
index 7e1335d59cd..91c1340a6ce 100644
--- a/pom.xml
+++ b/pom.xml
@@ -51,16 +51,52 @@
                     </archive>
                 </configuration>
             </plugin>
+
+            <!-- JaCoCo Code Coverage report generation -->
+            <!-- Examples: http://tdongsi.github.io/blog/2017/09/23/jacoco-in-maven-project/ -->
+            <plugin>
+                <groupId>org.jacoco</groupId>
+                <artifactId>jacoco-maven-plugin</artifactId>
+                <version>0.8.7</version>
+                <executions>
+                    <execution>
+                        <!-- prepare command line to inject in java agent (collect code executing stats) -->
+                        <goals>
+                            <goal>prepare-agent</goal>
+                        </goals>
+                    </execution>
+                    <execution>
+                        <!-- generate current module's report -->
+                        <id>generate-code-coverage-report</id>
+                        <phase>verify</phase>
+                        <goals>
+                            <goal>report</goal>
+                        </goals>
+                    </execution>
+                    <execution>
+                        <!-- generate combined report (current and dependency modules) -->
+                        <id>report-aggregate</id>
+                        <phase>verify</phase>
+                        <goals>
+                            <goal>report-aggregate</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
         </plugins>
+
         <pluginManagement>
             <plugins>
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-surefire-plugin</artifactId>
                     <version>2.22.2</version>
+                    <!--
+                    for compatible with jacoco code coverage - argLine moved to properties section
                     <configuration>
                         <argLine>-Dfile.encoding=UTF-8</argLine>
                     </configuration>
+                    -->
                 </plugin>
             </plugins>
         </pluginManagement>
@@ -99,8 +135,24 @@
 
     <properties>
         <mage-version>1.4.49</mage-version>
+        <argLine>-Dfile.encoding=UTF-8</argLine>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <maven.build.timestamp.format>yyyy-MM-dd'T'HH:mm:ss'Z'</maven.build.timestamp.format>
+
+        <!--
+            JaCoCo code coverage disabled by default. If you need to generate
+            execute stats and reports then run tests by maven command
+            like "mvn install -Djacoco.skip=false".
+            Stats:
+            - load coverage data for IntelliJ IDEA from ./target/jacoco.exec
+            - web report in ./target/site/jacoco/index.html
+            - for sonar support see below
+        -->
+        <jacoco.skip>true</jacoco.skip>
+
+        <!-- Sonar settings for code coverage. Must be only one report for all modules (use report-aggregate goal report from JaCoCo) -->
+        <aggregate.report.dir>Mage.Verify/target/site/jacoco-aggregate/jacoco.xml</aggregate.report.dir>
+        <sonar.coverage.jacoco.xmlReportPaths>${project.basedir}/../${aggregate.report.dir}</sonar.coverage.jacoco.xmlReportPaths>
     </properties>
 
     <dependencyManagement>

From 91d577b0afad07b76bb8081d3be2e5d18894bc2c Mon Sep 17 00:00:00 2001
From: Oleg Agafonov <jaydi85@gmail.com>
Date: Fri, 27 Aug 2021 21:42:25 +0400
Subject: [PATCH 3/3] Test framework: fixed that ai play step command doesn't
 work correctly on declare attackers and blockers steps;

---
 .../cards/single/stx/DaemogothTitanTest.java  | 47 +++++++++++++++++++
 .../java/org/mage/test/player/TestPlayer.java |  8 ++--
 2 files changed, 51 insertions(+), 4 deletions(-)
 create mode 100644 Mage.Tests/src/test/java/org/mage/test/cards/single/stx/DaemogothTitanTest.java

diff --git a/Mage.Tests/src/test/java/org/mage/test/cards/single/stx/DaemogothTitanTest.java b/Mage.Tests/src/test/java/org/mage/test/cards/single/stx/DaemogothTitanTest.java
new file mode 100644
index 00000000000..3c1c9209d0d
--- /dev/null
+++ b/Mage.Tests/src/test/java/org/mage/test/cards/single/stx/DaemogothTitanTest.java
@@ -0,0 +1,47 @@
+package org.mage.test.cards.single.stx;
+
+import mage.constants.PhaseStep;
+import mage.constants.Zone;
+import org.junit.Test;
+import org.mage.test.serverside.base.CardTestPlayerBaseWithAIHelps;
+
+/**
+ * @author JayDi85
+ */
+public class DaemogothTitanTest extends CardTestPlayerBaseWithAIHelps {
+
+    @Test
+    public void test_Attack_Manual() {
+        // Whenever Daemogoth Titan attacks or blocks, sacrifice a creature.
+        addCard(Zone.BATTLEFIELD, playerA, "Daemogoth Titan"); // 11/10
+        addCard(Zone.BATTLEFIELD, playerA, "Grizzly Bears", 1);
+
+        attack(1, playerA, "Daemogoth Titan");
+        addTarget(playerA, "Grizzly Bears");
+
+        setStrictChooseMode(true);
+        setStopAt(1, PhaseStep.END_TURN);
+        execute();
+        assertAllCommandsUsed();
+
+        assertLife(playerB, 20 - 11);
+        assertGraveyardCount(playerA, "Grizzly Bears", 1);
+    }
+
+    @Test
+    public void test_Attack_AI() {
+        // Whenever Daemogoth Titan attacks or blocks, sacrifice a creature.
+        addCard(Zone.BATTLEFIELD, playerA, "Daemogoth Titan"); // 11/10
+        addCard(Zone.BATTLEFIELD, playerA, "Grizzly Bears", 1);
+
+        aiPlayStep(1, PhaseStep.DECLARE_ATTACKERS, playerA);
+
+        setStrictChooseMode(true);
+        setStopAt(1, PhaseStep.END_TURN);
+        execute();
+        assertAllCommandsUsed();
+
+        assertLife(playerB, 20 - 11);
+        assertGraveyardCount(playerA, "Grizzly Bears", 1);
+    }
+}
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 a4e77c9a2ab..cd6def1ac98 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
@@ -1683,8 +1683,8 @@ public class TestPlayer implements Player {
                 mustAttackByAction = true;
                 madeAttackByAction = true;
                 this.computerPlayer.selectAttackers(game, attackingPlayerId);
-                it.remove();
-                break;
+                // play step action will be removed on step end
+                continue;
             }
 
             if (action.getTurnNum() == game.getTurnNum() && action.getAction().startsWith("attack:")) {
@@ -1774,8 +1774,8 @@ public class TestPlayer implements Player {
             if (action.getTurnNum() == game.getTurnNum() && action.getAction().equals(AI_PREFIX + AI_COMMAND_PLAY_STEP)) {
                 mustBlockByAction = true;
                 this.computerPlayer.selectBlockers(source, game, defendingPlayerId);
-                actions.remove(action);
-                break;
+                // play step action will be removed on step end
+                continue;
             }
 
             if (action.getTurnNum() == game.getTurnNum() && action.getAction().startsWith("block:")) {