Mark tests that sleep as @Medium (#94)

* Fix expiration tests

Setting a +1 second expiration is very unreliable.
A +1 second expiration doesn't mean the value will expire
"in a second", but "next second". If we're already at
950ms within our second, then by the time "set" and the
follow-up "get" go out, we may already be within the
next second. As a result, the item will already have been
expired and we would no longer be able to find the value.

Better to be safe and set +2 seconds (which will give us
anywhere between 1 and 2 seconds)

Meanwhile also reduced the amount of sleeps: we could
manage with only 1 per function.

* Mark tests that sleep with `@medium`

* Trigger travis

Co-authored-by: Matthias Mullie <git@mullie.eu>
Co-authored-by: Tobias Nyholm <tobias.nyholm@gmail.com>

Author: 3 people

src/CachePoolTest.php

@@ -482,6 +482,9 @@ public function testCommit()
         $this->assertTrue($return, 'commit() should return true even if no items were deferred. ');
     }
 
+    /**
+     * @medium
+     */
     public function testExpiration()
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
@@ -848,6 +851,9 @@ public function testSavingObject()
         $this->assertInstanceOf('DateTime', $value, 'You must be able to store objects in cache.');
     }
 
+    /**
+     * @medium
+     */
     public function testHasItemReturnsFalseWhenDeferredItemIsExpired()
     {
         if (isset($this->skippedTests[__FUNCTION__])) {

src/SimpleCacheTest.php

@@ -165,21 +165,25 @@ public function testSet()
         $this->assertEquals('value', $this->cache->get('key'));
     }
 
+    /**
+     * @medium
+     */
     public function testSetTtl()
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
             $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
         }
 
-        $result = $this->cache->set('key1', 'value', 1);
+        $result = $this->cache->set('key1', 'value', 2);
         $this->assertTrue($result, 'set() must return true if success');
         $this->assertEquals('value', $this->cache->get('key1'));
-        $this->advanceTime(2);
-        $this->assertNull($this->cache->get('key1'), 'Value must expire after ttl.');
 
-        $this->cache->set('key2', 'value', new \DateInterval('PT1S'));
+        $this->cache->set('key2', 'value', new \DateInterval('PT2S'));
         $this->assertEquals('value', $this->cache->get('key2'));
-        $this->advanceTime(2);
+
+        $this->advanceTime(3);
+
+        $this->assertNull($this->cache->get('key1'), 'Value must expire after ttl.');
         $this->assertNull($this->cache->get('key2'), 'Value must expire after ttl.');
     }
 
@@ -259,22 +263,25 @@ public function testSetMultipleWithIntegerArrayKey()
         $this->assertEquals('value0', $this->cache->get('0'));
     }
 
+    /**
+     * @medium
+     */
     public function testSetMultipleTtl()
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
             $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
         }
 
-        $this->cache->setMultiple(['key2' => 'value2', 'key3' => 'value3'], 1);
+        $this->cache->setMultiple(['key2' => 'value2', 'key3' => 'value3'], 2);
         $this->assertEquals('value2', $this->cache->get('key2'));
         $this->assertEquals('value3', $this->cache->get('key3'));
-        $this->advanceTime(2);
-        $this->assertNull($this->cache->get('key2'), 'Value must expire after ttl.');
-        $this->assertNull($this->cache->get('key3'), 'Value must expire after ttl.');
 
-        $this->cache->setMultiple(['key4' => 'value4'], new \DateInterval('PT1S'));
+        $this->cache->setMultiple(['key4' => 'value4'], new \DateInterval('PT2S'));
         $this->assertEquals('value4', $this->cache->get('key4'));
-        $this->advanceTime(2);
+
+        $this->advanceTime(3);
+        $this->assertNull($this->cache->get('key2'), 'Value must expire after ttl.');
+        $this->assertNull($this->cache->get('key3'), 'Value must expire after ttl.');
         $this->assertNull($this->cache->get('key4'), 'Value must expire after ttl.');
     }