[Cleanup] Pass Player into execution constructor instead of PlayerID (#1022)

## Description:
Answering issue:  #1017 
[Cleanup] Pass Player into the execution constructor instead of PlayerID

I have tested the changes running and playing a full game. I do not know
other way to test the changes, please inform me ❤️

## Please complete the following:

- [x] I have added screenshots for all UI updates
- [x] I process any text displayed to the user through translateText()
and I've added it to the en.json file
- [x] I have added relevant tests to the test directory
- [x] I confirm I have thoroughly tested these changes and take full
responsibility for any bugs introduced
- [x] I understand that submitting code with bugs that could have been
caught through manual testing blocks releases and new features for all
contributors

## Please put your Discord username so you can be contacted if a bug or
regression is found:

Lele

---------

Co-authored-by: lva <lva@rovsing.dk>

tests/util/utils.ts

@@ -4,18 +4,18 @@
 // If you also need execution use function below. Does not work with things not
 
 import { ConstructionExecution } from "../../src/core/execution/ConstructionExecution";
-import { Game, PlayerID, UnitType } from "../../src/core/game/Game";
+import { Game, Player, UnitType } from "../../src/core/game/Game";
 
 // built via UI (e.g.: trade ships)
 export function constructionExecution(
   game: Game,
-  playerID: PlayerID,
+  _owner: Player,
   x: number,
   y: number,
   unit: UnitType,
   ticks = 4,
 ) {
-  game.addExecution(new ConstructionExecution(playerID, game.ref(x, y), unit));
+  game.addExecution(new ConstructionExecution(_owner, game.ref(x, y), unit));
 
   // 4 ticks by default as it usually goes like this
   // Init of construction execution