The locale uninstall tests are currently failing.

This is because LocaleUninstallFunctionalTest::testUninstallProcess():

  • Enabled a new language using locale_add_language()
  • Calls DrupalWebTestCase::drupalCreateUser() that calls user_save() that calls the Field API to store the fields
  • The field API fails in field_multilingual_valid_language() because the newly enabled language is not yet reflected in the static cache of language_list()

This was passing before because the static cache of language_list() was never primed before calling user_save(). It is now primed somewhere during the initial cron run.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Damien Tournoud’s picture

Priority: Normal » Critical
Status: Active » Needs review
FileSize
713 bytes

This is actually a critical bug (core broken). And here is a patch, that kills the static cache of language_list() in locale_add_language().

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Confirmed and tested locally. All tests pass with this patch and #699842-20: Move permissions to a tab at admin/people/permissions

webchick’s picture

Title: [Core broken] Locale uninstall tests fail » Locale uninstall tests fail
Status: Reviewed & tested by the community » Fixed

Committed to HEAD, thanks!

Status: Fixed » Closed (fixed)

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