improved error message if user doesn't exists

This commit is contained in:
Bjoern Schiessle 2015-09-21 13:13:38 +02:00
parent 6006a03ef1
commit d2e90b6050
2 changed files with 20 additions and 0 deletions

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;

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
*/