Move Entity Reference Tests out of views folder [#1906448]

Comment	File	Size	Author
#1	drupal-1906448-1.patch	898 bytes	dawehner

File	Size
drupal-1906448-1.patch	898 bytes

Comment #2

3 February 2013 at 02:16

Status:	Needs review	» Needs work
Issue tags:	-Entity Reference, -VDC

The last submitted patch, drupal-1906448-1.patch, failed testing.

Log in or register to post comments

Comment #3

dawehner

German

CreditAttribution: dawehner commented 3 February 2013 at 09:15

Status:	Needs work	» Needs review
Issue tags:		+Entity Reference, +VDC

#1: drupal-1906448-1.patch queued for re-testing.

Log in or register to post comments

Comment #4

aspilicious CreditAttribution: aspilicious commented 3 February 2013 at 09:52

Status:

Needs review

» Reviewed & tested by the community

Log in or register to post comments

Comment #5

amitaibu

Hebrew

Israel

CreditAttribution: amitaibu commented 3 February 2013 at 10:13

This is something we actually did on purpose. Since Views and ER are both in core, there's no longer the notion of "who is more important" like in contrib. So in this case Views is really testing it's own implementation for ER.

Log in or register to post comments

Comment #6

webchick

she/they

English

Vancouver 🇨🇦

CreditAttribution: webchick commented 3 February 2013 at 11:02

Status:

Reviewed & tested by the community

» Needs review

Sounds like that's a needs review, or possibly a won't fix.

Log in or register to post comments

Comment #7

dawehner

German

CreditAttribution: dawehner commented 3 February 2013 at 13:39

This is something we actually did on purpose. Since Views and ER are both in core, there's no longer the notion of "who is more important" like in contrib. So in this case Views is really testing it's own implementation for ER.

So I think the tests should be at the place where the code exists. Here entity reference places it's views plugins in it's own module.
We currently put the (node|user|...) specific tests/views plugins already in the module folder, so this patch just applies how every other piece does it.