From e0184317347e93b6643a51d1a40579e355b5314b Mon Sep 17 00:00:00 2001
From: Joao Gilberto Magalhaes <joao@byjg.com.br>
Date: Sun, 11 Mar 2018 00:27:27 -0300
Subject: [PATCH] Fix Import and Comments on Tests

---
 tests/CachePSR16Test.php | 26 +++++++++++++++++---------
 tests/CachePSR6Test.php  | 13 +++++++++----
 2 files changed, 26 insertions(+), 13 deletions(-)

diff --git a/tests/CachePSR16Test.php b/tests/CachePSR16Test.php
index a28b5f1..e6ff0c7 100644
--- a/tests/CachePSR16Test.php
+++ b/tests/CachePSR16Test.php
@@ -2,6 +2,7 @@
 
 namespace Test;
 
+use ByJG\Cache\Psr16\BaseCacheEngine;
 use ByJG\Cache\Psr16\NoCacheEngine;
 
 require_once 'BaseCacheTest.php';
@@ -11,8 +12,9 @@ class CachePSR16Test extends BaseCacheTest
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testGetOneItem(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testGetOneItem(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -27,7 +29,7 @@ public function testGetOneItem(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
             $cacheEngine->set('chave', 'valor');
 
             // Get Object
-            if (!($cacheEngine instanceof \ByJG\Cache\Psr16\NoCacheEngine)) {
+            if (!($cacheEngine instanceof NoCacheEngine)) {
                 $item2 = $cacheEngine->get('chave', 'default');
                 $this->assertEquals('valor', $item2);
             }
@@ -46,8 +48,10 @@ public function testGetOneItem(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \ByJG\Cache\InvalidArgumentException
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testGetMultipleItems(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testGetMultipleItems(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -65,7 +69,7 @@ public function testGetMultipleItems(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEng
             $cacheEngine->set('chave2', 'valor2');
 
             // Get Object
-            if (!($cacheEngine instanceof \ByJG\Cache\Psr16\NoCacheEngine)) {
+            if (!($cacheEngine instanceof NoCacheEngine)) {
                 $item2 = $cacheEngine->getMultiple(['chave1', 'chave2']);
                 $this->assertEquals('valor1', $item2['chave1']);
                 $this->assertEquals('valor2', $item2['chave2']);
@@ -86,8 +90,9 @@ public function testGetMultipleItems(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEng
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testTtl(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testTtl(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -105,7 +110,7 @@ public function testTtl(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
             $cacheEngine->set('chave2', 'valor2', 2);
 
             // Get Object
-            if (!($cacheEngine instanceof \ByJG\Cache\Psr16\NoCacheEngine)) {
+            if (!($cacheEngine instanceof NoCacheEngine)) {
                 $item2 = $cacheEngine->get('chave');
                 $this->assertEquals('valor', $item2);
                 $this->assertTrue($cacheEngine->has('chave2'));
@@ -122,8 +127,9 @@ public function testTtl(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testCacheObject(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testCacheObject(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -137,7 +143,7 @@ public function testCacheObject(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
             $cacheEngine->set('chave', $model);
 
             // Get Object
-            if (!($cacheEngine instanceof \ByJG\Cache\Psr16\NoCacheEngine)) {
+            if (!($cacheEngine instanceof NoCacheEngine)) {
                 $item2 = $cacheEngine->get('chave');
                 $this->assertEquals($model, $item2);
             }
@@ -154,8 +160,10 @@ public function testCacheObject(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \ByJG\Cache\InvalidArgumentException
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testClear(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testClear(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
diff --git a/tests/CachePSR6Test.php b/tests/CachePSR6Test.php
index aa956ab..787b28b 100644
--- a/tests/CachePSR6Test.php
+++ b/tests/CachePSR6Test.php
@@ -2,6 +2,7 @@
 
 namespace Test;
 
+use ByJG\Cache\Psr16\BaseCacheEngine;
 use ByJG\Cache\Psr6\CachePool;
 
 require_once 'BaseCacheTest.php';
@@ -11,8 +12,9 @@ class CachePSR6Test extends BaseCacheTest
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testGetOneItem(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testGetOneItem(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -46,8 +48,9 @@ public function testGetOneItem(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testGetMultipleItems(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testGetMultipleItems(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -89,8 +92,9 @@ public function testGetMultipleItems(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEng
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testTtl(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testTtl(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;
 
@@ -129,8 +133,9 @@ public function testTtl(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
     /**
      * @dataProvider CachePoolProvider
      * @param \ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine
+     * @throws \Psr\SimpleCache\InvalidArgumentException
      */
-    public function testCacheObject(\ByJG\Cache\Psr16\BaseCacheEngine $cacheEngine)
+    public function testCacheObject(BaseCacheEngine $cacheEngine)
     {
         $this->cacheEngine = $cacheEngine;