Remove extra spaces
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
This commit is contained in:
parent
7cfa2701b9
commit
72f51422ec
|
@ -772,7 +772,6 @@ class ManagerTest extends \Test\TestCase {
|
|||
$this->expectException(\OCP\Share\Exceptions\GenericShareException::class);
|
||||
$this->expectExceptionMessage('Expiration date is in the past');
|
||||
|
||||
|
||||
// Expire date in the past
|
||||
$past = new \DateTime();
|
||||
$past->sub(new \DateInterval('P1D'));
|
||||
|
@ -783,7 +782,6 @@ class ManagerTest extends \Test\TestCase {
|
|||
self::invokePrivate($this->manager, 'validateExpirationDate', [$share]);
|
||||
}
|
||||
|
||||
|
||||
public function testValidateExpirationDateEnforceButNotSet() {
|
||||
$this->expectException(\InvalidArgumentException::class);
|
||||
$this->expectExceptionMessage('Expiration date is enforced');
|
||||
|
@ -878,7 +876,7 @@ class ManagerTest extends \Test\TestCase {
|
|||
]);
|
||||
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
$hookListener->expects($this->once())->method('listener')->with($this->callback(function ($data) use ($future) {
|
||||
return $data['expirationDate'] == $future;
|
||||
}));
|
||||
|
@ -899,7 +897,7 @@ class ManagerTest extends \Test\TestCase {
|
|||
$share->setExpirationDate($date);
|
||||
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
$hookListener->expects($this->once())->method('listener')->with($this->callback(function ($data) use ($expected) {
|
||||
return $data['expirationDate'] == $expected && $data['passwordSet'] === false;
|
||||
}));
|
||||
|
@ -911,7 +909,7 @@ class ManagerTest extends \Test\TestCase {
|
|||
|
||||
public function testValidateExpirationDateNoDateNoDefault() {
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
$hookListener->expects($this->once())->method('listener')->with($this->callback(function ($data) {
|
||||
return $data['expirationDate'] === null && $data['passwordSet'] === true;
|
||||
}));
|
||||
|
@ -941,7 +939,7 @@ class ManagerTest extends \Test\TestCase {
|
|||
]);
|
||||
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
$hookListener->expects($this->once())->method('listener')->with($this->callback(function ($data) use ($expected) {
|
||||
return $data['expirationDate'] == $expected;
|
||||
}));
|
||||
|
@ -959,7 +957,7 @@ class ManagerTest extends \Test\TestCase {
|
|||
$save = clone $nextWeek;
|
||||
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
$hookListener->expects($this->once())->method('listener')->willReturnCallback(function ($data) {
|
||||
$data['expirationDate']->sub(new \DateInterval('P2D'));
|
||||
});
|
||||
|
@ -973,7 +971,6 @@ class ManagerTest extends \Test\TestCase {
|
|||
$this->assertEquals($save, $share->getExpirationDate());
|
||||
}
|
||||
|
||||
|
||||
public function testValidateExpirationDateHookException() {
|
||||
$this->expectException(\Exception::class);
|
||||
$this->expectExceptionMessage('Invalid date!');
|
||||
|
@ -986,7 +983,7 @@ class ManagerTest extends \Test\TestCase {
|
|||
$share->setExpirationDate($nextWeek);
|
||||
|
||||
$hookListener = $this->getMockBuilder('Dummy')->setMethods(['listener'])->getMock();
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
\OCP\Util::connectHook('\OC\Share', 'verifyExpirationDate', $hookListener, 'listener');
|
||||
$hookListener->expects($this->once())->method('listener')->willReturnCallback(function ($data) {
|
||||
$data['accepted'] = false;
|
||||
$data['message'] = 'Invalid date!';
|
||||
|
@ -1011,7 +1008,6 @@ class ManagerTest extends \Test\TestCase {
|
|||
$this->assertEquals(null, $share->getExpirationDate());
|
||||
}
|
||||
|
||||
|
||||
public function testUserCreateChecksShareWithGroupMembersOnlyDifferentGroups() {
|
||||
$this->expectException(\Exception::class);
|
||||
$this->expectExceptionMessage('Sharing is only allowed with group members');
|
||||
|
|
Loading…
Reference in New Issue