Added Attributes to Renderer, updated PHPUnit version

Author: geggleto

composer.json

@@ -19,7 +19,7 @@
         }
     },
   "require-dev": {
-    "phpunit/phpunit": "^4.0",
+    "phpunit/phpunit": "^5.0",
     "slim/slim" : "^3.0"
   }
 }

src/PhpRenderer.php

@@ -8,6 +8,7 @@
  */
 namespace Slim\Views;
 
+use \InvalidArgumentException;
 use Psr\Http\Message\ResponseInterface;
 
 /**
@@ -23,14 +24,21 @@ class PhpRenderer
      */
     protected $templatePath;
 
+    /**
+     * @var array
+     */
+    protected $attributes;
+
     /**
      * SlimRenderer constructor.
      *
      * @param string $templatePath
+     * @param array $attributes
      */
-    public function __construct($templatePath = "")
+    public function __construct($templatePath = "", $attributes = [])
     {
         $this->templatePath = $templatePath;
+        $this->attributes = $attributes;
     }
 
     /**
@@ -58,6 +66,46 @@ public function render(ResponseInterface $response, $template, array $data = [])
         return $response;
     }
 
+    /**
+     * Get the attributes for the renderer
+     *
+     * @return array
+     */
+    public function getAttributes()
+    {
+        return $this->attributes;
+    }
+
+    /**
+     * Set the attributes for the renderer
+     *
+     * @param array $attributes
+     */
+    public function setAttributes(array $attributes)
+    {
+        $this->attributes = $attributes;
+    }
+
+    /**
+     * Get the template path
+     *
+     * @return string
+     */
+    public function getTemplatePath()
+    {
+        return $this->templatePath;
+    }
+
+    /**
+     * Set the template path
+     *
+     * @param string $templatePath
+     */
+    public function setTemplatePath($templatePath)
+    {
+        $this->templatePath = $templatePath;
+    }
+
     /**
      * Renders a template and returns the result as a string
      *
@@ -82,6 +130,12 @@ public function fetch($template, array $data = []) {
             throw new \RuntimeException("View cannot render `$template` because the template does not exist");
         }
 
+        foreach ($data as $k=>$val) {
+            if (in_array($k, array_keys($this->attributes))) {
+                throw new \InvalidArgumentException("Duplicate key found in data and renderer attributes. " . $k);
+            }
+        }
+
         ob_start();
         $this->protectedIncludeScope($this->templatePath . $template, $data);
         $output = ob_get_clean();

tests/PhpRendererTest.php

@@ -9,7 +9,7 @@
  * Date: 2015-11-12
  * Time: 1:19 PM
  */
-class PhpRendererTest extends \PHPUnit_Framework_TestCase
+class PhpRendererTest extends PHPUnit_Framework_TestCase
 {
 
     public function testRenderer() {
@@ -25,4 +25,52 @@ public function testRenderer() {
 
         $this->assertEquals("Hi", $newResponse->getBody()->getContents());
     }
+
+    /**
+     * @expectedException InvalidArgumentException
+     */
+    public function testExceptionForAttr() {
+
+        $renderer = new \Slim\Views\PhpRenderer("tests/", [
+            "hello" => "Hi"
+        ]);
+
+        $headers = new Headers();
+        $body = new Body(fopen('php://temp', 'r+'));
+        $response = new Response(200, $headers, $body);
+
+        $renderer->render($response, "testTemplate.php", [
+            "hello" => "Hi"
+        ]);
+    }
+
+    /**
+     * @expectedException InvalidArgumentException
+     */
+    public function testExceptionForTemplateInData() {
+
+        $renderer = new \Slim\Views\PhpRenderer("tests/");
+
+        $headers = new Headers();
+        $body = new Body(fopen('php://temp', 'r+'));
+        $response = new Response(200, $headers, $body);
+
+        $renderer->render($response, "testTemplate.php", [
+            "template" => "Hi"
+        ]);
+    }
+
+    /**
+     * @expectedException RuntimeException
+     */
+    public function testTemplateNotFound() {
+
+        $renderer = new \Slim\Views\PhpRenderer("tests/");
+
+        $headers = new Headers();
+        $body = new Body(fopen('php://temp', 'r+'));
+        $response = new Response(200, $headers, $body);
+
+        $renderer->render($response, "adfadftestTemplate.php", []);
+    }
 }