diff --git a/app/controllers/api/users.php b/app/controllers/api/users.php index dce493b024..651efbfbeb 100644 --- a/app/controllers/api/users.php +++ b/app/controllers/api/users.php @@ -414,7 +414,7 @@ App::get('/v1/users/:userId') $user = $dbForProject->getDocument('users', $userId); if ($user->isEmpty()) { - throw new Exception('User not found', 404, Exception::USER_NOT_FOUND); + throw new Exception(Exception::USER_NOT_FOUND); } $response->dynamic($user, Response::MODEL_USER); diff --git a/tests/e2e/Services/Users/UsersBase.php b/tests/e2e/Services/Users/UsersBase.php index ac3b116b32..f5e364cb68 100644 --- a/tests/e2e/Services/Users/UsersBase.php +++ b/tests/e2e/Services/Users/UsersBase.php @@ -644,6 +644,19 @@ trait UsersBase $this->assertIsInt($users['body']['total']); $this->assertGreaterThan(0, $users['body']['total']); + /** + * Test for FAILURE + */ + $user = $this->client->call(Client::METHOD_GET, '/users/non_existent', array_merge([ + 'content-type' => 'application/json', + 'x-appwrite-project' => $this->getProject()['$id'], + ], $this->getHeaders())); + + $this->assertEquals($user['headers']['status-code'], 404); + $this->assertEquals($user['body']['code'], 404); + $this->assertEquals($user['body']['message'], 'User with the requested ID could not be found.'); + $this->assertEquals($user['body']['type'], 'user_not_found'); + return $data; }