ConfigurableLanguageTest is in the wrong directory and the wrong namespace

The problem

  1. The ConfigurableLanguageTest is currently located in the directory "core/modules/language/tests/src/". Test classes that extends KernelTestCase class should be located in the directory "core/modules/language/src/Tests/".
  2. The ConfigurableLanguageTest is currently located in the namespace "Drupal\Tests\language". Test classes that extends KernelTestCase class should be located in the namespace "Drupal\language\Tests".
CommentFileSizeAuthor
#2 2427889-2.patch794 bytesdaffie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

daffie’s picture

daffie CreditAttribution: daffie commented
Related issues: +#2407125: LanguageInterface should not support setName

The bug originated from #2407125: LanguageInterface should not support setName.

daffie’s picture

daffie CreditAttribution: daffie commented
Status: Active » Needs review
FileSize
2427889-2.patch794 bytes
Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Title: ConfigurableLanguageTest is in the wrong directory and the wrong namespace » ConfigurableLanguageTest not found by SimpleTest
Status: Needs review » Reviewed & tested by the community

ConfigurableLanguageTest isn't found by SimpleTest. Apply this patch and it shows up. Run it and it passes.

Setting to RTBC, and then I'll run the testbot again.

Mile23 queued 2: 2427889-2.patch for re-testing.

  • xjm committed 88aaa6c on 8.0.x 
    Issue #2427889 by daffie, Mile23: ConfigurableLanguageTest not found by...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Reviewed & tested by the community » Fixed

Good catch, thanks! Committed and pushed to 8.0.x.

Status: Fixed » Closed (fixed)

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