Problem/Motivation

BlockContentFieldTest has the following information in the docblock:

<?php
/**
 * Tests block fieldability.
 *
 * @group block_content
 * @todo Consider removing this test when https://drupal.org/node/1822000 is
 * fixed.
 */
class BlockContentFieldTest extends BlockContentTestBase {
?>

And #1822000: Remove Drupal\field_test\Plugin\Entity\Type\TestEntity in favor of EntityTest already went in.

After asking Berdir, this adds an equivalent test coverage for the link field: #1957748-24: hook_field_prepare_view() is passed empty field values for EntityNG Entities.

Proposed resolution

Remove the testfile

Remaining tasks

None

User interface changes

None

API changes

None

Comments

pcambra’s picture

Status: Active » Needs review
StatusFileSize
new3.23 KB
amateescu’s picture

Status: Needs review » Reviewed & tested by the community

Yay for one less web test :)

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Tests are not frozen in beta. Committed 374a5bd and pushed to 8.0.x. Thanks!

  • alexpott committed 374a5bd on 8.0.x
    Issue #2428633 by pcambra: Remove unnecessary BlockContentFieldTest
    
larowlan’s picture

Damn I was using this test in the block content cache contexts critical

Status: Fixed » Closed (fixed)

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