mirror of
https://github.com/BookStackApp/BookStack.git
synced 2024-10-01 01:36:00 -04:00
56d07f1909
- Cast send_invite value in cases where it might not have been a boolean, which occurs on non-JSON requests. - Added test to cover. - Updated API docs to mention and shown boolean usage.
327 lines
11 KiB
PHP
327 lines
11 KiB
PHP
<?php
|
|
|
|
namespace Tests\Api;
|
|
|
|
use BookStack\Access\Notifications\UserInviteNotification;
|
|
use BookStack\Activity\ActivityType;
|
|
use BookStack\Activity\Models\Activity as ActivityModel;
|
|
use BookStack\Entities\Models\Entity;
|
|
use BookStack\Facades\Activity;
|
|
use BookStack\Users\Models\Role;
|
|
use BookStack\Users\Models\User;
|
|
use Illuminate\Support\Facades\Hash;
|
|
use Illuminate\Support\Facades\Notification;
|
|
use Tests\TestCase;
|
|
|
|
class UsersApiTest extends TestCase
|
|
{
|
|
use TestsApi;
|
|
|
|
protected string $baseEndpoint = '/api/users';
|
|
|
|
protected array $endpointMap = [
|
|
['get', '/api/users'],
|
|
['post', '/api/users'],
|
|
['get', '/api/users/1'],
|
|
['put', '/api/users/1'],
|
|
['delete', '/api/users/1'],
|
|
];
|
|
|
|
public function test_users_manage_permission_needed_for_all_endpoints()
|
|
{
|
|
$this->actingAsApiEditor();
|
|
foreach ($this->endpointMap as [$method, $uri]) {
|
|
$resp = $this->json($method, $uri);
|
|
$resp->assertStatus(403);
|
|
$resp->assertJson($this->permissionErrorResponse());
|
|
}
|
|
}
|
|
|
|
public function test_no_endpoints_accessible_in_demo_mode()
|
|
{
|
|
config()->set('app.env', 'demo');
|
|
$this->actingAsApiAdmin();
|
|
|
|
foreach ($this->endpointMap as [$method, $uri]) {
|
|
$resp = $this->json($method, $uri);
|
|
$resp->assertStatus(403);
|
|
$resp->assertJson($this->permissionErrorResponse());
|
|
}
|
|
}
|
|
|
|
public function test_index_endpoint_returns_expected_user()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $firstUser */
|
|
$firstUser = User::query()->orderBy('id', 'asc')->first();
|
|
|
|
$resp = $this->getJson($this->baseEndpoint . '?count=1&sort=+id');
|
|
$resp->assertJson(['data' => [
|
|
[
|
|
'id' => $firstUser->id,
|
|
'name' => $firstUser->name,
|
|
'slug' => $firstUser->slug,
|
|
'email' => $firstUser->email,
|
|
'profile_url' => $firstUser->getProfileUrl(),
|
|
'edit_url' => $firstUser->getEditUrl(),
|
|
'avatar_url' => $firstUser->getAvatar(),
|
|
],
|
|
]]);
|
|
}
|
|
|
|
public function test_index_endpoint_has_correct_created_and_last_activity_dates()
|
|
{
|
|
$user = $this->users->editor();
|
|
$user->created_at = now()->subYear();
|
|
$user->save();
|
|
|
|
$this->actingAs($user);
|
|
Activity::add(ActivityType::AUTH_LOGIN, 'test login activity');
|
|
/** @var ActivityModel $activity */
|
|
$activity = ActivityModel::query()->where('user_id', '=', $user->id)->latest()->first();
|
|
|
|
$resp = $this->asAdmin()->getJson($this->baseEndpoint . '?filter[id]=3');
|
|
$resp->assertJson(['data' => [
|
|
[
|
|
'id' => $user->id,
|
|
'created_at' => $user->created_at->toJSON(),
|
|
'last_activity_at' => $activity->created_at->toJson(),
|
|
],
|
|
]]);
|
|
}
|
|
|
|
public function test_create_endpoint()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var Role $role */
|
|
$role = Role::query()->first();
|
|
|
|
$resp = $this->postJson($this->baseEndpoint, [
|
|
'name' => 'Benny Boris',
|
|
'email' => 'bboris@example.com',
|
|
'password' => 'mysuperpass',
|
|
'language' => 'it',
|
|
'roles' => [$role->id],
|
|
'send_invite' => false,
|
|
]);
|
|
|
|
$resp->assertStatus(200);
|
|
$resp->assertJson([
|
|
'name' => 'Benny Boris',
|
|
'email' => 'bboris@example.com',
|
|
'external_auth_id' => '',
|
|
'roles' => [
|
|
[
|
|
'id' => $role->id,
|
|
'display_name' => $role->display_name,
|
|
],
|
|
],
|
|
]);
|
|
$this->assertDatabaseHas('users', ['email' => 'bboris@example.com']);
|
|
|
|
/** @var User $user */
|
|
$user = User::query()->where('email', '=', 'bboris@example.com')->first();
|
|
$this->assertActivityExists(ActivityType::USER_CREATE, null, $user->logDescriptor());
|
|
$this->assertEquals(1, $user->roles()->count());
|
|
$this->assertEquals('it', setting()->getUser($user, 'language'));
|
|
}
|
|
|
|
public function test_create_with_send_invite()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
Notification::fake();
|
|
|
|
$resp = $this->postJson($this->baseEndpoint, [
|
|
'name' => 'Benny Boris',
|
|
'email' => 'bboris@example.com',
|
|
'send_invite' => true,
|
|
]);
|
|
|
|
$resp->assertStatus(200);
|
|
/** @var User $user */
|
|
$user = User::query()->where('email', '=', 'bboris@example.com')->first();
|
|
Notification::assertSentTo($user, UserInviteNotification::class);
|
|
}
|
|
|
|
public function test_create_with_send_invite_works_with_value_of_1()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
Notification::fake();
|
|
|
|
$resp = $this->postJson($this->baseEndpoint, [
|
|
'name' => 'Benny Boris',
|
|
'email' => 'bboris@example.com',
|
|
'send_invite' => '1', // Submissions via x-www-form-urlencoded/form-data may use 1 instead of boolean
|
|
]);
|
|
|
|
$resp->assertStatus(200);
|
|
/** @var User $user */
|
|
$user = User::query()->where('email', '=', 'bboris@example.com')->first();
|
|
Notification::assertSentTo($user, UserInviteNotification::class);
|
|
}
|
|
|
|
public function test_create_name_and_email_validation()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $existingUser */
|
|
$existingUser = User::query()->first();
|
|
|
|
$resp = $this->postJson($this->baseEndpoint, [
|
|
'email' => 'bboris@example.com',
|
|
]);
|
|
$resp->assertStatus(422);
|
|
$resp->assertJson($this->validationResponse(['name' => ['The name field is required.']]));
|
|
|
|
$resp = $this->postJson($this->baseEndpoint, [
|
|
'name' => 'Benny Boris',
|
|
]);
|
|
$resp->assertStatus(422);
|
|
$resp->assertJson($this->validationResponse(['email' => ['The email field is required.']]));
|
|
|
|
$resp = $this->postJson($this->baseEndpoint, [
|
|
'email' => $existingUser->email,
|
|
'name' => 'Benny Boris',
|
|
]);
|
|
$resp->assertStatus(422);
|
|
$resp->assertJson($this->validationResponse(['email' => ['The email has already been taken.']]));
|
|
}
|
|
|
|
public function test_read_endpoint()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $user */
|
|
$user = User::query()->first();
|
|
/** @var Role $userRole */
|
|
$userRole = $user->roles()->first();
|
|
|
|
$resp = $this->getJson($this->baseEndpoint . "/{$user->id}");
|
|
|
|
$resp->assertStatus(200);
|
|
$resp->assertJson([
|
|
'id' => $user->id,
|
|
'slug' => $user->slug,
|
|
'email' => $user->email,
|
|
'external_auth_id' => $user->external_auth_id,
|
|
'roles' => [
|
|
[
|
|
'id' => $userRole->id,
|
|
'display_name' => $userRole->display_name,
|
|
],
|
|
],
|
|
]);
|
|
}
|
|
|
|
public function test_update_endpoint()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $user */
|
|
$user = $this->users->admin();
|
|
$roles = Role::query()->pluck('id');
|
|
$resp = $this->putJson($this->baseEndpoint . "/{$user->id}", [
|
|
'name' => 'My updated user',
|
|
'email' => 'barrytest@example.com',
|
|
'roles' => $roles,
|
|
'external_auth_id' => 'btest',
|
|
'password' => 'barrytester',
|
|
'language' => 'fr',
|
|
]);
|
|
|
|
$resp->assertStatus(200);
|
|
$resp->assertJson([
|
|
'id' => $user->id,
|
|
'name' => 'My updated user',
|
|
'email' => 'barrytest@example.com',
|
|
'external_auth_id' => 'btest',
|
|
]);
|
|
$user->refresh();
|
|
$this->assertEquals('fr', setting()->getUser($user, 'language'));
|
|
$this->assertEquals(count($roles), $user->roles()->count());
|
|
$this->assertNotEquals('barrytester', $user->password);
|
|
$this->assertTrue(Hash::check('barrytester', $user->password));
|
|
}
|
|
|
|
public function test_update_endpoint_does_not_remove_info_if_not_provided()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $user */
|
|
$user = $this->users->admin();
|
|
$roleCount = $user->roles()->count();
|
|
$resp = $this->putJson($this->baseEndpoint . "/{$user->id}", []);
|
|
|
|
$resp->assertStatus(200);
|
|
$this->assertDatabaseHas('users', [
|
|
'id' => $user->id,
|
|
'name' => $user->name,
|
|
'email' => $user->email,
|
|
'password' => $user->password,
|
|
]);
|
|
$this->assertEquals($roleCount, $user->roles()->count());
|
|
}
|
|
|
|
public function test_delete_endpoint()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $user */
|
|
$user = User::query()->where('id', '!=', $this->users->admin()->id)
|
|
->whereNull('system_name')
|
|
->first();
|
|
|
|
$resp = $this->deleteJson($this->baseEndpoint . "/{$user->id}");
|
|
|
|
$resp->assertStatus(204);
|
|
$this->assertActivityExists('user_delete', null, $user->logDescriptor());
|
|
}
|
|
|
|
public function test_delete_endpoint_with_ownership_migration_user()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $user */
|
|
$user = User::query()->where('id', '!=', $this->users->admin()->id)
|
|
->whereNull('system_name')
|
|
->first();
|
|
$entityChain = $this->entities->createChainBelongingToUser($user);
|
|
/** @var User $newOwner */
|
|
$newOwner = User::query()->where('id', '!=', $user->id)->first();
|
|
|
|
/** @var Entity $entity */
|
|
foreach ($entityChain as $entity) {
|
|
$this->assertEquals($user->id, $entity->owned_by);
|
|
}
|
|
|
|
$resp = $this->deleteJson($this->baseEndpoint . "/{$user->id}", [
|
|
'migrate_ownership_id' => $newOwner->id,
|
|
]);
|
|
|
|
$resp->assertStatus(204);
|
|
/** @var Entity $entity */
|
|
foreach ($entityChain as $entity) {
|
|
$this->assertEquals($newOwner->id, $entity->refresh()->owned_by);
|
|
}
|
|
}
|
|
|
|
public function test_delete_endpoint_fails_deleting_only_admin()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
$adminRole = Role::getSystemRole('admin');
|
|
$adminToDelete = $adminRole->users()->first();
|
|
$adminRole->users()->where('id', '!=', $adminToDelete->id)->delete();
|
|
|
|
$resp = $this->deleteJson($this->baseEndpoint . "/{$adminToDelete->id}");
|
|
|
|
$resp->assertStatus(500);
|
|
$resp->assertJson($this->errorResponse('You cannot delete the only admin', 500));
|
|
}
|
|
|
|
public function test_delete_endpoint_fails_deleting_public_user()
|
|
{
|
|
$this->actingAsApiAdmin();
|
|
/** @var User $publicUser */
|
|
$publicUser = User::query()->where('system_name', '=', 'public')->first();
|
|
|
|
$resp = $this->deleteJson($this->baseEndpoint . "/{$publicUser->id}");
|
|
|
|
$resp->assertStatus(500);
|
|
$resp->assertJson($this->errorResponse('You cannot delete the guest user', 500));
|
|
}
|
|
}
|