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.

CommentFileSizeAuthor
#3 2791955-convert-node-access-test.patch13.46 KBlarowlan

Comments

larowlan created an issue. See original summary.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Issue summary: View changes
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Active » Needs review
StatusFileSize
new 2791955-convert-node-access-test.patch13.46 KB
dawehner’s picture

dawehner
Primary language German
commented
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

catch
he/him
Primary language English
commented
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.