Fix that two DirectoryResources with different patterns would be deduplicated
ResourceInterface::__toString is mainly important because in various places, array_uniqe() is called to perform a de-duplication of resources and will use the string representation for objects. Thus, we need to take care that if DirectoryResources apply different patterns they must be kept after array_unique calls.
This commit is contained in:
parent
cfd8cc24ff
commit
2b36ac5a61
@ -38,7 +38,7 @@ class DirectoryResource implements ResourceInterface, \Serializable
|
|||||||
*/
|
*/
|
||||||
public function __toString()
|
public function __toString()
|
||||||
{
|
{
|
||||||
return (string) $this->resource;
|
return md5(serialize(array($this->resource, $this->pattern)));
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -54,7 +54,6 @@ class DirectoryResourceTest extends \PHPUnit_Framework_TestCase
|
|||||||
{
|
{
|
||||||
$resource = new DirectoryResource($this->directory);
|
$resource = new DirectoryResource($this->directory);
|
||||||
$this->assertSame($this->directory, $resource->getResource(), '->getResource() returns the path to the resource');
|
$this->assertSame($this->directory, $resource->getResource(), '->getResource() returns the path to the resource');
|
||||||
$this->assertSame($this->directory, (string) $resource, '->__toString() returns the path to the resource');
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testGetPattern()
|
public function testGetPattern()
|
||||||
@ -87,6 +86,13 @@ class DirectoryResourceTest extends \PHPUnit_Framework_TestCase
|
|||||||
$this->assertFalse($resource->isFresh(time() + 10), '->isFresh() returns false if a new file is added');
|
$this->assertFalse($resource->isFresh(time() + 10), '->isFresh() returns false if a new file is added');
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function testIsFreshNewFileWithDifferentPattern()
|
||||||
|
{
|
||||||
|
$resource = new DirectoryResource($this->directory, '/.xml$/');
|
||||||
|
touch($this->directory.'/new.yaml', time() + 20);
|
||||||
|
$this->assertTrue($resource->isFresh(time() + 10), '->isFresh() returns true if a new file with a non-matching pattern is added');
|
||||||
|
}
|
||||||
|
|
||||||
public function testIsFreshDeleteFile()
|
public function testIsFreshDeleteFile()
|
||||||
{
|
{
|
||||||
$resource = new DirectoryResource($this->directory);
|
$resource = new DirectoryResource($this->directory);
|
||||||
@ -149,4 +155,12 @@ class DirectoryResourceTest extends \PHPUnit_Framework_TestCase
|
|||||||
$this->assertSame($this->directory, $resource->getResource());
|
$this->assertSame($this->directory, $resource->getResource());
|
||||||
$this->assertSame('/\.(foo|xml)$/', $resource->getPattern());
|
$this->assertSame('/\.(foo|xml)$/', $resource->getPattern());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function testResourcesWithDifferentPatternsAreDifferent()
|
||||||
|
{
|
||||||
|
$resourceA = new DirectoryResource($this->directory, '/.xml$/');
|
||||||
|
$resourceB = new DirectoryResource($this->directory, '/.yaml$/');
|
||||||
|
|
||||||
|
$this->assertEquals(2, count(array_unique(array($resourceA, $resourceB))));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user