Problem/Motivation

\Drupal\Tests\node\Functional\Views\RowPluginTest Makes no HTTP requests but is a functional test.

Steps to reproduce

Proposed resolution

Convert \\Drupal\Tests\node\Functional\Views\RowPluginTest into a Kernel test.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3415557

Command icon Show commands

Start 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:

About issue forks

Comments

Spokje created an issue. See original summary.

Spokje opened merge request !6228

spokje’s picture

spokje
Primary language Dutch
commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Coverage still seems to be there.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs work

One arguably out of scope comment on the arguably out of scope change in the MR.

spokje’s picture

spokje
Primary language Dutch
commented
Status: Needs work » Needs review

No arguments from me on this one, changed as arguably never requested...

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Feedback appears to be addressed.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

Backported to 10.2.x to keep tests in sync.

Committed and pushed 929da61ebc to 11.x and 0d176b28b4 to 10.2.x. Thanks!

  • longwave committed 0d176b28 on 10.2.x 
    Issue #3415557 by Spokje, catch: Convert RowPluginTest into a Kernel...

  • longwave committed 929da61e on 11.x 
    Issue #3415557 by Spokje, catch: Convert RowPluginTest into a Kernel...

longwave closed merge request !6228

Status: Fixed » Closed (fixed)

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