Decouple NodeDisplayConfigurableTest from RDF module

Thanks for picking this up.

1. +++ b/core/modules/node/tests/src/Functional/NodeDisplayConfigurableTest.php
   @@ -90,15 +90,12 @@ public function testDisplayConfigurable(string $theme, string $metadata_region,
   -    $assert->elementExists('css', 'div[rel="schema:author"]');
   ...
   -    $assert->elementNotExists('css', 'div[rel="schema:author"]');
   

   Not sure we should just remove these, we need to find another way of identifying the div.

2. +++ b/core/modules/node/tests/src/Functional/NodeDisplayConfigurableTest.php
   @@ -148,12 +145,14 @@ protected function assertNodeHtml(NodeInterface $node, UserInterface $user, bool
   +      $assert->elementTextContains('css', "$uid_selector", $user->getAccountName());
   

   Should this also check $html_element still?

Thanks for the swift response.

#4.1 Agree after looking again we don't need these, they aren't testing anything useful except the RDF output which is now gone.

#4.2 The number of elements is the same, the issue is that it's difficult to find the correct one now. The Stark output looks like this:

<article role="article">
  <div>
    <div>Sat, 08/13/2022 - 07:58</div>
    <div><p>oSIlpyA8gKsXFoOqdGMdRa76OCaXyD49</p></div>
    <div>
      <div>Authored by</div>
      <div>k9INAms0KBHjCd</div>
    </div>
  </div>
</article>

The RDF output conveniently tagged the author name, but now it's pretty difficult to find except by counting divs - and that will make the test more fragile for future changes IMO, so I think it's fine just to assert the existence of the output.

Two more minor review comments:

1. +++ b/core/modules/node/tests/src/Functional/NodeDisplayConfigurableTest.php
   @@ -148,12 +145,14 @@ protected function assertNodeHtml(NodeInterface $node, UserInterface $user, bool
   +      $assert->elementTextContains('css', "$uid_selector", $user->getAccountName());
   

   The quotes can be removed from around $uid_selector.

2. +++ b/core/modules/node/tests/src/Functional/NodeDisplayConfigurableTest.php
   @@ -148,12 +145,14 @@ protected function assertNodeHtml(NodeInterface $node, UserInterface $user, bool
   +      if ($field_classes) {
   +        $assert->elementExists('css', $created_selector);
   +      }
   

   Not sure this is necessary; we check elementTextContains() for this above, so we already know that it exists?

Re #5.2 but why was this added in the first place? I don't see what this is replacing.

OK, the work here is done, I think this is ready to go in.