I accidentally found this on the plane home from Barcelona. :)

In /core/modules/field/lib/Drupal/field/Tests there are a bunch of test files called FieldBlahBlah.php, but one is called FielBlahBlah.php (no d). Needs a rename.

I'm not sure the exact Git commands for this (check #drupal-gitsupport), but otherwise this should be a novice patch. :)

CommentFileSizeAuthor
#3 rename.patch901 bytesdroplet
#1 rename.patch329 bytesdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

Status: Active » Needs review
FileSize
329 bytes
git mv core/modules/field/lib/Drupal/field/Tests/FielInstanceCrudTest.php core/modules/field/lib/Drupal/field/Tests/FieldInstanceCrudTest.php
git commit -m "rename to FieldInstanceCrudTest"

also attached a patch.

webchick’s picture

Status: Needs review » Needs work

No, it's more than that. The innards of the class are wrong as well.

droplet’s picture

Status: Needs work » Needs review
FileSize
901 bytes

ack Fiel[^d]
core/modules/field/lib/Drupal/field/Tests/FielInstanceCrudTest.php
5: * Definition of Drupal\field\Tests\FielInstanceCrudTest.
12:class FielInstanceCrudTest extends FieldTestBase {

PATCH

ack Fiel[^d]
(nothing)

Go Go Go Bot.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

done?

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks! :)

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