8000 add (pdo|chain) cache (adapter|simple) prune method · symfony/symfony@d89caf3 · GitHub
[go: up one dir, main page]
Skip to content

Commit d89caf3

Browse files
robfrawleyrobfrawley
committed
add (pdo|chain) cache (adapter|simple) prune method
1 parent 7695112 commit d89caf3

File tree

15 files changed

+292
-8
lines changed

15 files changed

+292
-8
lines changed

‎src/Symfony/Component/Cache/Adapter/ChainAdapter.php‎

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
use Psr\Cache\CacheItemPoolInterface;
1616
use Symfony\Component\Cache\CacheItem;
1717
use Symfony\Component\Cache\Exception\InvalidArgumentException;
18+
use Symfony\Component\Cache\PruneableInterface;
1819

1920
/**
2021
* Chains several adapters together.
@@ -24,7 +25,7 @@
2425
*
2526
* @author Kévin Dunglas <dunglas@gmail.com>
2627
*/
27-
class ChainAdapter implements AdapterInterface
28+
class ChainAdapter implements AdapterInterface, PruneableInterface
2829
{
2930
private $adapters = array();
3031
private $adapterCount;
@@ -231,4 +232,20 @@ public function commit()
231232

232233
return $committed;
233234
}
235+
236+
/**
237+
* {@inheritdoc}
238+
*/
239+
public function prune()
240+
{
241+
$pruned = true;
242+
243+
foreach ($this->adapters as $adapter) {
244+
if ($adapter instanceof PruneableInterface) {
245+
$pruned = $adapter->prune() && $pruned;
246+
}
247+
}
248+
249+
return $pruned;
250+
}
234251
}

‎src/Symfony/Component/Cache/Adapter/PdoAdapter.php‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,10 @@
1111

1212
namespace Symfony\Component\Cache\Adapter;
1313

14+
use Symfony\Component\Cache\PruneableInterface;
1415
use Symfony\Component\Cache\Traits\PdoTrait;
1516

16-
class PdoAdapter extends AbstractAdapter
17+
class PdoAdapter extends AbstractAdapter implements PruneableInterface
1718
{
1819
use PdoTrait;
1920

‎src/Symfony/Component/Cache/CHANGELOG.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@ CHANGELOG
55
-----
66

77
* added PruneableInterface so PSR-6 or PSR-16 cache implementations can declare support for manual stale cache pruning
8-
* added FilesystemTrait::prune() and PhpFilesTrait::prune() implementations
9-
* now FilesystemAdapter, PhpFilesAdapter, FilesystemCache, and PhpFilesCache implement PruneableInterface and support
10-
manual stale cache pruning
8+
* added prune logic to FilesystemTrait, PhpFilesTrait, PdoTrait, and ChainTrait
9+
* now FilesystemAdapter, PhpFilesAdapter, FilesystemCache, PhpFilesCache, PdoAdapter, PdoCache, ChainAdapter, and
10+
ChainCache implement PruneableInterface and support manual stale cache pruning
1111

1212
3.3.0
1313
-----

‎src/Symfony/Component/Cache/Simple/ChainCache.php‎

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
use Psr\SimpleCache\CacheInterface;
1515
use Symfony\Component\Cache\Exception\InvalidArgumentException;
16+
use Symfony\Component\Cache\PruneableInterface;
1617

1718
/**
1819
* Chains several caches together.
@@ -22,7 +23,7 @@
2223
*
2324
* @author Nicolas Grekas <p@tchwork.com>
2425
*/
25-
class ChainCache implements CacheInterface
26+
class ChainCache implements CacheInterface, PruneableInterface
2627
{
2728
private $miss;
2829
private $caches = array();
@@ -219,4 +220,20 @@ public function setMultiple($values, $ttl = null)
219220

220221
return $saved;
221222
}
223+
224+
/**
225+
* {@inheritdoc}
226+
*/
227+
public function prune()
228+
{
229+
$pruned = true;
230+
231+
foreach ($this->caches as $cache) {
232+
if ($cache instanceof PruneableInterface) {
233+
$pruned = $cache->prune() && $pruned;
234+
}
235+
}
236+
237+
return $pruned;
238+
}
222239
}

‎src/Symfony/Component/Cache/Simple/PdoCache.php‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,10 @@
1111

1212
namespace Symfony\Component\Cache\Simple;
1313

14+
use Symfony\Component\Cache\PruneableInterface;
1415
use Symfony\Component\Cache\Traits\PdoTrait;
1516

16-
class PdoCache extends AbstractCache
17+
class PdoCache extends AbstractCache implements PruneableInterface
1718
{
1819
use PdoTrait;
1920

‎src/Symfony/Component/Cache/Tests/Adapter/AdapterTestCase.php‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

1414
use Cache\IntegrationTests\CachePoolTest;
15+
use Psr\Cache\CacheItemPoolInterface;
1516
use Symfony\Component\Cache\PruneableInterface;
1617

1718
abstract class AdapterTestCase extends CachePoolTest
@@ -83,6 +84,7 @@ public function testPrune()
8384
$this->fail('Test classes for pruneable caches must implement `isPruned($cache, $name)` method.');
8485
}
8586

87+
/** @var PruneableInterface|CacheItemPoolInterface $cache */
8688
$cache = $this->createCachePool();
8789

8890
$doSet = function ($name, $value, \DateInterval $expiresAfter = null) use ($cache) {
@@ -96,6 +98,18 @@ public function testPrune()
9698
$cache->save($item);
9799
};
98100

101+
$doSet('foo', 'foo-val', new \DateInterval('PT05S'));
102+
$doSet('bar', 'bar-val', new \DateInterval('PT10S'));
103+
$doSet('baz', 'baz-val', new \DateInterval('PT15S'));
104+
$doSet('qux', 'qux-val', new \DateInterval('PT20S'));
105+
106+
sleep(30);
107+
$cache->prune();
108+
$this->assertTrue($this->isPruned($cache, 'foo'));
109+
$this->assertTrue($this->isPruned($cache, 'bar'));
110+
$this->assertTrue($this->isPruned($cache, 'baz'));
111+
$this->assertTrue($this->isPruned($cache, 'qux'));
112+
99113
$doSet('foo', 'foo-val');
100114
$doSet('bar', 'bar-val', new \DateInterval('PT20S'));
101115
$doSet('baz', 'baz-val', new \DateInterval('PT40S'));

‎src/Symfony/Component/Cache/Tests/Adapter/ChainAdapterTest.php‎

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,11 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14+
use Symfony\Component\Cache\Adapter\AdapterInterface;
1415
use Symfony\Component\Cache\Adapter\FilesystemAdapter;
1516
use Symfony\Component\Cache\Adapter\ArrayAdapter;
1617
use Symfony\Component\Cache\Adapter\ChainAdapter;
18+
use Symfony\Component\Cache\PruneableInterface;
1719
use Symfony\Component\Cache\Tests\Fixtures\ExternalAdapter;
1820

1921
/**
@@ -44,4 +46,73 @@ public function testInvalidAdapterException()
4446
{
4547
new ChainAdapter(array(new \stdClass()));
4648
}
49+
50+
public function testPrune()
51+
{
52+
if (isset($this->skippedTests[__FUNCTION__])) {
53+
$this->markTestSkipped($this->skippedTests[__FUNCTION__]);
54+
}
55+
56+
$cache = new ChainAdapter(array(
57+
$this->getPruneableMock(),
58+
$this->getNonPruneableMock(),
59+
$this->getPruneableMock(),
60+
));
61+
$this->assertTrue($cache->prune());
62+
63+
$cache = new ChainAdapter(array(
64+
$this->getPruneableMock(),
65+
$this->getFailingPruneableMock(),
66+
$this->getPruneableMock(),
67+
));
68+
$this->assertFalse($cache->prune());
69+
}
70+
71+
/**
72+
* @return \PHPUnit_Framework_MockObject_MockObject|PruneableCacheInterface
73+
*/
74+
private function getPruneableMock()
75+
{
76+
$pruneable = $this
77+
->getMockBuilder(PruneableCacheInterface::class)
78+
->getMock();
79+
80+
$pruneable
81+
->expects($this->atLeastOnce())
82+
->method('prune')
83+
->will($this->returnValue(true));
84+
85+
return $pruneable;
86+
}
87+
88+
/**
89+
* @return \PHPUnit_Framework_MockObject_MockObject|PruneableCacheInterface
90+
*/
91+
private function getFailingPruneableMock()
92+
{
93+
$pruneable = $this
94+
->getMockBuilder(PruneableCacheInterface::class)
95+
->getMock();
96+
97+
$pruneable
98+
->expects($this->atLeastOnce())
99+
->method('prune')
100+
->will($this->returnValue(false));
101+
102+
return $pruneable;
103+
}
104+
105+
/**
106+
* @return \PHPUnit_Framework_MockObject_MockObject|AdapterInterface
107+
*/
108+
private function getNonPruneableMock()
109+
{
110+
return $this
111+
->getMockBuilder(AdapterInterface::class)
112+
->getMock();
113+
}
114+
}
115+
116+
interface PruneableCacheInterface extends PruneableInterface, AdapterInterface
117+
{
47118
}

‎src/Symfony/Component/Cache/Tests/Adapter/PdoAdapterTest.php‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,15 @@
1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

1414
use Symfony\Component\Cache\Adapter\PdoAdapter;
15+
use Symfony\Component\Cache\Tests\Traits\PdoPruneableTrait;
1516

1617
/**
1718
* @group time-sensitive
1819
*/
1920
class PdoAdapterTest extends AdapterTestCase
2021
{
22+
use PdoPruneableTrait;
23+
2124
protected static $dbFile;
2225

2326
public static function setupBeforeClass()

‎src/Symfony/Component/Cache/Tests/Adapter/PdoDbalAdapterTest.php‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,15 @@
1313

1414
use Doctrine\DBAL\DriverManager;
1515
use Symfony\Component\Cache\Adapter\PdoAdapter;
16+
use Symfony\Component\Cache\Tests\Traits\PdoPruneableTrait;
1617

1718
/**
1819
* @group time-sensitive
1920
*/
2021
class PdoDbalAdapterTest extends AdapterTestCase
2122
{
23+
use PdoPruneableTrait;
24+
2225
protected static $dbFile;
2326

2427
public static function setupBeforeClass()

‎src/Symfony/Component/Cache/Tests/Simple/CacheTestCase.php‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
namespace Symfony\Component\Cache\Tests\Simple;
1313

1414
use Cache\IntegrationTests\SimpleCacheTest;
15+
use Psr\SimpleCache\CacheInterface;
1516
use Symfony\Component\Cache\PruneableInterface;
1617

1718
abstract class CacheTestCase extends SimpleCacheTest
@@ -80,8 +81,21 @@ public function testPrune()
8081
$this->fail('Test classes for pruneable caches must implement `isPruned($cache, $name)` method.');
8182
}
8283

84+
/** @var PruneableInterface|CacheInterface $cache */
8385
$cache = $this->createSimpleCache();
8486

87+
$cache->set('foo', 'foo-val', new \DateInterval('PT05S'));
88+
$cache->set('bar', 'bar-val', new \DateInterval('PT10S'));
89+
$cache->set('baz', 'baz-val', new \DateInterval('PT15S'));
90+
$cache->set('qux', 'qux-val', new \DateInterval('PT20S'));
91+
92+
sleep(30);
93+
$cache->prune();
94+
$this->assertTrue($this->isPruned($cache, 'foo'));
95+
$this->assertTrue($this->isPruned($cache, 'bar'));
96+
$this->assertTrue($this->isPruned($cache, 'baz'));
97+
$this->assertTrue($this->isPruned($cache, 'qux'));
98+
8599
$cache->set('foo', 'foo-val');
86100
$cache->set('bar', 'bar-val', new \DateInterval('PT20S'));
87101
$cache->set('baz', 'baz-val', < 3950 span class=pl-k>new \DateInterval('PT40S'));

0 commit comments

Comments
 (0)
0