It is now in views module..should be in views_ui and since it tests RearrangeFilter it should be named as such

#1 drupal-move_ViewsUITest-2012170-1.patch1.22 KBParisLiakos
PASSED: [[SimpleTest]]: [MySQL] 55,341 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


ParisLiakos’s picture

Status: Active » Needs review
1.22 KB
PASSED: [[SimpleTest]]: [MySQL] 55,341 pass(es). View

Status: Needs review » Needs work
Issue tags: -phpunit

The last submitted patch, drupal-move_ViewsUITest-2012170-1.patch, failed testing.

ParisLiakos’s picture

Status: Needs work » Needs review
Issue tags: +phpunit
Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Looks good, noticed this yesterday when working on the change notice for PHPUnit, patch looks good and test is correctly listed on qa.d.o.

damiankloip’s picture

I think we originally planned for more tests to go into this unit test. But then we moved that method to RearrangeFilter and the rest is history. I guess this is good how it is. Not sure it it makes sense to still have a unit test class that tests lots of random methods like this? Seems like if we created a new one every time we would have alot of test classes with not alot in?

ParisLiakos’s picture

thats what we should do (and already do for Core and Component). a test class per file in the same location, so you know exactly where to find it

damiankloip’s picture

Yeah, how this is now is totally fine, and it's not like PHPUnit is slow :)

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 081a59c and pushed to 8.x. Thanks!

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