More pronoun fixes

Mage.Tests/src/test/java/org/mage/test/cards/abilities/flicker/CloudshiftTest.java

@@ -157,7 +157,7 @@ public class CloudshiftTest extends CardTestPlayerBase {
 
     /*
     I had a Stoneforge Mystic equipped with a Umesawa's Jitte. I activated Jitte 4 times to make
-    Stoneforge Mystic 9/10. My opponent put into play a Flickerwisp with his Aether Vial and
+    Stoneforge Mystic 9/10. My opponent put into play a Flickerwisp with their Aether Vial and
     targeted my Stoneforge Mystic. At the end of my turn, Stoneforge Mystic came back as a 9/10,
     before going down to 1/2 normally once my turn ended.
      */

Mage.Tests/src/test/java/org/mage/test/cards/abilities/keywords/SpliceOnArcaneTest.java

@@ -133,12 +133,12 @@ public class SpliceOnArcaneTest extends CardTestPlayerBase {
      * The scenario was that I cast a Nourishing Shoal with a Goryo's Vengeance
      * spliced to it targeting Griselbrand in my graveyard and exiling
      * Worldspine Wurm. My opponent responded with a Snapcaster Mage, so to
-     * deprive him of his ability to reuse his counterspell, I cast the Goryo's
-     * Vengeance on the Griselbrand. This one resolved. He then used Terminate
-     * on the Griselbrand after I had activated it once. When the Shoal tried to
-     * resolve, it should have been countered due to no legal target. However,
-     * it caused me to gain 11 life. It did not resurrect Griselbrand
-     * (correctly), but it should have done nothing at all.
+     * deprive them of their ability to reuse their counterspell, I cast the
+     * Goryo's Vengeance on the Griselbrand. This one resolved. They then used
+     * Terminate on the Griselbrand after I had activated it once. When the
+     * Shoal tried to resolve, it should have been countered due to no legal
+     * target. However, it caused me to gain 11 life. It did not resurrect
+     * Griselbrand (correctly), but it should have done nothing at all.
      * <p>
      * I include the info about the Terminate because thinking through, it could
      * be pertinent. I would guess what is going on here is one of two things.

Mage.Tests/src/test/java/org/mage/test/cards/abilities/oneshot/exile/RoonOfTheHiddenRealmTest.java

@@ -16,7 +16,7 @@ public class RoonOfTheHiddenRealmTest extends CardTestPlayerBase {
      * Roon of the Hidden Realm is returning cards to their controler's control
      * instead of the owner's control at the end of the turn. I used his ability
      * on a Perplexing Chimera I gave my opponent and in the end of the turn it
-     * returned to the battlefield in his control.
+     * returned to the battlefield in their control.
      */
     @Test
     public void testReturnToBattlefieldForOwner() {

Mage.Tests/src/test/java/org/mage/test/cards/conditional/TragicSlipTest.java

@@ -76,7 +76,7 @@ public class TragicSlipTest extends CardTestPlayerBase {
     }
 
     /*
-        Killed an opponent's Young Pyromancer with Ulcerate then flashed back Tragic Slip with Snapcaster Mage targeting his Tarmogoyf.
+        Killed an opponent's Young Pyromancer with Ulcerate then flashed back Tragic Slip with Snapcaster Mage targeting their Tarmogoyf.
         Morbid didn't seem to work and only applied -1/-1 to the Tarmogoyf.
     */
     @Test

Mage.Tests/src/test/java/org/mage/test/cards/copy/CryptoplasmTest.java

@@ -80,17 +80,17 @@ public class CryptoplasmTest extends CardTestPlayerBase {
      * that ETB copying the first Cryptoplasm and is currently also a copy of
      * the Divinity.
      *
-     * Opponent attacks with his only Divinity of Pride (4/4) and a Serra
+     * Opponent attacks with their only Divinity of Pride (4/4) and a Serra
      * Avenger (3/3). I block the Divinity with two of my Divinity copies (the
      * Clever Impersonator and unenchanted Cryptoplasm) and the Avenger with the
      * enchanted Divinity (originally a Cryptoplasm). My opponent's Divinity
-     * kills my two copies and dies, and then his Avenger dies and kills the
+     * kills my two copies and dies, and then their Avenger dies and kills the
      * Divinity blocking it, also sending my Followed Footsteps down with it.
      *
-     * How does any of that add up? Not only should his Divinity only kill one
+     * How does any of that add up? Not only should their Divinity only kill one
      * of mine since it was a 4/4 and only becomes an 8/8 after dealing its
      * damage (at which point it should be too late to go back and say the 4
-     * damage are now 8, since it was that exact damage that put him at 25
+     * damage are now 8, since it was that exact damage that put them at 25
      * life), but even more confusing is how the Serra Avenger, which is a 3/3,
      * somehow kills my 4/4 that had suffered no other damage that turn.
      *

Mage.Tests/src/test/java/org/mage/test/cards/copy/FlameshadowConjuringTest.java

@@ -14,7 +14,7 @@ public class FlameshadowConjuringTest extends CardTestPlayerBase {
 
     /**
      * My opponent ran into an issue with Priest of the Blood Rite being copied
-     * with Flameshadow Conjuring. His copy was made and removed correctly at
+     * with Flameshadow Conjuring. Their copy was made and removed correctly at
      * the end of the turn, but the "lose two life a turn" trigger still
      * happened twice.
      *

Mage.Tests/src/test/java/org/mage/test/cards/copy/PhyrexianMetamorphTest.java

@@ -52,9 +52,9 @@ public class PhyrexianMetamorphTest extends CardTestPlayerBase {
      * An opponent cast Phyrexian Metamorph and cloned another opponent's
      * Maelstrom Wanderer(his Commander). The first opponent then dealt combat
      * damage with Brago, King Eternal and chose to flicker several permanents,
-     * including the Phyrexian Metamorph/Maelstrom Wanderer, but he was not able
-     * to choose a new creature to clone when the Phyrexian Metamorph re-entered
-     * the battlefield.
+     * including the Phyrexian Metamorph/Maelstrom Wanderer, but that player was
+     * not able to choose a new creature to clone when the Phyrexian Metamorph
+     * re-entered the battlefield.
      */
     @Test
     public void testFlickerWithBrago() {

Mage.Tests/src/test/java/org/mage/test/cards/copy/ProgenitorMimicTest.java

@@ -140,7 +140,7 @@ public class ProgenitorMimicTest extends CardTestPlayerBase {
      * In a Commander FFA game, I controlled 5 vampires (one of which was
      * Captivating Vampire). My opponent cast Progenitor Mimic, copying
      * Captivating Vampire. I used the ability of my Captivating Vampire to gain
-     * control of his Mimic/Vampire but the buff didn't switch control. His
+     * control of their Mimic/Vampire but the buff didn't switch control. Their
      * other vampire still got the buff even after I gained control of the
      * Mimic/Vampire.
      *

Mage.Tests/src/test/java/org/mage/test/cards/enchantments/AuraTargetRemovedTest.java

@@ -18,7 +18,7 @@ public class AuraTargetRemovedTest extends CardTestPlayerBase {
 
     /**
      * Spreading Seas is bugged, opp casted it on my Field of Ruin, I sacced
-     * with the spell on stack but it resolved anyway and let him draw.
+     * with the spell on stack but it resolved anyway and let them draw.
      *
      * 303.4. Some enchantments have the subtype “Aura.” An Aura enters the
      * battlefield attached to an object or player. What an Aura can be attached

Mage.Tests/src/test/java/org/mage/test/cards/single/NecroticPlagueTest.java

@@ -49,8 +49,8 @@ public class NecroticPlagueTest extends CardTestPlayerBase {
          * Enchantment — Aura, 2BB
          * Enchant creature
          * Enchanted creature has "At the beginning of your upkeep, sacrifice this creature."
-         * When enchanted creature dies, its controller chooses target creature one of his or
-         * her opponents controls. Return Necrotic Plague from its owner's graveyard to the
+         * When enchanted creature dies, its controller chooses target creature one of their
+         * opponents controls. Return Necrotic Plague from its owner's graveyard to the
          * battlefield attached to that creature.
         */
         addCard(Zone.HAND, playerA, "Necrotic Plague");

Mage.Tests/src/test/java/org/mage/test/cards/single/emn/SpellQuellerTest.java

@@ -99,7 +99,7 @@ public class SpellQuellerTest extends CardTestPlayerBase {
     }
     
     /*
-     Reported bug: "...Spell Queller exiled my Nissa, Vastwood Seeker. Next turn he processed Nissa with Wasteland Strangler and killed my Tireless Tracker. 
+     Reported bug: "...Spell Queller exiled my Nissa, Vastwood Seeker. Next turn they processed Nissa with Wasteland Strangler and killed my Tireless Tracker. 
     I then cast Quarantine Field, targeting Spell Queller and Wasteland Strangler. That's when the error message occurred. (fatal exception)"
     */
     @Test

Mage.Tests/src/test/java/org/mage/test/cards/triggers/BurningEarthTest.java

@@ -15,7 +15,7 @@ public class BurningEarthTest extends CardTestPlayerBase {
     /**
      * Burning Earth - It doesn't cause the damage it should. My opponent taps a
      * Blood Crypt and an Overgrown Tomb for black and green mana respectively
-     * and casts his card all the while without taking any damage.
+     * and casts their card all the while without taking any damage.
      *
      */
     @Test

Mage.Tests/src/test/java/org/mage/test/cards/triggers/ShuffleTriggeredTest.java

@@ -40,7 +40,7 @@ public class ShuffleTriggeredTest extends CardTestPlayerBase {
      * lasse, dann triggert Widespread Panic für ihn (sollte garnicht triggern).
      * Bei Bribery ist es genauso.
      * 
-     * If I have an opponent shuffle his library using Knowledge Exploitation, Widespread Panic triggers for him (shoudn't trigger at all). Same thing with Bribery.
+     * If I have an opponent shuffle their library using Knowledge Exploitation, Widespread Panic triggers for them (shoudn't trigger at all). Same thing with Bribery.
      */
     @Test
     public void testWidespreadPanicDoesNotTriggerIfOpponentShufflesPlayersLibrary() {

Mage.Tests/src/test/java/org/mage/test/cards/triggers/dies/AngelicDestinyTest.java

@@ -16,7 +16,7 @@ public class AngelicDestinyTest extends CardTestPlayerBase {
 
     /**
      * I killed my opponent's Champion of the Parish, which was enchanted with Angelic Destiny. 
-     * However the Angelic Destiny went to the graveyard instead of returning his hand.
+     * However the Angelic Destiny went to the graveyard instead of returning to their hand.
      * 
      */
     @Test

Mage.Tests/src/test/java/org/mage/test/cards/triggers/dies/BrainMaggotTest.java

@@ -11,8 +11,8 @@ import org.mage.test.serverside.base.CardTestPlayerBase;
 public class BrainMaggotTest extends CardTestPlayerBase {
 
     /**
-     * When Brain Maggot enters the battlefield, target opponent reveals his or
-     * her hand and you choose a nonland card from it. Exile that card until
+     * When Brain Maggot enters the battlefield, target opponent reveals their
+     * hand and you choose a nonland card from it. Exile that card until
      * Brain Maggot leaves the battlefield.
      */
     @Test

Mage.Tests/src/test/java/org/mage/test/multiplayer/PlayerLeftGameRange1Test.java

@@ -125,9 +125,9 @@ public class PlayerLeftGameRange1Test extends CardTestMultiPlayerBase {
      * source if it has a proc. To reproduce, a Planeswalker was taken from an
      * original player's control, such as using Scrambleverse to shuffle Jace,
      * Unraveler of Secrets, to a second player and then the second player uses
-     * Jace's ability to create an emblem ("Whenever an opponent casts his or
-     * her first spell each turn, counter that spell."). Then the original
-     * player concedes the game and removes the Planeswalker. Once it becomes an
+     * Jace's ability to create an emblem ("Whenever an opponent casts their
+     * first spell each turn, counter that spell."). Then the original player
+     * concedes the game and removes the Planeswalker. Once it becomes an
      * opponent of the original player's turn and that opponent plays a spell,
      * Xmage throws an error and rollsback the turn.
      * <p>

Mage.Tests/src/test/java/org/mage/test/multiplayer/PlayerLeftGameRangeAllTest.java

@@ -124,9 +124,9 @@ public class PlayerLeftGameRangeAllTest extends CardTestMultiPlayerBase {
      * source if it has a proc. To reproduce, a Planeswalker was taken from an
      * original player's control, such as using Scrambleverse to shuffle Jace,
      * Unraveler of Secrets, to a second player and then the second player uses
-     * Jace's ability to create an emblem ("Whenever an opponent casts his or
-     * her first spell each turn, counter that spell."). Then the original
-     * player concedes the game and removes the Planeswalker. Once it becomes an
+     * Jace's ability to create an emblem ("Whenever an opponent casts their
+     * first spell each turn, counter that spell."). Then the original player
+     * concedes the game and removes the Planeswalker. Once it becomes an
      * opponent of the original player's turn and that opponent plays a spell,
      * Xmage throws an error and rollsback the turn.
      * <p>