mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #19221 from owncloud/improved_error_messages 

decrypt-all: improved error message if user doesn't exists
This commit is contained in:
Thomas Müller 2015-09-21 16:06:43 +02:00
parent b23a5e6886 d2e90b6050
commit d8901cf7cd
2 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/private/encryption/decryptall.php
View File

@ -80,6 +80,11 @@ class DecryptAll {
$this->input = $input;
$this->output = $output;
if ($user !== '' && $this->userManager->userExists($user) === false) {
$this->output->writeln('User "' . $user . '" does not exist. Please check the username and try again');
return false;
}
$this->output->writeln('prepare encryption modules...');
if ($this->prepareEncryptionModules($user) === false) {
return false;

15
tests/lib/encryption/decryptalltest.php
View File

@ -80,11 +80,13 @@ class DecryptAllTest extends TestCase {
/**
* @dataProvider dataTrueFalse
* @param bool $prepareResult
*/
public function testDecryptAll($prepareResult) {
$user = 'user1';
$this->userManager->expects($this->once())->method('userExists')->willReturn(true);
/** @var DecryptAll | \PHPUnit_Framework_MockObject_MockObject | $instance */
$instance = $this->getMockBuilder('OC\Encryption\DecryptAll')
->setConstructorArgs(
@ -120,6 +122,19 @@ class DecryptAllTest extends TestCase {
];
}
/**
* test decrypt all call with a user who doesn't exists
*/
public function testDecryptAllWrongUser() {
$this->userManager->expects($this->once())->method('userExists')->willReturn(false);
$this->outputInterface->expects($this->once())->method('writeln')
->with('User "user1" does not exist. Please check the username and try again');
$this->assertFalse(
$this->instance->decryptAll($this->inputInterface, $this->outputInterface, 'user1')
);
}
/**
* @dataProvider dataTrueFalse
*/