Subclasses of EntityListBuilder incorrectly override the #empty message

I think this might be a more widespread problem, seems like NodeTypeListBuilder has the same bug. Might as well make the same fix everywhere?

This structure was changed in #1798332: Add paging to the EntityListBuilder and we didn't fix any of the child classes.

Well they could be merged... and we could stop using urlGenerator and just use Url::fromRoute(), and we could have consistent tests.

We can then remove a ton of boilerplate!

Since #2473215: Wrong text if no content types are available was knocked back, can we please combine them and fix all of the subclasses of EntityListBuilder at once?

We should add some tests here.

Here's a test for image styles.

This will need a reroll once this one lands: #2556731: Remove usage of deprecated UrlGeneratorInterface::generateFromPath() from ImageStyleListBuilder::build()

Added a test for the current behavior of ActionListBuilder.

1. +++ b/core/modules/action/src/Tests/ActionListTest.php
   @@ -0,0 +1,42 @@
   +  function testEmptyActionList() {
   

   public

2. +++ b/core/modules/action/src/Tests/ActionListTest.php
   @@ -0,0 +1,42 @@
   +    $this->drupalLogin(
   +      $this->drupalCreateUser(array('administer actions'))
   +    );
   

   One line. Might as well use []

3. +++ b/core/modules/action/src/Tests/ActionListTest.php
   @@ -0,0 +1,42 @@
   +    entity_delete_multiple('action', array_keys($actions));
   

   Deprecated, do not use.

4. +++ b/core/modules/action/src/Tests/ActionListTest.php
   @@ -0,0 +1,42 @@
   +  }
   +}
   

   Missing blank line

Addressing the issues in #22.

Committed/pushed to 8.3.x and cherry-picked to 8.2.x. Thanks

Should be fine for 8.1.x but sending for re-test.