Fixes

cakephp · Mar 6, 2017 · 2b580b0 · 2b580b0
1 parent 6dd7479
commit 2b580b0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/src/Validation/Validator.php b/src/Validation/Validator.php
@@ -57,7 +57,7 @@ class Validator implements ArrayAccess, IteratorAggregate, Countable
      *
      * @var array
      */
-    private static $_defaultProviders = [];
+    protected static $_defaultProviders = [];
 
     /**
      * Contains the validation messages associated with checking the presence

diff --git a/tests/TestCase/Validation/ValidatorTest.php b/tests/TestCase/Validation/ValidatorTest.php
@@ -1887,11 +1887,11 @@ protected function assertProxyMethod($validator, $method, $extra = null, $pass =
      */
     public function testAddingDefaultProvider()
     {
-        $validator = new Validator;
+        $validator = new Validator();
         $this->assertEmpty($validator->providers(), 'Providers should be empty');
 
-        Validator::addDefaultProvider('test-provider', '\MyNameSpace\Validation\MyProvider');
-        $validator = new Validator;
+        Validator::addDefaultProvider('test-provider', 'MyNameSpace\Validation\MyProvider');
+        $validator = new Validator();
         $this->assertEquals($validator->providers(), ['test-provider'], 'Default provider `test-provider` is missing');
     }
 
@@ -1902,13 +1902,13 @@ public function testAddingDefaultProvider()
      */
     public function testGetDefaultProvider()
     {
-        Validator::addDefaultProvider('test-provider', '\MyNameSpace\Validation\MyProvider');
-        $this->assertEquals(Validator::getDefaultProvider('test-provider'), '\MyNameSpace\Validation\MyProvider', 'Default provider `test-provider` is missing');
+        Validator::addDefaultProvider('test-provider', 'MyNameSpace\Validation\MyProvider');
+        $this->assertEquals(Validator::getDefaultProvider('test-provider'), 'MyNameSpace\Validation\MyProvider', 'Default provider `test-provider` is missing');
 
         $this->assertNull(Validator::getDefaultProvider('invalid-provider'), 'Default provider (`invalid-provider`) should be missing');
         $this->assertNull(Validator::getDefaultProvider(null), 'Default provider (null) should be missing');
-        
-        Validator::addDefaultProvider('test-provider2', '\MyNameSpace\Validation\MySecondProvider');
+
+        Validator::addDefaultProvider('test-provider2', 'MyNameSpace\Validation\MySecondProvider');
         $this->assertEquals(Validator::getDefaultProviders(), ['test-provider', 'test-provider2'], 'Default providers incorrect');
     }
 }