Remove deprecated method usage in tests. Make typehinting possible which reveals coding errors.

tests/TestCase/View/HelperRegistryTest.php

@@ -44,6 +44,11 @@ class HelperRegistryTest extends TestCase
     public $Helpers;
 
     /**
+     * @var \Cake\Event\EventManager
+     */
+    public $Events;
+
+    /**
      * setUp
      *
      * @return void
@@ -241,7 +246,7 @@ class HelperRegistryTest extends TestCase
         );
         $this->assertCount(1, $this->Events->listeners('View.beforeRender'));
 
-        $this->assertNull($this->Helpers->reset(), 'No return expected');
+        $this->Helpers->reset();
         $this->assertCount(0, $this->Events->listeners('View.beforeRender'));
 
         $this->assertNotSame($instance, $this->Helpers->load('EventListenerTest'));

tests/TestCase/View/HelperTest.php

@@ -44,19 +44,6 @@ class TestHelper extends Helper
      */
     public $helpers = ['Html', 'TestPlugin.OtherHelper'];
 
-    /**
-     * expose a method as public
-     *
-     * @param string $options
-     * @param string $exclude
-     * @param string $insertBefore
-     * @param string $insertAfter
-     * @return void
-     */
-    public function parseAttributes($options, $exclude = null, $insertBefore = ' ', $insertAfter = null)
-    {
-        return $this->_parseAttributes($options, $exclude, $insertBefore, $insertAfter);
-    }
 }
 
 /**
@@ -66,6 +53,11 @@ class HelperTest extends TestCase
 {
 
     /**
+     * @var \Cake\View\View
+     */
+    public $View;
+
+    /**
      * setUp method
      *
      * @return void
@@ -76,8 +68,6 @@ class HelperTest extends TestCase
 
         Router::reload();
         $this->View = new View();
-        $this->Helper = new Helper($this->View);
-        $this->Helper->request = new Request();
     }
 
     /**
@@ -91,7 +81,7 @@ class HelperTest extends TestCase
         Configure::delete('Asset');
 
         Plugin::unload();
-        unset($this->Helper, $this->View);
+        unset($this->View);
     }
 
     /**

tests/TestCase/View/JsonViewTest.php

@@ -241,7 +241,7 @@ class JsonViewTest extends TestCase
         $Controller->set($data);
         $Controller->set('_serialize', $serialize);
         $Controller->set('_jsonOptions', $jsonOptions);
-        $Controller->viewBuilder()->className('Json');
+        $Controller->viewBuilder()->setClassName('Json');
         $View = $Controller->createView();
         $output = $View->render(false);
 
@@ -264,7 +264,7 @@ class JsonViewTest extends TestCase
             'tags' => ['cakephp', 'framework'],
             '_serialize' => 'tags'
         ]);
-        $Controller->viewBuilder()->className('Json');
+        $Controller->viewBuilder()->setClassName('Json');
         $View = $Controller->createView();
         $View->render();
 
@@ -288,7 +288,7 @@ class JsonViewTest extends TestCase
             '_serialize' => 'data',
             '_jsonp' => true
         ]);
-        $Controller->viewBuilder()->className('Json');
+        $Controller->viewBuilder()->setClassName('Json');
         $View = $Controller->createView();
         $output = $View->render(false);
 
@@ -330,7 +330,7 @@ class JsonViewTest extends TestCase
             ]
         ];
         $Controller->set('user', $data);
-        $Controller->viewBuilder()->className('Json');
+        $Controller->viewBuilder()->setClassName('Json');
         $View = $Controller->createView();
         $View->viewPath = $Controller->name;
         $output = $View->render('index');

tests/TestCase/View/StringTemplateTraitTest.php

@@ -42,6 +42,11 @@ class StringTemplateTraitTest extends TestCase
 {
 
     /**
+     * @var TestStringTemplate
+     */
+    public $Template;
+
+    /**
      * setUp method
      *
      * @return void
@@ -62,13 +67,13 @@ class StringTemplateTraitTest extends TestCase
         $templates = [
             'text' => '<p>{{text}}</p>',
         ];
-        $this->Template->templates($templates);
+        $this->Template->setTemplates($templates);
 
         $this->assertEquals(
             [
                 'text' => '<p>{{text}}</p>'
             ],
-            $this->Template->templates(),
+            $this->Template->getTemplates(),
             'newly added template should be included in template list'
         );
     }
@@ -89,7 +94,7 @@ class StringTemplateTraitTest extends TestCase
             [
                 'text' => '<p>{{text}}</p>'
             ],
-            $this->Template->templates(),
+            $this->Template->getTemplates(),
             'Configured templates should be included in template list'
         );
     }
@@ -104,7 +109,7 @@ class StringTemplateTraitTest extends TestCase
         $templates = [
             'text' => '<p>{{text}}</p>',
         ];
-        $this->Template->templates($templates);
+        $this->Template->setTemplates($templates);
         $result = $this->Template->formatTemplate('text', [
             'text' => 'CakePHP'
         ]);
@@ -124,7 +129,7 @@ class StringTemplateTraitTest extends TestCase
         $templates = [
             'text' => '<p>{{text}}</p>',
         ];
-        $this->Template->templates($templates);
+        $this->Template->setTemplates($templates);
         $result = $this->Template->templater();
         $this->assertInstanceOf('Cake\View\StringTemplate', $result);
     }

tests/TestCase/View/ViewBuilderTest.php

@@ -114,15 +114,15 @@ class ViewBuilderTest extends TestCase
         $events = $this->getMockBuilder('Cake\Event\EventManager')->getMock();
 
         $builder = new ViewBuilder();
-        $builder->name('Articles')
-            ->className('Ajax')
-            ->template('edit')
-            ->layout('default')
-            ->templatePath('Articles/')
-            ->helpers(['Form', 'Html'])
-            ->layoutPath('Admin/')
-            ->theme('TestTheme')
-            ->plugin('TestPlugin');
+        $builder->setName('Articles')
+            ->setClassName('Ajax')
+            ->setTemplate('edit')
+            ->setLayout('default')
+            ->setTemplatePath('Articles/')
+            ->setHelpers(['Form', 'Html'])
+            ->setLayoutPath('Admin/')
+            ->setTheme('TestTheme')
+            ->setPlugin('TestPlugin');
         $view = $builder->build(
             ['one' => 'value'],
             $request,
@@ -180,7 +180,7 @@ class ViewBuilderTest extends TestCase
     public function testBuildMissingViewClass()
     {
         $builder = new ViewBuilder();
-        $builder->className('Foo');
+        $builder->setClassName('Foo');
         $builder->build();
     }
 
@@ -194,10 +194,10 @@ class ViewBuilderTest extends TestCase
         $builder = new ViewBuilder();
 
         $builder
-            ->template('default')
-            ->layout('test')
-            ->helpers(['Html'])
-            ->className('JsonView');
+            ->setTemplate('default')
+            ->setLayout('test')
+            ->setHelpers(['Html'])
+            ->setClassName('JsonView');
 
         $result = json_decode(json_encode($builder), true);
 
@@ -223,19 +223,19 @@ class ViewBuilderTest extends TestCase
         $builder = new ViewBuilder();
 
         $builder
-            ->template('default')
-            ->layout('test')
-            ->helpers(['Html'])
-            ->className('JsonView');
+            ->setTemplate('default')
+            ->setLayout('test')
+            ->setHelpers(['Html'])
+            ->setClassName('JsonView');
 
         $result = json_encode($builder);
 
         $builder = new ViewBuilder();
         $builder->createFromArray(json_decode($result, true));
 
-        $this->assertEquals('default', $builder->template());
-        $this->assertEquals('test', $builder->layout());
-        $this->assertEquals(['Html'], $builder->helpers());
-        $this->assertEquals('JsonView', $builder->className());
+        $this->assertEquals('default', $builder->getTemplate());
+        $this->assertEquals('test', $builder->getLayout());
+        $this->assertEquals(['Html'], $builder->getHelpers());
+        $this->assertEquals('JsonView', $builder->getClassName());
     }
 }

tests/TestCase/View/ViewTest.php

@@ -280,6 +280,16 @@ class ViewTest extends TestCase
     public $fixtures = ['core.posts', 'core.users'];
 
     /**
+     * @var \Cake\View\View
+     */
+    public $View;
+
+    /**
+     * @var ViewPostsController
+     */
+    public $PostsController;
+
+    /**
      * setUp method
      *
      * @return void

tests/TestCase/View/ViewVarsTraitTest.php

@@ -23,6 +23,11 @@ class ViewVarsTraitTest extends TestCase
 {
 
     /**
+     * @var \Cake\Controller\Controller;
+     */
+    public $subject;
+
+    /**
      * setup
      *
      * @return void
@@ -79,7 +84,7 @@ class ViewVarsTraitTest extends TestCase
      *
      * @return void
      */
-    public function testSetTwoParamCombind()
+    public function testSetTwoParamCombined()
     {
         $keys = ['one', 'key'];
         $vals = ['two', 'val'];
@@ -183,7 +188,7 @@ class ViewVarsTraitTest extends TestCase
     {
         $this->subject->passedArgs = 'test';
         $this->subject->createView();
-        $result = $this->subject->viewbuilder()->options();
+        $result = $this->subject->viewBuilder()->getOptions();
         $this->assertEquals(['passedArgs' => 'test'], $result);
     }
 
@@ -196,7 +201,7 @@ class ViewVarsTraitTest extends TestCase
     {
         $this->subject->viewClass = 'Json';
         $view = $this->subject->createView();
-        $this->assertInstanceof('Cake\View\JsonView', $view);
+        $this->assertInstanceOf('Cake\View\JsonView', $view);
     }
 
     /**
@@ -206,10 +211,10 @@ class ViewVarsTraitTest extends TestCase
      */
     public function testCreateViewViewBuilder()
     {
-        $this->subject->viewBuilder()->className('Xml');
+        $this->subject->viewBuilder()->setClassName('Xml');
         $this->subject->viewClass = 'Json';
         $view = $this->subject->createView();
-        $this->assertInstanceof('Cake\View\XmlView', $view);
+        $this->assertInstanceOf('Cake\View\XmlView', $view);
     }
 
     /**
@@ -219,10 +224,10 @@ class ViewVarsTraitTest extends TestCase
      */
     public function testCreateViewParameter()
     {
-        $this->subject->viewBuilder()->className('View');
+        $this->subject->viewBuilder()->setClassName('View');
         $this->subject->viewClass = 'Json';
         $view = $this->subject->createView('Xml');
-        $this->assertInstanceof('Cake\View\XmlView', $view);
+        $this->assertInstanceOf('Cake\View\XmlView', $view);
     }
 
     /**