made code changes

Author: LunguGeorgeProgramator

.phpunit.result.cache

@@ -1 +1 @@
-C:30:"PHPUnit\Runner\TestResultCache":802:{a:2:{s:7:"defects";a:9:{s:7:"Warning";i:6;s:17:"HeroTest::testOne";i:3;s:18:"HeroTest::testHero";i:4;s:31:"GameTest::testFirstTurnFunction";i:4;s:28:"GameTest::testSkillsFunction";i:4;s:26:"GameTest::testLogsMessages";i:4;s:24:"HeroTest::testHeroUpdate";i:4;s:21:"MonsterTest::testHero";i:4;s:27:"MonsterTest::testHeroUpdate";i:4;}s:5:"times";a:12:{s:7:"Warning";d:0.008;s:17:"HeroTest::testOne";d:0.008;s:18:"HeroTest::testHero";d:0.001;s:24:"HeroTest::testHeroUpdate";d:0;s:31:"GameTest::testFirstTurnFunction";d:0.031;s:28:"GameTest::testSkillsFunction";d:0.027;s:26:"GameTest::testLogsMessages";d:0.03;s:21:"MonsterTest::testHero";d:0;s:27:"MonsterTest::testHeroUpdate";d:0;s:24:"MonsterTest::testMonster";d:0;s:30:"MonsterTest::testMonsterUpdate";d:0;s:28:"HeroTest::testSkillsFunction";d:0.002;}}}
+C:30:"PHPUnit\Runner\TestResultCache":1048:{a:2:{s:7:"defects";a:12:{s:7:"Warning";i:6;s:17:"HeroTest::testOne";i:3;s:18:"HeroTest::testHero";i:4;s:31:"GameTest::testFirstTurnFunction";i:4;s:28:"GameTest::testSkillsFunction";i:4;s:26:"GameTest::testLogsMessages";i:4;s:24:"HeroTest::testHeroUpdate";i:4;s:21:"MonsterTest::testHero";i:4;s:27:"MonsterTest::testHeroUpdate";i:4;s:28:"MonsterTest::testLogInstance";i:4;s:27:"LoggerTest::testLogInstance";i:4;s:25:"GameTest::testMonsterCall";i:5;}s:5:"times";a:15:{s:7:"Warning";d:0.008;s:17:"HeroTest::testOne";d:0.008;s:18:"HeroTest::testHero";d:0.114;s:24:"HeroTest::testHeroUpdate";d:0;s:31:"GameTest::testFirstTurnFunction";d:0.035;s:28:"GameTest::testSkillsFunction";d:0.027;s:26:"GameTest::testLogsMessages";d:0.014;s:21:"MonsterTest::testHero";d:0;s:27:"MonsterTest::testHeroUpdate";d:0;s:24:"MonsterTest::testMonster";d:0;s:30:"MonsterTest::testMonsterUpdate";d:0;s:28:"HeroTest::testSkillsFunction";d:0.846;s:28:"MonsterTest::testLogInstance";d:0.035;s:27:"LoggerTest::testLogInstance";d:0.005;s:25:"GameTest::testMonsterCall";d:0.013;}}}

class/gameEngineClass.php

@@ -13,16 +13,14 @@ public function __construct()
 
     public function hero()
     {
-        $buildHero = new PlayerFactory;
-        return $buildHero->createPlayer('hero');
+        $build_hero = new PlayerFactory;
+        return $build_hero->createPlayer('hero');
     }
 
     public function monster()
     {
-        // $monsterEncounter = (new DataBase)->runQuery('SELECT `id` FROM `monster` ORDER BY RAND() LIMIT 0,1;');
-        $monsterEncounter = (new DataBase)->runQuery('SELECT `id` FROM `monster` where id = 1');
-        $buildMonster = new PlayerFactory;
-        return $buildMonster->createPlayer('monster', $monsterEncounter[0]['id']);
+        $build_monster = new PlayerFactory;
+        return $build_monster->createPlayer('monster');
     }
 
     public function figth($hero, $monster, $turn, $turnCounter = 1, $log_pass = '')

class/playerFactory.php

@@ -7,6 +7,8 @@ public function createPlayer($player, $player_id=1)
             $player_obj = new Hero();
         } elseif ($player === "monster") {
             $player_obj = new Monster();
+            $monster_encounter = (new DataBase)->runQuery('SELECT `id` FROM `monster` ORDER BY RAND() LIMIT 0,1;'); // for random id's
+            $player_id = $monster_encounter[0]['id'];
         }
 
         $player_obj = $this->getDBResults($player_obj, $player, $player_id);

tests/GameTest.php

@@ -59,4 +59,9 @@ public function testLogsMessages()
         $results = $engine->figth($this->populateHero(), $this->populateMonster(), [], 1, Logger::getInstance());
         $this->assertStringContainsString('Damage', $results[5]->getLog());
     }
+
+    public function testMonsterCall(){
+        $engine = new engine;
+        var_dump($engine->monster());
+    }
 }

tests/LoggerTest.php

@@ -0,0 +1,13 @@
+<?php
+include_once 'class/logger.php';
+
+use PHPUnit\Framework\TestCase;
+
+final class LoggerTest extends TestCase
+{
+    public function testLogInstance(){
+        $log = new Logger;
+        $this->assertNotNull($log->getLog());
+    }
+}
+?>