Update tests for #12024

Attempt to capture use case of using `_buildValidator` hook to define validators.
cakephp · Apr 30, 2018 · 309658f · 309658f
1 parent 9b16f36
commit 309658f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/tests/TestCase/Form/FormTest.php b/tests/TestCase/Form/FormTest.php
@@ -136,6 +136,8 @@ public function testValidateDeprected()
     {
         $this->deprecated(function () {
             $form = new ValidateForm();
+            $this->assertCount(1, $form->validator(), 'should have one rule');
+
             $data = [];
             $this->assertFalse($form->validate($data));
             $this->assertCount(1, $form->errors());

diff --git a/tests/test_app/TestApp/Form/ValidateForm.php b/tests/test_app/TestApp/Form/ValidateForm.php
@@ -19,7 +19,7 @@
 class ValidateForm extends Form
 {
 
-    public function validator(\Cake\Validation\Validator $validator = null)
+    protected function _buildValidator(\Cake\Validation\Validator $validator)
     {
         return parent::validator($validator)
             ->requirePresence('title');