X-Git-Url: http://source.bookstackapp.com/bookstack/blobdiff_plain/07b889547d28e68e5fc8f923c166bd607da17ad7..refs/pull/2515/head:/tests/Entity/PageRevisionTest.php diff --git a/tests/Entity/PageRevisionTest.php b/tests/Entity/PageRevisionTest.php index 521ea79a4..68a8f01a9 100644 --- a/tests/Entity/PageRevisionTest.php +++ b/tests/Entity/PageRevisionTest.php @@ -1,6 +1,6 @@ -updatePage($page, $page->book_id, ['name' => 'updated page', 'html' => '

new content

', 'summary' => 'page revision testing']); + $pageRepo->update($page, ['name' => 'updated page', 'html' => '

new content

', 'summary' => 'page revision testing']); $pageRevision = $page->revisions->last(); $revisionView = $this->get($page->getUrl() . '/revisions/' . $pageRevision->id); @@ -24,14 +24,29 @@ class PageRevisionTest extends TestCase $revisionView->assertSee('new content'); } + public function test_page_revision_preview_shows_content_of_revision() + { + $this->asEditor(); + + $pageRepo = app(PageRepo::class); + $page = Page::first(); + $pageRepo->update($page, ['name' => 'updated page', 'html' => '

new revision content

', 'summary' => 'page revision testing']); + $pageRevision = $page->revisions->last(); + $pageRepo->update($page, ['name' => 'updated page', 'html' => '

Updated content

', 'summary' => 'page revision testing 2']); + + $revisionView = $this->get($page->getUrl() . '/revisions/' . $pageRevision->id); + $revisionView->assertStatus(200); + $revisionView->assertSee('new revision content'); + } + public function test_page_revision_restore_updates_content() { $this->asEditor(); $pageRepo = app(PageRepo::class); $page = Page::first(); - $pageRepo->updatePage($page, $page->book_id, ['name' => 'updated page abc123', 'html' => '

new contente def456

', 'summary' => 'initial page revision testing']); - $pageRepo->updatePage($page, $page->book_id, ['name' => 'updated page again', 'html' => '

new content

', 'summary' => 'page revision testing']); + $pageRepo->update($page, ['name' => 'updated page abc123', 'html' => '

new contente def456

', 'summary' => 'initial page revision testing']); + $pageRepo->update($page, ['name' => 'updated page again', 'html' => '

new content

', 'summary' => 'page revision testing']); $page = Page::find($page->id); @@ -51,6 +66,28 @@ class PageRevisionTest extends TestCase $pageView->assertSee('def456'); } + public function test_page_revision_restore_sets_new_revision_with_summary() + { + $this->asEditor(); + + $pageRepo = app(PageRepo::class); + $page = Page::first(); + $pageRepo->update($page, ['name' => 'updated page abc123', 'html' => '

new contente def456

', 'summary' => 'My first update']); + $pageRepo->update($page, ['name' => 'updated page again', 'html' => '

new content

', 'summary' => '']); + $page->refresh(); + + $revToRestore = $page->revisions()->where('name', 'like', '%abc123')->first(); + $this->put($page->getUrl() . '/revisions/' . $revToRestore->id . '/restore'); + $page->refresh(); + + $this->assertDatabaseHas('page_revisions', [ + 'page_id' => $page->id, + 'text' => 'new contente def456', + 'type' => 'version', + 'summary' => "Restored from #{$revToRestore->id}; My first update", + ]); + } + public function test_page_revision_count_increments_on_update() { $page = Page::first(); @@ -87,7 +124,7 @@ class PageRevisionTest extends TestCase // Delete the first revision $revision = $page->revisions->get(1); $resp = $this->asEditor()->delete($revision->getUrl('/delete/')); - $resp->assertStatus(200); + $resp->assertRedirect($page->getUrl('/revisions')); $page = Page::find($page->id); $afterRevisionCount = $page->revisions->count(); @@ -98,7 +135,7 @@ class PageRevisionTest extends TestCase $beforeRevisionCount = $page->revisions->count(); $currentRevision = $page->getCurrentRevision(); $resp = $this->asEditor()->delete($currentRevision->getUrl('/delete/')); - $resp->assertStatus(400); + $resp->assertRedirect($page->getUrl('/revisions')); $page = Page::find($page->id); $afterRevisionCount = $page->revisions->count();