alex-kalanis / kw_rules

php-tests/BasicTests/FormatRulesTest.php

Spacing +3 added lines, -3 removed lines

@@ -57,9 +57,9 @@
     public function compareFormatProvider()
     {
         return [
-            [false,           true,  false, false],
-            [123,             false, true,  false],
-            ['abc',           false, false, true],
+            [false, true, false, false],
+            [123, false, true, false],
+            ['abc', false, false, true],
             [new \stdClass(), false, false, false],
         ];
     }

Braces +9 added lines, -3 removed lines

@@ -18,7 +18,9 @@ 
     {
         $data = new Rules\IsBool();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
-        if (!$resultBool) $this->expectException(RuleException::class);
+        if (!$resultBool) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -34,7 +36,9 @@ 
     {
         $data = new Rules\IsNumeric();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
-        if (!$resultNum) $this->expectException(RuleException::class);
+        if (!$resultNum) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -50,7 +54,9 @@ 
     {
         $data = new Rules\IsString();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
-        if (!$resultStr) $this->expectException(RuleException::class);
+        if (!$resultStr) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 

php-tests/BasicTests/LengthRulesTest.php

Spacing +3 added lines, -3 removed lines

@@ -63,9 +63,9 @@
     public function compareLenProvider()
     {
         return [
-            ['yxcvbnm',   8, false, false, true ],
-            ['asdfghjk',  8, true,  true,  true ],
-            ['qwertutop', 8, true,  false, false],
+            ['yxcvbnm', 8, false, false, true],
+            ['asdfghjk', 8, true, true, true],
+            ['qwertutop', 8, true, false, false],
         ];
     }
 }

Braces +9 added lines, -3 removed lines

@@ -20,7 +20,9 @@ 
         $data = new Rules\LengthMin();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $data->setAgainstValue($length);
-        if (!$passMin) $this->expectException(RuleException::class);
+        if (!$passMin) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -38,7 +40,9 @@ 
         $data = new Rules\LengthEquals();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $data->setAgainstValue($length);
-        if (!$passEq) $this->expectException(RuleException::class);
+        if (!$passEq) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -56,7 +60,9 @@ 
         $data = new Rules\LengthMax();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $data->setAgainstValue($length);
-        if (!$passMax) $this->expectException(RuleException::class);
+        if (!$passMax) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 

php-tests/BasicTests/BasicRulesTest.php

Spacing +5 added lines, -5 removed lines

@@ -91,11 +91,11 @@
         return [
             ['5', '7', true, false],
             [[new \stdClass()], '7', true, false],
-            [[1,3,5,9], '7', false, false],
-            [[1,3,5,9], '5', false, true],
-            [['2','4','6','8'], 6, false, true],
-            [['abc','def','ghi','jkl'], 'mno', false, false],
-            [['abc','def','ghi','jkl'], 'def', false, true],
+            [[1, 3, 5, 9], '7', false, false],
+            [[1, 3, 5, 9], '5', false, true],
+            [['2', '4', '6', '8'], 6, false, true],
+            [['abc', 'def', 'ghi', 'jkl'], 'mno', false, false],
+            [['abc', 'def', 'ghi', 'jkl'], 'def', false, true],
         ];
     }
 

Braces +39 added lines, -13 removed lines

@@ -22,7 +22,9 @@ 
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
 
         $mock = MockEntry::init($key, $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -40,7 +42,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init($key, $checkValue);
-        if ($gotResult) $this->expectException(RuleException::class);
+        if ($gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -56,11 +60,15 @@ 
     {
         $data = new Rules\IsInArray();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
-        if ($gotError) $this->expectException(RuleException::class);
+        if ($gotError) {
+            $this->expectException(RuleException::class);
+        }
         $data->setAgainstValue($expectedValues);
         if (!$gotError) {
             $mock = MockEntry::init('foo', $checkValue);
-            if (!$gotResult) $this->expectException(RuleException::class);
+            if (!$gotResult) {
+                $this->expectException(RuleException::class);
+            }
             $data->validate($mock);
         }
     }
@@ -77,11 +85,15 @@ 
     {
         $data = new Rules\IsNotInArray();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
-        if ($gotError) $this->expectException(RuleException::class);
+        if ($gotError) {
+            $this->expectException(RuleException::class);
+        }
         $data->setAgainstValue($expectedValues);
         if (!$gotError) {
             $mock = MockEntry::init('foo', $checkValue);
-            if ($gotResult) $this->expectException(RuleException::class);
+            if ($gotResult) {
+                $this->expectException(RuleException::class);
+            }
             $data->validate($mock);
         }
     }
@@ -112,7 +124,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -129,7 +143,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -146,7 +162,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if ($gotResult) $this->expectException(RuleException::class);
+        if ($gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -163,7 +181,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if ($gotResult) $this->expectException(RuleException::class);
+        if ($gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -211,7 +231,9 @@ 
         $data = new Rules\IsEmpty();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if ($gotResult) $this->expectException(RuleException::class);
+        if ($gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -226,7 +248,9 @@ 
         $data = new Rules\IsFilled();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -252,7 +276,9 @@ 
         $data = new Rules\IsJsonString();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/FileRulesTest.php

Spacing +3 added lines, -3 removed lines

@@ -79,7 +79,7 @@ 
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\File\AFileRule', $data);
         $data->setAgainstValue($maxSize);
         $mock = MockFile::init('foo', 'text0.txt', 'text/plain',
-            '', $fileSize, UPLOAD_ERR_OK );
+            '', $fileSize, UPLOAD_ERR_OK);
         if (!$match) $this->expectException(RuleException::class);
         $data->validate($mock);
     }
@@ -87,10 +87,10 @@ 
     public function sizeMatchProvider()
     {
         return [
-            ['32',  128,   false],
+            ['32', 128, false],
             ['10g', 46843, true],
             ['15m', 84641, true],
-            ['30k', 3534,  true],
+            ['30k', 3534, true],
             ['30k', 35534, false],
         ];
     }

Braces +3 added lines, -1 removed lines

@@ -80,7 +80,9 @@
         $data->setAgainstValue($maxSize);
         $mock = MockFile::init('foo', 'text0.txt', 'text/plain',
             '', $fileSize, UPLOAD_ERR_OK );
-        if (!$match) $this->expectException(RuleException::class);
+        if (!$match) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/PatternRulesTest.php

Spacing +4 added lines, -4 removed lines

@@ -45,11 +45,11 @@
     public function stringsProvider()
     {
         return [
-            ['foo@bar.example',     true,  false],
-            ['foo@bar.baz.example', true,  false],
+            ['foo@bar.example', true, false],
+            ['foo@bar.baz.example', true, false],
             ['foo@bar@baz.example', false, false],
-            ['6',                   false, false],
-            ['bar.example',         false, true ],
+            ['6', false, false],
+            ['bar.example', false, true],
             ['foo.bar.baz.example', false, false],
         ];
     }

Braces +6 added lines, -2 removed lines

@@ -20,7 +20,9 @@ 
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
 
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$isEmail) $this->expectException(RuleException::class);
+        if (!$isEmail) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -38,7 +40,9 @@ 
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
 
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$isDomain) $this->expectException(RuleException::class);
+        if (!$isDomain) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/SafeRulesTest.php

Braces +6 added lines, -2 removed lines

@@ -20,7 +20,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init($key, $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -38,7 +40,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init($key, $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/CallbackRulesTest.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
             [123, '', false, false],
             ['asdf', '', false, false],
             ['CallbackRulesTest::callMeStatic', null, true, false],
-            ['CallbackRulesTest::callMeStatic', 1, true, true ],
+            ['CallbackRulesTest::callMeStatic', 1, true, true],
             [[$this, 'callMeDynamic'], null, true, false],
             [[$this, 'callMeDynamic'], 1, true, true],
         ];

Braces +6 added lines, -2 removed lines

@@ -18,11 +18,15 @@
     {
         $data = new Rules\ProcessCallback();
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->setAgainstValue($expectedCall);
         if ($gotResult) {
             $mock = MockEntry::init('foo', $checkValue);
-            if (!$pass) $this->expectException(RuleException::class);
+            if (!$pass) {
+                $this->expectException(RuleException::class);
+            }
             $data->validate($mock);
         }
     }

php-tests/BasicTests/MatchRulesTest.php

Braces +6 added lines, -2 removed lines

@@ -69,7 +69,9 @@ 
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
 
         $mock = MockEntry::init('foo', 'bar');
-        if (!$matchAll) $this->expectException(RuleException::class);
+        if (!$matchAll) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -86,7 +88,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf('\kalanis\kw_rules\Rules\ARule', $data);
         $mock = MockEntry::init('foo', 'bar');
-        if (!$matchAny) $this->expectException(RuleException::class);
+        if (!$matchAny) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/FileFactoryTest.php

Braces +6 added lines, -2 removed lines

@@ -25,9 +25,13 @@
     public function testFileFactoryAvailability(string $rule, bool $gotResult)
     {
         $factory = new Rules\File\Factory();
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data = $factory->getRule($rule);
-        if ($data) $this->assertInstanceOf('\kalanis\kw_rules\Rules\File\AFileRule', $data);
+        if ($data) {
+            $this->assertInstanceOf('\kalanis\kw_rules\Rules\File\AFileRule', $data);
+        }
     }
 
     public function inputFilesProvider()