Splitting out the test coverage addition from #2877924: Remove ContentPreprocess and associated code into its own issue.
See https://www.drupal.org/project/drupal/issues/2877924#comment-16226099 and https://www.drupal.org/project/drupal/issues/2877924#comment-16237435
N/A
Add test coverage confirming that only 1 node title is displayed on the latest version route.
N/A
N/A
N/A
N/A
N/A
N/A
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes, plain diff MR !14313
Comments
Comment #3
acbramley commentedComment #4
berdirI'm always wondering whether something like this should be an addition to an existing test method or a new one since functional tests are fairly heavy. But I think it's OK like this.
This test coverage will allow us to remove the is page code in content _moderation without risk of regressions.
Comment #9
catchThis looks like a good candidate for #3390193: Add a drupalGet() method to KernelTestBase eventually, where as separate method will make less difference, so I think it's fine here. While I've combined some test methods in other issues, tend to try to reserve that only for very expensive tests.
Committed/pushed to main and 11.x, thanks!