Skip to content

Commit

Permalink
Removed old book sort permission test
Browse files Browse the repository at this point in the history
Permission handling now done via other means with more extensive
permissions testing in SortTest class.
  • Loading branch information
ssddanbrown committed Jan 5, 2022
1 parent fbd388b commit 2312d07
Showing 1 changed file with 0 additions and 45 deletions.
45 changes: 0 additions & 45 deletions tests/Permissions/EntityPermissionsTest.php
Expand Up @@ -670,51 +670,6 @@ public function test_book_sort_view_permission()
$this->actingAs($this->user)->get($firstBook->getUrl('/sort'));
}

public function test_book_sort_permission()
{
/** @var Book $firstBook */
$firstBook = Book::query()->first();
/** @var Book $secondBook */
$secondBook = Book::query()->find(2);

$this->setRestrictionsForTestRoles($firstBook, ['view', 'update']);
$this->setRestrictionsForTestRoles($secondBook, ['view']);

$firstBookChapter = $this->newChapter(['name' => 'first book chapter'], $firstBook);
$secondBookChapter = $this->newChapter(['name' => 'second book chapter'], $secondBook);

// Create request data
$reqData = [
[
'id' => $firstBookChapter->id,
'sort' => 0,
'parentChapter' => false,
'type' => 'chapter',
'book' => $secondBook->id,
],
];

// Move chapter from first book to a second book
$this->actingAs($this->user)->put($firstBook->getUrl() . '/sort', ['sort-tree' => json_encode($reqData)])
->assertRedirect('/');
$this->get('/')->assertSee('You do not have permission');

$reqData = [
[
'id' => $secondBookChapter->id,
'sort' => 0,
'parentChapter' => false,
'type' => 'chapter',
'book' => $firstBook->id,
],
];

// Move chapter from second book to first book
$this->actingAs($this->user)->put($firstBook->getUrl() . '/sort', ['sort-tree' => json_encode($reqData)])
->assertRedirect('/');
$this->get('/')->assertSee('You do not have permission');
}

public function test_can_create_page_if_chapter_has_permissions_when_book_not_visible()
{
/** @var Book $book */
Expand Down

0 comments on commit 2312d07

Please sign in to comment.