diff --git a/core/modules/book/tests/src/Functional/BookContentModerationTest.php b/core/modules/book/tests/src/Functional/BookContentModerationTest.php
index 6838297..5b5fe34 100644
--- a/core/modules/book/tests/src/Functional/BookContentModerationTest.php
+++ b/core/modules/book/tests/src/Functional/BookContentModerationTest.php
@@ -2,6 +2,7 @@
namespace Drupal\Tests\book\Functional;
+use Drupal\Tests\BrowserTestBase;
use Drupal\workflows\Entity\Workflow;
/**
@@ -9,7 +10,9 @@
*
* @group book
*/
-class BookContentModerationTest extends BookTest {
+class BookContentModerationTest extends BrowserTestBase {
+
+ use BookTestTrait;
/**
* Modules to install.
@@ -24,6 +27,9 @@ class BookContentModerationTest extends BookTest {
protected function setUp() {
parent::setUp();
+ $this->drupalPlaceBlock('system_breadcrumb_block');
+ $this->drupalPlaceBlock('page_title_block');
+
$workflow = Workflow::load('editorial');
$workflow->getTypePlugin()->addEntityTypeAndBundle('node', 'book');
$workflow->save();
@@ -37,10 +43,10 @@ protected function setUp() {
*/
public function testBookWithForwardRevisions() {
// Create two books.
- $book_1_nodes = $this->createBook();
+ $book_1_nodes = $this->createBook(t('Save and Publish'));
$book_1 = $this->book;
- $this->createBook();
+ $this->createBook(t('Save and Publish'));
$book_2 = $this->book;
$this->drupalLogin($this->bookAuthor);
@@ -74,15 +80,4 @@ public function testBookWithForwardRevisions() {
}
- /**
- * {@inheritdoc}
- */
- public function createBookNode($book_nid, $parent = NULL, $submit = NULL) {
- // This test has Content Moderation enabled so we have a different value for
- // the submit button.
- $submit = t('Save and Publish');
-
- return parent::createBookNode($book_nid, $parent, $submit);
- }
-
}
diff --git a/core/modules/book/tests/src/Functional/BookTest.php b/core/modules/book/tests/src/Functional/BookTest.php
index d7771f0..df19242 100644
--- a/core/modules/book/tests/src/Functional/BookTest.php
+++ b/core/modules/book/tests/src/Functional/BookTest.php
@@ -2,9 +2,7 @@
namespace Drupal\Tests\book\Functional;
-use Drupal\Component\Utility\SafeMarkup;
use Drupal\Core\Cache\Cache;
-use Drupal\Core\Entity\EntityInterface;
use Drupal\Tests\BrowserTestBase;
use Drupal\user\RoleInterface;
@@ -15,6 +13,8 @@
*/
class BookTest extends BrowserTestBase {
+ use BookTestTrait;
+
/**
* Modules to install.
*
@@ -76,39 +76,6 @@ protected function setUp() {
}
/**
- * Creates a new book with a page hierarchy.
- *
- * @return \Drupal\node\NodeInterface[]
- */
- public function createBook() {
- // Create new book.
- $this->drupalLogin($this->bookAuthor);
-
- $this->book = $this->createBookNode('new');
- $book = $this->book;
-
- /*
- * Add page hierarchy to book.
- * Book
- * |- Node 0
- * |- Node 1
- * |- Node 2
- * |- Node 3
- * |- Node 4
- */
- $nodes = [];
- $nodes[] = $this->createBookNode($book->id()); // Node 0.
- $nodes[] = $this->createBookNode($book->id(), $nodes[0]->book['nid']); // Node 1.
- $nodes[] = $this->createBookNode($book->id(), $nodes[0]->book['nid']); // Node 2.
- $nodes[] = $this->createBookNode($book->id()); // Node 3.
- $nodes[] = $this->createBookNode($book->id()); // Node 4.
-
- $this->drupalLogout();
-
- return $nodes;
- }
-
- /**
* Tests the book navigation cache context.
*
* @see \Drupal\book\Cache\BookNavigationCacheContext
@@ -230,152 +197,6 @@ public function testBook() {
}
/**
- * Checks the outline of sub-pages; previous, up, and next.
- *
- * Also checks the printer friendly version of the outline.
- *
- * @param \Drupal\Core\Entity\EntityInterface $node
- * Node to check.
- * @param $nodes
- * Nodes that should be in outline.
- * @param $previous
- * (optional) Previous link node. Defaults to FALSE.
- * @param $up
- * (optional) Up link node. Defaults to FALSE.
- * @param $next
- * (optional) Next link node. Defaults to FALSE.
- * @param array $breadcrumb
- * The nodes that should be displayed in the breadcrumb.
- */
- public function checkBookNode(EntityInterface $node, $nodes, $previous = FALSE, $up = FALSE, $next = FALSE, array $breadcrumb) {
- // $number does not use drupal_static as it should not be reset
- // since it uniquely identifies each call to checkBookNode().
- static $number = 0;
- $this->drupalGet('node/' . $node->id());
-
- // Check outline structure.
- if ($nodes !== NULL) {
- $this->assertPattern($this->generateOutlinePattern($nodes), format_string('Node @number outline confirmed.', ['@number' => $number]));
- }
- else {
- $this->pass(format_string('Node %number does not have outline.', ['%number' => $number]));
- }
-
- // Check previous, up, and next links.
- if ($previous) {
- /** @var \Drupal\Core\Url $url */
- $url = $previous->urlInfo();
- $url->setOptions(['attributes' => ['rel' => ['prev'], 'title' => t('Go to previous page')]]);
- $text = SafeMarkup::format('‹ @label', ['@label' => $previous->label()]);
- $this->assertRaw(\Drupal::l($text, $url), 'Previous page link found.');
- }
-
- if ($up) {
- /** @var \Drupal\Core\Url $url */
- $url = $up->urlInfo();
- $url->setOptions(['attributes' => ['title' => t('Go to parent page')]]);
- $this->assertRaw(\Drupal::l('Up', $url), 'Up page link found.');
- }
-
- if ($next) {
- /** @var \Drupal\Core\Url $url */
- $url = $next->urlInfo();
- $url->setOptions(['attributes' => ['rel' => ['next'], 'title' => t('Go to next page')]]);
- $text = SafeMarkup::format('@label ›', ['@label' => $next->label()]);
- $this->assertRaw(\Drupal::l($text, $url), 'Next page link found.');
- }
-
- // Compute the expected breadcrumb.
- $expected_breadcrumb = [];
- $expected_breadcrumb[] = \Drupal::url('');
- foreach ($breadcrumb as $a_node) {
- $expected_breadcrumb[] = $a_node->url();
- }
-
- // Fetch links in the current breadcrumb.
- $links = $this->xpath('//nav[@class="breadcrumb"]/ol/li/a');
- $got_breadcrumb = [];
- foreach ($links as $link) {
- $got_breadcrumb[] = $link->getAttribute('href');
- }
-
- // Compare expected and got breadcrumbs.
- $this->assertIdentical($expected_breadcrumb, $got_breadcrumb, 'The breadcrumb is correctly displayed on the page.');
-
- // Check printer friendly version.
- $this->drupalGet('book/export/html/' . $node->id());
- $this->assertText($node->label(), 'Printer friendly title found.');
- $this->assertRaw($node->body->processed, 'Printer friendly body found.');
-
- $number++;
- }
-
- /**
- * Creates a regular expression to check for the sub-nodes in the outline.
- *
- * @param array $nodes
- * An array of nodes to check in outline.
- *
- * @return string
- * A regular expression that locates sub-nodes of the outline.
- */
- public function generateOutlinePattern($nodes) {
- $outline = '';
- foreach ($nodes as $node) {
- $outline .= '(node\/' . $node->id() . ')(.*?)(' . $node->label() . ')(.*?)';
- }
-
- return '/