No need for it to be a web test.

Also NodeAccessGrantTest in same namespace inherits from it, so both run the same tests twice. Let's combine them.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

larowlan created an issue. See original summary.

larowlan’s picture

Issue summary: View changes
larowlan’s picture

Status: Active » Needs review
FileSize
13.46 KB
dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Nice catch!

  • catch committed 29133d3 on 8.3.x
    Issue #2791955 by larowlan: Convert \Drupal\node\Tests\NodeAccessTest to...

  • catch committed f68d6fa on 8.2.x
    Issue #2791955 by larowlan: Convert \Drupal\node\Tests\NodeAccessTest to...
catch’s picture

Status: Reviewed & tested by the community » Fixed

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

Status: Fixed » Closed (fixed)

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