This is used to aggregate EntityResolvers and iterate over them. Let's unit test this class.

Comments

dawehner’s picture

  1. +++ b/core/modules/serialization/tests/Drupal/serialization/Tests/EntityResolver/ChainEntityResolverTest.php
    @@ -0,0 +1,130 @@
    + * @group Serialization
    

    @group Drupal would be cool.

  2. +++ b/core/modules/serialization/tests/Drupal/serialization/Tests/EntityResolver/ChainEntityResolverTest.php
    @@ -0,0 +1,130 @@
    +
    

    {@inheritdoc} :(

  3. +++ b/core/modules/serialization/tests/Drupal/serialization/Tests/EntityResolver/ChainEntityResolverTest.php
    @@ -0,0 +1,130 @@
    +  protected function createEntityResolverMock($return = NULL, $called = TRUE) {
    

    This is quite a cool trick to save typing!

damiankloip’s picture

StatusFileSize
new5.36 KB
new918 bytes

Nice, thank you!

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Thank you!

xjm’s picture

2: 2159755-2.patch queued for re-testing.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.