I assume that setUp() is not needed on NodeCacheTagsTest, as it only gives permissions to a user, but there is a typo in the permission name.

CommentFileSizeAuthor
#1 2357185.patch851 bytespenyaskito

Comments

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new 2357185.patch851 bytes

Patch.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Ha, good catch!

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

Great catch! Since testbot says it's green, it's good to go. Thanks :)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 29c2974 and pushed to 8.0.x. Thanks!

  • alexpott committed 29c2974 on 8.0.x 
    Issue #2357185 by penyaskito: Remove unnecessary setup in...

Status: Fixed » Closed (fixed)

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