From 6743106d9a8c760580690aab704f908766731801 Mon Sep 17 00:00:00 2001
From: Igor Scheller <igor.scheller@igorshp.de>
Date: Wed, 10 Jul 2019 13:34:15 +0200
Subject: Replaced validation with `respect/validation`

---
 tests/Unit/Controllers/AuthControllerTest.php  |   3 +-
 tests/Unit/Http/Validation/Rules/InTest.php    |  19 ++
 tests/Unit/Http/Validation/Rules/NotInTest.php |  20 ++
 tests/Unit/Http/Validation/ValidatesTest.php   | 308 -------------------------
 tests/Unit/Http/Validation/ValidatorTest.php   |  42 +++-
 5 files changed, 75 insertions(+), 317 deletions(-)
 create mode 100644 tests/Unit/Http/Validation/Rules/InTest.php
 create mode 100644 tests/Unit/Http/Validation/Rules/NotInTest.php
 delete mode 100644 tests/Unit/Http/Validation/ValidatesTest.php

(limited to 'tests')

diff --git a/tests/Unit/Controllers/AuthControllerTest.php b/tests/Unit/Controllers/AuthControllerTest.php
index d3dbfa4b..c3d9659c 100644
--- a/tests/Unit/Controllers/AuthControllerTest.php
+++ b/tests/Unit/Controllers/AuthControllerTest.php
@@ -8,7 +8,6 @@ use Engelsystem\Http\Exceptions\ValidationException;
 use Engelsystem\Http\Request;
 use Engelsystem\Http\Response;
 use Engelsystem\Http\UrlGeneratorInterface;
-use Engelsystem\Http\Validation\Validates;
 use Engelsystem\Http\Validation\Validator;
 use Engelsystem\Models\User\Settings;
 use Engelsystem\Models\User\User;
@@ -66,7 +65,7 @@ class AuthControllerTest extends TestCase
         list(, , $url, $auth) = $this->getMocks();
         $session = new Session(new MockArraySessionStorage());
         /** @var Validator|MockObject $validator */
-        $validator = new Validator(new Validates());
+        $validator = new Validator();
 
         $user = new User([
             'name'          => 'foo',
diff --git a/tests/Unit/Http/Validation/Rules/InTest.php b/tests/Unit/Http/Validation/Rules/InTest.php
new file mode 100644
index 00000000..e5688d90
--- /dev/null
+++ b/tests/Unit/Http/Validation/Rules/InTest.php
@@ -0,0 +1,19 @@
+<?php
+
+namespace Engelsystem\Test\Unit\Http\Validation\Rules;
+
+use Engelsystem\Http\Validation\Rules\In;
+use Engelsystem\Test\Unit\TestCase;
+
+class InTest extends TestCase
+{
+    /**
+     * @covers \Engelsystem\Http\Validation\Rules\In::__construct
+     */
+    public function testConstruct()
+    {
+        $rule = new In('foo,bar');
+
+        $this->assertEquals(['foo', 'bar'], $rule->haystack);
+    }
+}
diff --git a/tests/Unit/Http/Validation/Rules/NotInTest.php b/tests/Unit/Http/Validation/Rules/NotInTest.php
new file mode 100644
index 00000000..9be12336
--- /dev/null
+++ b/tests/Unit/Http/Validation/Rules/NotInTest.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace Engelsystem\Test\Unit\Http\Validation\Rules;
+
+use Engelsystem\Http\Validation\Rules\NotIn;
+use Engelsystem\Test\Unit\TestCase;
+
+class NotInTest extends TestCase
+{
+    /**
+     * @covers \Engelsystem\Http\Validation\Rules\NotIn::validate
+     */
+    public function testConstruct()
+    {
+        $rule = new NotIn('foo,bar');
+
+        $this->assertTrue($rule->validate('lorem'));
+        $this->assertFalse($rule->validate('foo'));
+    }
+}
diff --git a/tests/Unit/Http/Validation/ValidatesTest.php b/tests/Unit/Http/Validation/ValidatesTest.php
deleted file mode 100644
index 5cf0447a..00000000
--- a/tests/Unit/Http/Validation/ValidatesTest.php
+++ /dev/null
@@ -1,308 +0,0 @@
-<?php
-
-namespace Engelsystem\Test\Unit\Http\Validation;
-
-use Engelsystem\Http\Validation\Validates;
-use InvalidArgumentException;
-use PHPUnit\Framework\TestCase;
-
-class ValidatesTest extends TestCase
-{
-    /**
-     * @return array
-     */
-    public function provideAccepted()
-    {
-        return [
-            ['true'],
-            ['1'],
-            ['y'],
-            ['yes'],
-            ['on'],
-            ['1test', false],
-            ['false', false],
-            ['no', false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::accepted
-     * @param mixed $value
-     * @param bool  $result
-     * @dataProvider provideAccepted
-     */
-    public function testAccepted($value, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->accepted($value) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideBetween()
-    {
-        return [
-            ['42', [10, 100]],
-            [42.5, [42, 43]],
-            [42, [42, 1000]],
-            [1337, [0, 99], false],
-            [-17, [32, 45], false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::between
-     * @param mixed $value
-     * @param array $parameters
-     * @param bool  $result
-     * @dataProvider provideBetween
-     */
-    public function testBetween($value, array $parameters, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->between($value, $parameters) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideBool()
-    {
-        return [
-            ['1'],
-            [1],
-            [true],
-            ['0'],
-            [0],
-            [false],
-            ['true', false],
-            ['false', false],
-            ['yes', false],
-            ['no', false],
-            ['bool', false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::bool
-     * @param mixed $value
-     * @param bool  $result
-     * @dataProvider provideBool
-     */
-    public function testBool($value, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->bool($value) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideIn()
-    {
-        return [
-            ['lorem', ['lorem,ipsum,dolor']],
-            [99, ['66,77,88,99,111']],
-            [4, ['1,3,5,7'], false],
-            ['toggle', ['on,off'], false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::in
-     * @param mixed $value
-     * @param array $parameters
-     * @param bool  $result
-     * @dataProvider provideIn
-     */
-    public function testIn($value, array $parameters, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->in($value, $parameters) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideInt()
-    {
-        return [
-            ['1337'],
-            [42],
-            ['0'],
-            [false, false],
-            ['12asd1', false],
-            ['one', false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::int
-     * @param mixed $value
-     * @param bool  $result
-     * @dataProvider provideInt
-     */
-    public function testInt($value, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->int($value) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideMax()
-    {
-        return [
-            ['99', [100]],
-            [-42, [1024]],
-            [99, [99]],
-            [100, [10], false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::max
-     * @param mixed $value
-     * @param array $parameters
-     * @param bool  $result
-     * @dataProvider provideMax
-     */
-    public function testMax($value, array $parameters, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->max($value, $parameters) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideMin()
-    {
-        return [
-            [32, [0]],
-            [7, [7]],
-            ['99', [10]],
-            [3, [42], false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::min
-     * @param mixed $value
-     * @param array $parameters
-     * @param bool  $result
-     * @dataProvider provideMin
-     */
-    public function testMin($value, array $parameters, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->min($value, $parameters) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideNotIn()
-    {
-        return [
-            [77, ['50,60,70']],
-            ['test', ['coding,deployment']],
-            ['PHP', ['Java,PHP,bash'], false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::notIn
-     * @param mixed $value
-     * @param array $parameters
-     * @param bool  $result
-     * @dataProvider provideNotIn
-     */
-    public function testNotIn($value, array $parameters, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->notIn($value, $parameters) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideNumeric()
-    {
-        return [
-            [77],
-            ['42'],
-            ['1337e0'],
-            ['123f00', false],
-            [null, false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::numeric
-     * @param mixed $value
-     * @param bool  $result
-     * @dataProvider provideNumeric
-     */
-    public function testNumeric($value, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->numeric($value) === $result);
-    }
-
-    /**
-     * @return array
-     */
-    public function provideRequired()
-    {
-        return [
-            ['Lorem ipsum'],
-            ['1234'],
-            [1234],
-            ['0'],
-            [0],
-            ['', false],
-            [' ', false],
-            [null, false],
-        ];
-    }
-
-    /**
-     * @covers       \Engelsystem\Http\Validation\Validates::required
-     * @param mixed $value
-     * @param bool  $result
-     * @dataProvider provideRequired
-     */
-    public function testRequired($value, bool $result = true)
-    {
-        $val = new Validates;
-        $this->assertTrue($val->required($value) === $result);
-    }
-
-    /**
-     * @covers \Engelsystem\Http\Validation\Validates::getSize
-     */
-    public function testGetSize()
-    {
-        $val = new Validates;
-        $this->assertTrue($val->max(42, [999]));
-        $this->assertTrue($val->max('99', [100]));
-        $this->assertFalse($val->max('101', [100]));
-        $this->assertTrue($val->max('lorem', [5]));
-        $this->assertFalse($val->max('Lorem Ipsum', [5]));
-    }
-
-    /**
-     * @covers \Engelsystem\Http\Validation\Validates::validateParameterCount
-     */
-    public function testValidateParameterCount()
-    {
-        $val = new Validates;
-        $this->assertTrue($val->between(42, [1, 100]));
-
-        $this->expectException(InvalidArgumentException::class);
-        $val->between(42, [1]);
-    }
-}
diff --git a/tests/Unit/Http/Validation/ValidatorTest.php b/tests/Unit/Http/Validation/ValidatorTest.php
index 799265ec..0790b7a8 100644
--- a/tests/Unit/Http/Validation/ValidatorTest.php
+++ b/tests/Unit/Http/Validation/ValidatorTest.php
@@ -2,7 +2,6 @@
 
 namespace Engelsystem\Test\Unit\Http\Validation;
 
-use Engelsystem\Http\Validation\Validates;
 use Engelsystem\Http\Validation\Validator;
 use InvalidArgumentException;
 use PHPUnit\Framework\TestCase;
@@ -10,19 +9,18 @@ use PHPUnit\Framework\TestCase;
 class ValidatorTest extends TestCase
 {
     /**
-     * @covers \Engelsystem\Http\Validation\Validator::__construct
      * @covers \Engelsystem\Http\Validation\Validator::validate
      * @covers \Engelsystem\Http\Validation\Validator::getData
      * @covers \Engelsystem\Http\Validation\Validator::getErrors
      */
     public function testValidate()
     {
-        $val = new Validator(new Validates);
+        $val = new Validator();
         $this->assertTrue($val->validate(
-            ['foo' => 'bar', 'lorem' => 'on'],
-            ['foo' => 'required|not_in:lorem,ipsum,dolor', 'lorem' => 'accepted']
+            ['foo' => 'bar', 'lorem' => 'on', 'dolor' => 'bla'],
+            ['lorem' => 'accepted']
         ));
-        $this->assertEquals(['foo' => 'bar', 'lorem' => 'on'], $val->getData());
+        $this->assertEquals(['lorem' => 'on'], $val->getData());
 
         $this->assertFalse($val->validate(
             [],
@@ -39,7 +37,7 @@ class ValidatorTest extends TestCase
      */
     public function testValidateNotImplemented()
     {
-        $val = new Validator(new Validates);
+        $val = new Validator();
         $this->expectException(InvalidArgumentException::class);
 
         $val->validate(
@@ -47,4 +45,34 @@ class ValidatorTest extends TestCase
             ['foo' => 'never_implemented']
         );
     }
+
+    /**
+     * @covers \Engelsystem\Http\Validation\Validator::map
+     * @covers \Engelsystem\Http\Validation\Validator::mapBack
+     */
+    public function testValidateMapping()
+    {
+        $val = new Validator();
+        $this->assertTrue($val->validate(
+            ['foo' => 'bar'],
+            ['foo' => 'required']
+        ));
+        $this->assertTrue($val->validate(
+            ['foo' => '0'],
+            ['foo' => 'int']
+        ));
+        $this->assertTrue($val->validate(
+            ['foo' => 'on'],
+            ['foo' => 'accepted']
+        ));
+
+        $this->assertFalse($val->validate(
+            [],
+            ['lorem' => 'required']
+        ));
+        $this->assertEquals(
+            ['lorem' => ['validation.lorem.required']],
+            $val->getErrors()
+        );
+    }
 }
-- 
cgit v1.2.3-54-g00ecf