From 91cd57e55b034ec3be06c5db8fb2a60bfdf3c749 Mon Sep 17 00:00:00 2001 From: Vincent Petry Date: Tue, 20 Dec 2016 16:12:51 +0100 Subject: Get user home folder before deletion After the deletion getHome() will fail because the user doesn't exist any more, so we need to fetch that value earlier. Signed-off-by: Lukas Reschke --- tests/lib/User/UserTest.php | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) (limited to 'tests') diff --git a/tests/lib/User/UserTest.php b/tests/lib/User/UserTest.php index 734e86c187b..edb8ac4224e 100644 --- a/tests/lib/User/UserTest.php +++ b/tests/lib/User/UserTest.php @@ -200,6 +200,38 @@ class UserTest extends TestCase { $this->assertTrue($user->delete()); } + public function testDeleteWithDifferentHome() { + /** + * @var Backend | \PHPUnit_Framework_MockObject_MockObject $backend + */ + $backend = $this->createMock(Dummy::class); + + $backend->expects($this->at(0)) + ->method('implementsActions') + ->will($this->returnCallback(function ($actions) { + if ($actions === Backend::GET_HOME) { + return true; + } else { + return false; + } + })); + + // important: getHome MUST be called before deleteUser because + // once the user is deleted, getHome implementations might not + // return anything + $backend->expects($this->at(1)) + ->method('getHome') + ->with($this->equalTo('foo')) + ->will($this->returnValue('/home/foo')); + + $backend->expects($this->at(2)) + ->method('deleteUser') + ->with($this->equalTo('foo')); + + $user = new User('foo', $backend); + $this->assertTrue($user->delete()); + } + public function testGetHome() { /** * @var Backend | \PHPUnit_Framework_MockObject_MockObject $backend -- cgit v1.2.3