Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
This is a follow up from #1911688: Change test group of entity reference SelectionTest.
"Views Modules" is a weird name for this grouping. Could we get a follow-up to rename that to something else, like maybe "Views Module Integration" or similar?
I think that name is pretty good, but suggestions are welcome. We can also remove the capitalisation abuse? i.e. 'Views module integration'.
This is a good novice task.
Comment | File | Size | Author |
---|---|---|---|
#3 | rename-views-modules-1912748-1.patch | 16.95 KB | samhassell |
Comments
Comment #1
dawehnerMaybe we should try to describe a bit more that this are tests for the integration of modules with views and not views with these modules,
but I'm not native and good with everything :)
Comment #2
damiankloip CreditAttribution: damiankloip commented:)
I'm open to changing this to whatever... I can't think of anything good right now though.
Comment #3
samhassell CreditAttribution: samhassell commentedwent with 'Views module integration'
Its hard to think of a better name coz this group covers testing lots of different Views "thingies". Like displays, relationships, filters etc.
Comment #4
ao5357 CreditAttribution: ao5357 commentedI applied this patch and there was a group "Views module integration" but not "View Modules", so everything seems good.
Comment #5
dawehnerIt's already an improvement, so go!
Comment #6
webchickLooks great, thanks! :D
Committed and pushed to 8.x.