Problem/Motivation

\Drupal\Tests\taxonomy\Functional\Views\TaxonomyTermFilterDepthTest Makes no HTTP requests and all its tests and assertions are already fully covered by \Drupal\Tests\taxonomy\Kernel\Views\TaxonomyTermFilterDepthTest.
In fact that Kernel test even has a few more assertions.

Steps to reproduce

Proposed resolution

- Delete \Drupal\Tests\taxonomy\Functional\Views\TaxonomyTermFilterDepthTest.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3414667

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 !6165

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

Reviewed \Drupal\Tests\taxonomy\Kernel\Views\TaxonomyTermFilterDepthTest and does seem to be testing the same as the functional.

  • catch committed 12b5db62 on 10.2.x 
    Issue #3414667 by Spokje, smustgrave: Delete TaxonomyTermFilterDepthTest...

  • catch committed f9756cc4 on 11.x 
    Issue #3414667 by Spokje, smustgrave: Delete TaxonomyTermFilterDepthTest...
catch’s picture

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

Committed/pushed to 11.x and cherry-picked to 10.2.x, thanks!

Spokje closed merge request !6165

Status: Fixed » Closed (fixed)

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