Fix code formatting and add new composer command: run-tidy

PoLaKoSz · Jul 26, 2019 · c019aa4 · c019aa4
1 parent dd03ed4
commit c019aa4
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -43,3 +43,7 @@ var_dump( $movies );
 - `$ composer run-i-tests`: runs only the integration tests (saved webpage parsing)
 - `$ composer run-r-tests`: runs only the regression tests (to detect HTML DOM changes in the endpoints - downloads webpage(s) from Port.hu and after try to parse them)
 - `$ composer run-u-tests`: runs only the unit tests
+
+## Code formatting
+
+Please use the `$ composer run-tidy` command before every commit to make sure the source code formatted properly.
diff --git a/composer.json b/composer.json
@@ -39,6 +39,8 @@
         "run-all-tests": ".\\vendor\\bin\\phpunit --bootstrap .\\vendor\\autoload.php --testdox .\\tests\\",
         "run-i-tests": ".\\vendor\\bin\\phpunit --bootstrap .\\vendor\\autoload.php --testdox .\\tests\\Integration",
         "run-r-tests": ".\\vendor\\bin\\phpunit --bootstrap .\\vendor\\autoload.php --testdox .\\tests\\Regression",
-        "run-u-tests": ".\\vendor\\bin\\phpunit --bootstrap .\\vendor\\autoload.php --testdox .\\tests\\Unit"
+        "run-u-tests": ".\\vendor\\bin\\phpunit --bootstrap .\\vendor\\autoload.php --testdox .\\tests\\Unit",
+
+        "run-tidy": ".\\vendor\\bin\\phpcbf --ignore=.\\tests\\Integration\\StaticResources\\*,vendor\\* --extensions=php ./ --standard=PSR2"
     }
 }
diff --git a/tests/Integration/Parsers/MoviePage/WithoutPosterTest.php b/tests/Integration/Parsers/MoviePage/WithoutPosterTest.php
@@ -10,7 +10,7 @@
 class WithoutPosterTest extends MoviePageParserBase
 {
     public function __construct()
-    {        
+    {
         $expectedModel = new WithoutPoster();
 
         parent::__construct('visko', $expectedModel);