konecnyjakub / mytester

src/Bridges/NetteDI/TestsRunner.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -5,9 +5,9 @@
 
 require_once __DIR__ . "/../../functions.php";
 
-use MyTester\TestCase,
-    MyTester\Environment,
-    Nette\Utils\Finder;
+use MyTester\TestCase;
+use MyTester\Environment;
+use Nette\Utils\Finder;
 
 /**
  * Tests Runner

Spacing +10 added lines, -10 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester\Bridges\NetteDI;
 
@@ -42,8 +42,8 @@ 
    * @return void
    */
   static function autoload(string $class): void {
-    foreach(static::$autoloader as $suit) {
-      if($suit[0] === $class) {
+    foreach (static::$autoloader as $suit) {
+      if ($suit[0] === $class) {
         require $suit[1];
         return;
       }
@@ -56,26 +56,26 @@ 
   function execute(): bool {
     $this->onExecute();
     $failed = false;
-    foreach($this->suits as $suit) {
+    foreach ($this->suits as $suit) {
       $result = $suit->run();
-      if(!$result) {
+      if (!$result) {
         $failed = true;
       }
     }
     Environment::printLine("");
-    foreach(Environment::getSkipped() as $skipped) {
+    foreach (Environment::getSkipped() as $skipped) {
       $reason = "";
-      if($skipped["reason"]) {
+      if ($skipped["reason"]) {
         $reason = ": {$skipped["reason"]}";
       }
       Environment::printLine("Skipped {$skipped["name"]}$reason");
     }
-    if($failed) {
+    if ($failed) {
       Environment::printLine("Failed");
       Environment::printLine("");
       $files = Finder::findFiles("*.errors")->in(\getTestsDirectory());
-      foreach($files as $name => $file) {
-        Environment::printLine("--- ". substr($file->getBaseName(), 0, -7));
+      foreach ($files as $name => $file) {
+        Environment::printLine("--- " . substr($file->getBaseName(), 0, -7));
         echo file_get_contents($name);
       }
     } else {

src/Tester.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -3,8 +3,8 @@
 
 namespace MyTester;
 
-use Nette\Utils\Finder,
-    Nette\Utils\FileSystem;
+use Nette\Utils\Finder;
+use Nette\Utils\FileSystem;
 
 /**
  * Automated tests runner

Spacing +10 added lines, -10 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester;
 
@@ -37,7 +37,7 @@ 
     $suits = [];
     $robot = new \Nette\Loaders\RobotLoader;
     $tempDir = "$folder/temp/cache/Robot.Loader";
-    if(is_dir("$folder/_temp")) {
+    if (is_dir("$folder/_temp")) {
       $tempDir = "$folder/_temp/cache/Robot.Loader";
     }
     FileSystem::createDir($tempDir);
@@ -47,9 +47,9 @@ 
     $robot->rebuild();
     $robot->register();
     $classes = $robot->getIndexedClasses();
-    foreach($classes as $class => $file) {
+    foreach ($classes as $class => $file) {
       $rc = new \Nette\Reflection\ClassType($class);
-      if(!$rc->isAbstract() AND $rc->isSubclassOf(TestCase::class)) {
+      if (!$rc->isAbstract() AND $rc->isSubclassOf(TestCase::class)) {
         $suits[] = [$rc->getName(), $file];
       }
     }
@@ -72,27 +72,27 @@ 
     Environment::setup();
     Environment::printInfo();
     $failed = false;
-    foreach($this->suits as $suit) {
+    foreach ($this->suits as $suit) {
       /** @var TestCase $suit */
       $suit = new $suit[0];
       $result = $suit->run();
-      if(!$result) {
+      if (!$result) {
         $failed = true;
       }
     }
     Environment::printLine("");
-    foreach(Environment::getSkipped() as $skipped) {
+    foreach (Environment::getSkipped() as $skipped) {
       $reason = "";
-      if($skipped["reason"]) {
+      if ($skipped["reason"]) {
         $reason = ": {$skipped["reason"]}";
       }
       Environment::printLine("Skipped {$skipped["name"]}$reason");
     }
-    if($failed) {
+    if ($failed) {
       Environment::printLine("Failed");
       Environment::printLine("");
       $files = Finder::findFiles("*.errors")->in(\getTestsDirectory());
-      foreach($files as $name => $file) {
+      foreach ($files as $name => $file) {
         Environment::printLine("--- " . substr($file->getBaseName(), 0, -7));
         echo file_get_contents($name);
       }

src/Bridges/NetteDI/MyTesterExtension.php

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester\Bridges\NetteDI;
 
@@ -27,12 +27,12 @@ 
     $builder = $this->getContainerBuilder();
     $builder->addDefinition($this->prefix("runner"))
       ->setClass(TestsRunner::class);
-    if(!is_dir($config["folder"])) {
+    if (!is_dir($config["folder"])) {
       throw new \Exception("Invalid folder {$config["folder"]} for $this->name.folder");
     }
     $tester = new \MyTester\Tester($config["folder"]);
     $this->suits = $tester->getSuits();
-    foreach($this->suits as $index => $suit) {
+    foreach ($this->suits as $index => $suit) {
       $builder->addDefinition($this->prefix("test." . ($index + 1)))
         ->setClass($suit[0])
         ->addTag(self::TAG);
@@ -51,14 +51,14 @@ 
     $initialize->addBody('spl_autoload_extensions(spl_autoload_extensions() . ",.phpt");
 MyTester\Bridges\NetteDI\TestsRunner::$autoloader = ?;
 spl_autoload_register(?);', [$this->suits, TestsRunner::class . "::autoload"]);
-    foreach($container->findByTag(self::TAG) as $suit => $foo) {
+    foreach ($container->findByTag(self::TAG) as $suit => $foo) {
       $initialize->addBody('$runner->addSuit($this->getService(?));', [$suit]);
     }
     $onExecute = array_merge(['MyTester\Environment::setup', 'MyTester\Environment::printInfo'], $config["onExecute"]);
-    foreach($onExecute as &$task) {
-      if(!is_array($task)) {
+    foreach ($onExecute as &$task) {
+      if (!is_array($task)) {
         $task = explode("::", $task);
-      } elseif(substr($task[0], 0, 1) === "@") {
+      } elseif (substr($task[0], 0, 1) === "@") {
         $initialize->addBody('$runner->onExecute[] = [$this->getService(?), ?];', [substr($task[0], 1), $task[1]]);
         continue;
       }

src/Assert.php

Spacing +22 added lines, -22 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester;
 
@@ -23,10 +23,10 @@ 
    */
   static function tryAssertion($code, string $successText = "", string $failureText = ""): void {
     $success = assert($code);
-    if(Environment::getShouldFail()) {
+    if (Environment::getShouldFail()) {
       $success = !$success;
     }
-    if($success) {
+    if ($success) {
       $message = ($successText === "") ? "Assertion \"$code\" is true." : $successText;
     } else {
       $message = ($failureText === "") ? "Assertion \"$code\" is not true." : $failureText;
@@ -43,10 +43,10 @@ 
    */
   static function same($expected, $actual): void {
     $success = ($expected == $actual);
-    if(Environment::getShouldFail()) {
+    if (Environment::getShouldFail()) {
       $success = !$success;
     }
-    if($success) {
+    if ($success) {
       $message = "The value is $expected.";
     } else {
       $message = "The value is not $expected but $actual.";
@@ -113,16 +113,16 @@ 
    * @return void
    */
   static function contains($needle, $actual): void {
-    if(!is_string($needle) AND !is_array($needle)) {
+    if (!is_string($needle) AND !is_array($needle)) {
       Environment::testResult("The variable is not string or array.", false);
-    } elseif(is_string($actual)) {
-      if($needle !== "" AND strpos($actual, $needle) !== FALSE) {
+    } elseif (is_string($actual)) {
+      if ($needle !== "" AND strpos($actual, $needle) !== FALSE) {
         Environment::testResult("$needle is in the variable.");
       } else {
         Environment::testResult("$needle is not in the variable.", false);
       }
-    } elseif(is_array($actual)) {
-      if(in_array($needle, $actual)) {
+    } elseif (is_array($actual)) {
+      if (in_array($needle, $actual)) {
         Environment::testResult("$needle is in the variable.");
       } else {
         Environment::testResult("$needle is not in the variable.", false);
@@ -140,16 +140,16 @@ 
    * @return void
    */
   static function notContains($needle, $actual): void {
-    if(!is_string($needle) AND !is_array($needle)) {
+    if (!is_string($needle) AND !is_array($needle)) {
       Environment::testResult("The variable is not string or array.", false);
-    } elseif(is_string($actual)) {
-      if($needle === "" OR strpos($actual, $needle) === FALSE) {
+    } elseif (is_string($actual)) {
+      if ($needle === "" OR strpos($actual, $needle) === FALSE) {
         Environment::testResult("$needle is not in the variable.");
       } else {
         Environment::testResult("$needle is in the variable.", false);
       }
-    } elseif(is_array($actual)) {
-      if(!in_array($needle, $actual)) {
+    } elseif (is_array($actual)) {
+      if (!in_array($needle, $actual)) {
         Environment::testResult("$needle is not in the variable.");
       } else {
         Environment::testResult("$needle is in the variable.", false);
@@ -167,9 +167,9 @@ 
    * @return void
    */
   static function count(int $count, $value): void {
-    if(!is_array($value) AND !$value instanceof \Countable) {
+    if (!is_array($value) AND !$value instanceof \Countable) {
       Environment::testResult("The variable is not array or countable object.", false);
-    } elseif(count($value) == $count) {
+    } elseif (count($value) == $count) {
       Environment::testResult("Count of the variable is $count.");
     } else {
       $actual = count($value);
@@ -185,9 +185,9 @@ 
    * @return void
    */
   static function notCount(int $count, $value): void {
-    if(!is_array($value) AND !$value instanceof \Countable) {
+    if (!is_array($value) AND !$value instanceof \Countable) {
       Environment::testResult("The variable is not array or countable object.", false);
-    } elseif(count($value) == $count) {
+    } elseif (count($value) == $count) {
       $actual = count($value);
       Environment::testResult("Count of the variable is $actual.", false);
     } else {
@@ -203,11 +203,11 @@ 
    * @return void
    */
   static function type($type, $value): void {
-    if(!is_object($type) AND !is_string($type)) {
+    if (!is_object($type) AND !is_string($type)) {
       Environment::testResult("Type must be string or object.", false);
-    } elseif(in_array($type, ["array", "bool", "callable", "float",
+    } elseif (in_array($type, ["array", "bool", "callable", "float",
       "int", "integer", "null", "object", "resource", "scalar", "string"], true)) {
-      if(!call_user_func("is_$type", $value)) {
+      if (!call_user_func("is_$type", $value)) {
         Environment::testResult("The variable is " . gettype($value) . ".", false);
       } else {
         Environment::testResult("The variable is $type.");

src/TestCase.php

Spacing +19 added lines, -19 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester;
 
@@ -22,37 +22,37 @@ 
    * @return bool|string
    */
   protected function checkSkip(\Nette\Reflection\Method $method) {
-    if(!$method->hasAnnotation("skip")) {
+    if (!$method->hasAnnotation("skip")) {
       return false;
     }
     $value = $method->getAnnotation("skip");
-    if(is_scalar($value)) {
+    if (is_scalar($value)) {
       return (bool) $value;
-    } elseif($value instanceof \Nette\Utils\ArrayHash) {
+    } elseif ($value instanceof \Nette\Utils\ArrayHash) {
       $skip = false;
       $reason = "";
-      foreach($value as $k => $v) {
+      foreach ($value as $k => $v) {
         switch ($k) {
           case "php":
             $skip = version_compare(PHP_VERSION, (string) $v, "<");
-            if($skip) {
+            if ($skip) {
               $reason = "PHP version is lesser than $v";
             }
             break;
           case "extension":
             $skip = !extension_loaded($v);
-            if($skip) {
+            if ($skip) {
               $reason = "extension $v is not loaded";
             }
             break;
           case "sapi":
             $skip = PHP_SAPI != $v;
-            if($skip) {
+            if ($skip) {
               $reason = "the sapi is not $v";
             }
             break;
         }
-        if($skip) {
+        if ($skip) {
           return $reason;
         }
       }
@@ -71,17 +71,17 @@ 
     $methods = array_values(preg_grep(static::METHOD_PATTERN, array_map(function(\ReflectionMethod $rm) {
       return $rm->getName();
     }, $r->getMethods())));
-    foreach($methods as $method) {
+    foreach ($methods as $method) {
       $rm = $r->getMethod($method);
       $data = NULL;
       $job = [
         "name" => $this->getJobName($rm), "callback" => [$this, $method], "params" => [], "skip" => $this->checkSkip($rm), "shouldFail" => $rm->hasAnnotation("fail")
       ];
-      if($rm->getNumberOfParameters() AND $rm->hasAnnotation("data")) {
+      if ($rm->getNumberOfParameters() AND $rm->hasAnnotation("data")) {
         $data = (array) $rm->getAnnotation("data");
       }
-      if(is_array($data)) {
-        foreach($data as $value) {
+      if (is_array($data)) {
+        foreach ($data as $value) {
           $job["params"][0] = $value;
           $jobs[] = new Job($job["name"], $job["callback"], $job["params"], $job["skip"], $job["shouldFail"]);
           $job["params"] = [];
@@ -101,7 +101,7 @@ 
   protected function getSuitName(): string {
     $suitName = get_class($this);
     $r = new \Nette\Reflection\ClassType($suitName);
-    if($r->hasAnnotation("testSuit")) {
+    if ($r->hasAnnotation("testSuit")) {
       $suitName = (string) $r->getAnnotation("testSuit");
     }
     return $suitName;
@@ -114,7 +114,7 @@ 
    * @return string
    */
   protected function getJobName(\Nette\Reflection\Method $method): string {
-    if($method->hasAnnotation("test")) {
+    if ($method->hasAnnotation("test")) {
       return (string) $method->getAnnotation("test");
     }
     return $this->getSuitName() . "::" . $method->getName();
@@ -163,11 +163,11 @@ 
   protected function runJob(Job $job): string {
     $jobName = $this->getJobName(\Nette\Reflection\Method::from($job->callback[0], $job->callback[1]));
     Environment::$currentJob = $jobName;
-    if(!$job->skip) {
+    if (!$job->skip) {
       $this->setUp();
     }
     $job->execute();
-    if(!$job->skip) {
+    if (!$job->skip) {
       $this->tearDown();
     }
     Environment::$currentJob = "";
@@ -195,9 +195,9 @@ 
     $jobs = $this->getJobs();
     $output = "";
     $passed = true;
-    foreach($jobs as $job) {
+    foreach ($jobs as $job) {
       $output .= $this->runJob($job);
-      if($job->result === "failed")  {
+      if ($job->result === "failed") {
         $passed = false;
       }
     }

src/Job.php

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester;
 
@@ -75,20 +75,20 @@ 
   function execute(): void {
     Environment::resetCounter();
     Environment::setShouldFail($this->shouldFail);
-    if($this->skip) {
+    if ($this->skip) {
       $this->result = "skipped";
       Environment::addSkipped($this->name, (!is_bool($this->skip) ? $this->skip : ""));
     } else {
       ob_start();
-      if(isset($this->callback)) {
+      if (isset($this->callback)) {
         call_user_func_array($this->callback, $this->params);
       }
       $output = ob_get_clean();
       $failed = Environment::checkFailed($output);
-      if($failed AND !$this->shouldFail) {
+      if ($failed AND !$this->shouldFail) {
         $this->result = "failed";
       }
-      if(strlen($output) AND $this->result === "failed") {
+      if (strlen($output) AND $this->result === "failed") {
         file_put_contents(\getTestsDirectory() . "/$this->name.errors", $output);
       }
     }

src/functions.php

Spacing +3 added lines, -3 removed lines

@@ -1,9 +1,9 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 function findVendorDirectory(): string {
   $recursionLimit = 10;
-  $findVendor = function ($dirName = "vendor/bin", $dir = __DIR__) use (&$findVendor, &$recursionLimit) {
+  $findVendor = function($dirName = "vendor/bin", $dir = __DIR__) use (&$findVendor, &$recursionLimit) {
     if (!$recursionLimit--) {
       throw new \Exception("Cannot find vendor directory.");
     }
@@ -17,7 +17,7 @@ 
 }
 
 function getTestsDirectory(): string {
-  if(!isset($argv[1])) {
+  if (!isset($argv[1])) {
     return dirname(findVendorDirectory()) . "/tests";
   }
   return $argv[1];

src/Environment.php

Indentation +4 added lines, -4 removed lines

@@ -45,10 +45,10 @@
     static::printLine($output . ". $text");
   }
    
-   /**
-   * @param string $results
-   * @return bool
-   */
+    /**
+     * @param string $results
+     * @return bool
+     */
   static function checkFailed(string $results): bool {
     $testsFailed = substr_count($results, " failed. ");
     return (bool) $testsFailed;

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace MyTester;
 
@@ -38,7 +38,7 @@ 
    */
   static function testResult(string $text, bool $success = true): void {
     static::incCounter();
-    if($success) {
+    if ($success) {
       return;
     }
     $output = "Test " . static::$taskCount . " failed";
@@ -102,7 +102,7 @@ 
    * @return void
    */
   static function printLine(string $text): void {
-    if(static::$mode == "http") {
+    if (static::$mode == "http") {
       $text .= "<br>";
     }
     echo "$text\n";
@@ -157,7 +157,7 @@ 
    * @return void
    */
   static function setup(): void {
-    if(static::$set) {
+    if (static::$set) {
       static::printLine("Warning: Testing Environment was already set up.");
       return;
     }

src/mytester.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 require_once __DIR__ . "/functions.php";