Fix unit test for expiration date without date but with default
The expiration date was explicitly given, so it was not testing what it should. Moreover, "link_defaultExpDays" needs to be mapped when testing the default value (even if it will just fallback to the value of "shareapi_expire_after_n_days"). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
This commit is contained in:
parent
2e0352182b
commit
1aed476937
|
@ -937,19 +937,17 @@ class ManagerTest extends \Test\TestCase {
|
|||
}
|
||||
|
||||
public function testValidateExpirationDateNoDateDefault() {
|
||||
$future = new \DateTime();
|
||||
$future->add(new \DateInterval('P3D'));
|
||||
$future->setTime(0,0,0);
|
||||
|
||||
$expected = clone $future;
|
||||
|
||||
$share = $this->manager->newShare();
|
||||
$share->setExpirationDate($future);
|
||||
|
||||
$expected = new \DateTime();
|
||||
$expected->add(new \DateInterval('P3D'));
|
||||
$expected->setTime(0,0,0);
|
||||
|
||||
$this->config->method('getAppValue')
|
||||
->willReturnMap([
|
||||
['core', 'shareapi_default_expire_date', 'no', 'yes'],
|
||||
['core', 'shareapi_expire_after_n_days', '7', '3'],
|
||||
['core', 'link_defaultExpDays', 3, '3'],
|
||||
]);
|
||||
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
|
|
Loading…
Reference in New Issue