Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
FileSize
34.34 KB

There we go.

tim.plunkett’s picture

Issue tags: -VDC

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

Status: Needs review » Needs work
Issue tags: +VDC

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

dawehner’s picture

Status: Needs work » Needs review
FileSize
34.75 KB

Rerole

dawehner’s picture

Another rerole.

dawehner’s picture

What a fail.

tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community

RTBC if green.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Some new stuff has been added afaics...

ll core/modules/views/tests/views_test_config/test_views/
total 16
drwxr-xr-x  2 alex  staff  136  9 Jun 15:25 .
drwxr-xr-x  3 alex  staff  102  9 Jun 15:25 ..
-rw-r--r--  1 alex  staff  636  4 Jun 19:18 views.view.test_display_invalid.yml
-rw-r--r--  1 alex  staff  554  9 Jun 13:28 views.view.test_entity_row.yml
dawehner’s picture

Status: Needs work » Needs review
FileSize
40.49 KB

A hopefully final rerole.

dawehner’s picture

Issue tags: +PHPUnit

Add another tag

ParisLiakos’s picture

Status: Needs review » Reviewed & tested by the community

yes this will allow using test classes in phpunit:)
applied patch locally and moves all files!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed ca484d9 and pushed to 8.x. Thanks!

tim.plunkett’s picture

Title: Move views test helper modules to views/tests/modules. » [HEAD BROKEN] Move views test helper modules to views/tests/modules.
Status: Fixed » Reviewed & tested by the community
FileSize
443 bytes

#2011006: Default local tasks provided by Views are broken added a test view that didn't get moved.

tim.plunkett’s picture

Category: task » bug
Priority: Normal » Critical
webchick’s picture

Title: [HEAD BROKEN] Move views test helper modules to views/tests/modules. » Move views test helper modules to views/tests/modules.
Category: bug » task
Priority: Critical » Normal
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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