From 6dd7479e8b4fa4899fad7944bd1db5dec519858f Mon Sep 17 00:00:00 2001
From: lilHermit <developer@lilhermit.co.uk>
Date: Mon, 6 Mar 2017 11:13:35 +0000
Subject: [PATCH] Changes suggested by dereuromark Added tests for
 getDefaultProvider and getDefaultProviders

---
 src/Validation/Validator.php                |  6 +++---
 tests/TestCase/Validation/ValidatorTest.php | 17 +++++++++++++++++
 2 files changed, 20 insertions(+), 3 deletions(-)

diff --git a/src/Validation/Validator.php b/src/Validation/Validator.php
index d2f3c381f69..c786b09cbfc 100644
--- a/src/Validation/Validator.php
+++ b/src/Validation/Validator.php
@@ -226,11 +226,11 @@ public function getProvider($name)
      */
     public static function getDefaultProvider($name)
     {
-        if (isset(self::$_defaultProviders[$name])) {
-            return self::$_defaultProviders[$name];
+        if (!isset(self::$_defaultProviders[$name])) {
+            return null;
         }
 
-        return null;
+        return self::$_defaultProviders[$name];
     }
 
     /**
diff --git a/tests/TestCase/Validation/ValidatorTest.php b/tests/TestCase/Validation/ValidatorTest.php
index beea3164b4c..93028b617c0 100644
--- a/tests/TestCase/Validation/ValidatorTest.php
+++ b/tests/TestCase/Validation/ValidatorTest.php
@@ -1894,4 +1894,21 @@ public function testAddingDefaultProvider()
         $validator = new Validator;
         $this->assertEquals($validator->providers(), ['test-provider'], 'Default provider `test-provider` is missing');
     }
+
+    /**
+     * Testing getting DefaultProvider(s)
+     *
+     * @return void
+     */
+    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');
+
+        $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');
+        $this->assertEquals(Validator::getDefaultProviders(), ['test-provider', 'test-provider2'], 'Default providers incorrect');
+    }
 }