Missing use for NodeInterface

1. +++ b/src/Tests/GoogleAnalyticsCustomDimensionsAndMetricsTest.php
   @@ -43,6 +58,11 @@ class GoogleAnalyticsCustomDimensionsAndMetricsTest extends WebTestBase {
   +    $node = $this->drupalCreateNode([
   +      'type' => $this->nodeType,
   +    ]);
   +    $node_id = $node->id();
   +    $node_title = $node->getTitle();
   

   This is the only place where you us $this->nodeType, everything else is hardcoded. I'd just hardcode this as well.

   I don't think $node_id and $node_title are used?

2. +++ b/src/Tests/GoogleAnalyticsCustomDimensionsAndMetricsTest.php
   @@ -105,6 +129,10 @@ class GoogleAnalyticsCustomDimensionsAndMetricsTest extends WebTestBase {
   +    $this->assertText($node_title);
   +    $this->assertRaw('ga("set", ' . Json::encode('dimension5') . ', ' . Json::encode('article') . ');', '[testGoogleAnalyticsCustomDimensionsAndMetrics]: Article is shown.');
   

   Something like "Node tokens are shown" might be more useful as an assertion message?

   While we're adding this, might be useful to add an assertRaw() on the frontpage or whatever page we visit above, and make sure that there is no ga('set' for dimension 5 at all, to make sure that we don't print out an empty value.

No, they aren't. Looks good now.

I don't think we should be testing that here. That part needs work (at least for comments/concepts.. content_translation in 8.x has no translation sets or multiple nodes.. a different language could have a different URL, though), so it seems like that should be it's own issue to improve those things too.

Why would you want to block additional test coverage on more.. let's get this committed and then we can write more.