diff --git a/core/modules/tracker/src/Tests/TrackerWithoutCommentTest.php b/core/modules/tracker/src/Tests/TrackerWithoutCommentTest.php
new file mode 100644
index 0000000..dfc5283
--- /dev/null
+++ b/core/modules/tracker/src/Tests/TrackerWithoutCommentTest.php
@@ -0,0 +1,296 @@
+drupalCreateContentType(['type' => 'page', 'name' => 'Basic page']);
+
+ $permissions = ['create page content'];
+ $this->user = $this->drupalCreateUser($permissions);
+ $this->otherUser = $this->drupalCreateUser($permissions);
+ user_role_grant_permissions(AccountInterface::ANONYMOUS_ROLE, [
+ 'access content',
+ 'access user profiles',
+ ]);
+ $this->drupalPlaceBlock('local_tasks_block', ['id' => 'page_tabs_block']);
+ $this->drupalPlaceBlock('local_actions_block', ['id' => 'page_actions_block']);
+ }
+
+ /**
+ * Tests for the presence of nodes on the global tracker listing.
+ */
+ public function testTrackerAll() {
+ $this->drupalLogin($this->user);
+
+ $unpublished = $this->drupalCreateNode([
+ 'title' => $this->randomMachineName(8),
+ 'status' => 0,
+ ]);
+ $published = $this->drupalCreateNode([
+ 'title' => $this->randomMachineName(8),
+ 'status' => 1,
+ ]);
+
+ $this->drupalGet('activity');
+ $this->assertNoText($unpublished->label(), 'Unpublished node does not show up in the tracker listing.');
+ $this->assertText($published->label(), 'Published node shows up in the tracker listing.');
+ $this->assertLink(t('My recent content'), 0, 'User tab shows up on the global tracker page.');
+
+ // Assert cache contexts, specifically the pager and node access contexts.
+ $this->assertCacheContexts(['languages:language_interface', 'route', 'theme', 'url.query_args:' . MainContentViewSubscriber::WRAPPER_FORMAT, 'url.query_args.pagers:0', 'user.node_grants:view', 'user']);
+ // Assert cache tags for the action/tabs blocks, visible node, and node list
+ // cache tag.
+ $expected_tags = Cache::mergeTags($published->getCacheTags(), $published->getOwner()->getCacheTags());
+ // Because the 'user.permissions' cache context is being optimized away.
+ $role_tags = [];
+ foreach ($this->user->getRoles() as $rid) {
+ $role_tags[] = "config:user.role.$rid";
+ }
+ $expected_tags = Cache::mergeTags($expected_tags, $role_tags);
+ $block_tags = [
+ 'block_view',
+ 'config:block.block.page_actions_block',
+ 'config:block.block.page_tabs_block',
+ 'config:block_list',
+ ];
+ $expected_tags = Cache::mergeTags($expected_tags, $block_tags);
+ $additional_tags = [
+ 'node_list',
+ 'rendered',
+ ];
+ $expected_tags = Cache::mergeTags($expected_tags, $additional_tags);
+ $this->assertCacheTags($expected_tags);
+
+ // Delete a node and ensure it no longer appears on the tracker.
+ $published->delete();
+ $this->drupalGet('activity');
+ $this->assertNoText($published->label(), 'Deleted node does not show up in the tracker listing.');
+ }
+
+ /**
+ * Tests for the presence of nodes on a user's tracker listing.
+ */
+ public function testTrackerUser() {
+ $this->drupalLogin($this->user);
+
+ $unpublished = $this->drupalCreateNode([
+ 'title' => $this->randomMachineName(8),
+ 'uid' => $this->user->id(),
+ 'status' => 0,
+ ]);
+ $my_published = $this->drupalCreateNode([
+ 'title' => $this->randomMachineName(8),
+ 'uid' => $this->user->id(),
+ 'status' => 1,
+ ]);
+
+ $this->drupalGet('user/' . $this->user->id() . '/activity');
+ $this->assertNoText($unpublished->label(), "Unpublished nodes do not show up in the user's tracker listing.");
+ $this->assertText($my_published->label(), "Published nodes show up in the user's tracker listing.");
+
+ // Assert cache contexts.
+ $this->assertCacheContexts(['languages:language_interface', 'route', 'theme', 'url.query_args:' . MainContentViewSubscriber::WRAPPER_FORMAT, 'url.query_args.pagers:0', 'user', 'user.node_grants:view']);
+ // Assert cache tags for the visible nodes (including owners) and node list
+ // cache tag.
+ $expected_tags = Cache::mergeTags($my_published->getCacheTags(), $my_published->getOwner()->getCacheTags());
+ // Because the 'user.permissions' cache context is being optimized away.
+ $role_tags = [];
+ foreach ($this->user->getRoles() as $rid) {
+ $role_tags[] = "config:user.role.$rid";
+ }
+ $expected_tags = Cache::mergeTags($expected_tags, $role_tags);
+ $block_tags = [
+ 'block_view',
+ 'config:block.block.page_actions_block',
+ 'config:block.block.page_tabs_block',
+ 'config:block_list',
+ ];
+ $expected_tags = Cache::mergeTags($expected_tags, $block_tags);
+ $additional_tags = [
+ 'node_list',
+ 'rendered',
+ ];
+ $expected_tags = Cache::mergeTags($expected_tags, $additional_tags);
+
+ $this->assertCacheTags($expected_tags);
+ $this->assertCacheContexts(['languages:language_interface', 'route', 'theme', 'url.query_args:' . MainContentViewSubscriber::WRAPPER_FORMAT, 'url.query_args.pagers:0', 'user', 'user.node_grants:view']);
+
+ $this->assertLink($my_published->label());
+ $this->assertNoLink($unpublished->label());
+ // Verify that title and tab title have been set correctly.
+ $this->assertText('Activity', 'The user activity tab has the name "Activity".');
+ $this->assertTitle(t('@name | @site', ['@name' => $this->user->getUsername(), '@site' => $this->config('system.site')->get('name')]), 'The user tracker page has the correct page title.');
+
+ // Test escaping of title on user's tracker tab.
+ \Drupal::service('module_installer')->install(['user_hooks_test']);
+ Cache::invalidateTags(['rendered']);
+ \Drupal::state()->set('user_hooks_test_user_format_name_alter', TRUE);
+ $this->drupalGet('user/' . $this->user->id() . '/activity');
+ $this->assertEscaped('' . $this->user->id() . '');
+
+ \Drupal::state()->set('user_hooks_test_user_format_name_alter_safe', TRUE);
+ Cache::invalidateTags(['rendered']);
+ $this->drupalGet('user/' . $this->user->id() . '/activity');
+ $this->assertNoEscaped('' . $this->user->id() . '');
+ $this->assertRaw('' . $this->user->id() . '');
+ }
+
+ /**
+ * Tests the metadata for the "new"/"updated" indicators.
+ */
+ public function testTrackerHistoryMetadata() {
+ $this->drupalLogin($this->user);
+
+ // Create a page node.
+ $edit = [
+ 'title' => $this->randomMachineName(8),
+ ];
+ $node = $this->drupalCreateNode($edit);
+
+ // Verify that the history metadata is present.
+ $this->drupalGet('activity');
+ $this->assertHistoryMetadata($node->id(), $node->getChangedTime());
+ $this->drupalGet('activity/' . $this->user->id());
+ $this->assertHistoryMetadata($node->id(), $node->getChangedTime());
+ $this->drupalGet('user/' . $this->user->id() . '/activity');
+ $this->assertHistoryMetadata($node->id(), $node->getChangedTime());
+ }
+
+ /**
+ * Tests that existing nodes are indexed by cron.
+ */
+ public function testTrackerCronIndexing() {
+ $this->drupalLogin($this->user);
+
+ // Create 3 nodes.
+ $edits = [];
+ $nodes = [];
+ for ($i = 1; $i <= 3; $i++) {
+ $edits[$i] = [
+ 'title' => $this->randomMachineName(),
+ ];
+ $nodes[$i] = $this->drupalCreateNode($edits[$i]);
+ }
+
+ // Start indexing backwards from node 3.
+ \Drupal::state()->set('tracker.index_nid', 3);
+
+ // Clear the current tracker tables and rebuild them.
+ db_delete('tracker_node')
+ ->execute();
+ db_delete('tracker_user')
+ ->execute();
+ tracker_cron();
+
+ $this->drupalLogin($this->user);
+
+ // Fetch the user's tracker.
+ $this->drupalGet('activity/' . $this->user->id());
+
+ // Assert that all node titles are displayed.
+ foreach ($nodes as $i => $node) {
+ $this->assertText($node->label(), format_string('Node @i is displayed on the tracker listing pages.', ['@i' => $i]));
+ }
+
+ // Fetch the site-wide tracker.
+ $this->drupalGet('activity');
+
+ // Assert that all node titles are displayed.
+ foreach ($nodes as $i => $node) {
+ $this->assertText($node->label(), format_string('Node @i is displayed on the tracker listing pages.', ['@i' => $i]));
+ }
+ }
+
+ /**
+ * Tests that publish/unpublish works at admin/content/node.
+ */
+ public function testTrackerAdminUnpublish() {
+ \Drupal::service('module_installer')->install(['views']);
+ \Drupal::service('router.builder')->rebuild();
+ $admin_user = $this->drupalCreateUser(['access content overview', 'administer nodes', 'bypass node access']);
+ $this->drupalLogin($admin_user);
+
+ $node = $this->drupalCreateNode([
+ 'title' => $this->randomMachineName(),
+ ]);
+
+ // Assert that the node is displayed.
+ $this->drupalGet('activity');
+ $this->assertText($node->label(), 'A node is displayed on the tracker listing pages.');
+
+ // Unpublish the node and ensure that it's no longer displayed.
+ $edit = [
+ 'action' => 'node_unpublish_action',
+ 'node_bulk_form[0]' => $node->id(),
+ ];
+ $this->drupalPostForm('admin/content', $edit, t('Apply to selected items'));
+
+ $this->drupalGet('activity');
+ $this->assertText(t('No content available.'), 'A node is displayed on the tracker listing pages.');
+ }
+
+ /**
+ * Passes if the appropriate history metadata exists.
+ *
+ * Verify the data-history-node-id, data-history-node-timestamp attribute,
+ * which is used by the drupal.tracker-history library to add the appropriate
+ * "new" and "updated" indicators, as well as the "x new" replies link to the
+ * tracker. We do this in JavaScript to prevent breaking the render cache.
+ *
+ * @param int $node_id
+ * A node ID, that must exist as a data-history-node-id attribute
+ * @param int $node_timestamp
+ * A node timestamp, that must exist as a data-history-node-timestamp
+ * attribute.
+ * @param bool $library_is_present
+ * Whether the drupal.tracker-history library should be present or not.
+ */
+ public function assertHistoryMetadata($node_id, $node_timestamp, $library_is_present = TRUE) {
+ $settings = $this->getDrupalSettings();
+ $this->assertIdentical($library_is_present, isset($settings['ajaxPageState']) && in_array('tracker/history', explode(',', $settings['ajaxPageState']['libraries'])), 'drupal.tracker-history library is present.');
+ $this->assertIdentical(1, count($this->xpath('//table/tbody/tr/td[@data-history-node-id="' . $node_id . '" and @data-history-node-timestamp="' . $node_timestamp . '"]')), 'Tracker table cell contains the data-history-node-id and data-history-node-timestamp attributes for the node.');
+ }
+
+}
diff --git a/core/modules/tracker/tests/src/Functional/TrackerNodeAccessTest.php b/core/modules/tracker/tests/src/Functional/TrackerNodeAccessTest.php
index 2b320ae..4dd32df 100644
--- a/core/modules/tracker/tests/src/Functional/TrackerNodeAccessTest.php
+++ b/core/modules/tracker/tests/src/Functional/TrackerNodeAccessTest.php
@@ -4,6 +4,7 @@
use Drupal\comment\Plugin\Field\FieldType\CommentItemInterface;
use Drupal\comment\Tests\CommentTestTrait;
+use Drupal\field\Entity\FieldStorageConfig;
use Drupal\node\Entity\NodeType;
use Drupal\Tests\BrowserTestBase;
@@ -61,6 +62,8 @@ public function testTrackerNodeAccess() {
$this->drupalGet('user/' . $access_user->id() . '/activity');
$this->assertText($private_node->getTitle(), 'Private node is visible to user with private access.');
$this->assertText($public_node->getTitle(), 'Public node is visible to user with private access.');
+ // Comments column displayed.
+ $this->assertSession()->pageTextContains('Comments');
// User without access should not see private node.
$this->drupalLogin($no_access_user);
@@ -70,6 +73,23 @@ public function testTrackerNodeAccess() {
$this->drupalGet('user/' . $access_user->id() . '/activity');
$this->assertNoText($private_node->getTitle(), 'Private node is not visible to user without private access.');
$this->assertText($public_node->getTitle(), 'Public node is visible to user without private access.');
+
+ // Ensure module pages works when comment module uninstalled.
+ $field = FieldStorageConfig::loadByName('node', 'comment');
+ $field->delete();
+ field_purge_batch(10);
+ /** @var \Drupal\Core\Extension\ModuleInstallerInterface $service */
+ $service = \Drupal::service('module_installer');
+ $service->uninstall(['comment']);
+ $this->drupalLogin($access_user);
+ $this->drupalGet('activity');
+ $this->assertSession()->pageTextContains($private_node->getTitle());
+ // Comments column hidden.
+ $this->assertSession()->pageTextNotContains('Comments');
+ $this->drupalGet('user/' . $access_user->id() . '/activity');
+ $this->assertSession()->pageTextContains($private_node->getTitle());
+ // Comments column hidden.
+ $this->assertSession()->pageTextNotContains('Comments');
}
}