(Based on http://qa.drupal.org/pifr/test/261023)

When running tests for #1189184: OOP & PSR-0-ify gettext .po file parsing and generation the testbot fails to pass my patch related tests from #66 where the new Components from 

namespace Drupal\Core\Gettext

are used for the first time. A lot of Locale Import/Export/Plural failures appear 'out of the blue' as they 'work locally'.

So the question is:
- is this related to http://drupal.org/node/346844#comment-1258161 (static cache)
- is this related to being unable to find the new Classes added by the patch (autoloader)
?

(I'll try to add a new patch to #1189184: OOP & PSR-0-ify gettext .po file parsing and generation before my 1 week vacation)

Comments

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented

Is it easy to grep all exceptions from 'Review Log' ?

I found another exception in http://qa.drupal.org/pifr/test/260823 #1539072: Support for disabled languages broken, drop it

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

#1 Hmm... http://qa.drupal.org/pifr/test/260823 shows no exceptions. Obviously you can copy the review log, dump it into a file, and grep it. Or use browser search. But I suspect you're talking about something I don't understand.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

@rfay, @clemens.tolboom just pointed me to this in the review log on http://qa.drupal.org/pifr/test/260823:

OpenID helper functions 45 passes, 0 fails, and 0 exceptions
exception 'PDOException' with message 'SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.locales_source' doesn't exist' in /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Statement.php:58
Stack trace:
#0 /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Statement.php(58): PDOStatement->execute(Array)
#1 /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Connection.php(506): Drupal\Core\Database\Statement->execute(Array, Array)
#2 /var/lib/drupaltestbot/sites/default/files/checkout/core/includes/database.inc(215): Drupal\Core\Database\Connection->query('SELECT s.lid, t...', Array, Array)
#3 /var/lib/drupaltestbot/sites/default/files/checkout/core/modules/locale/locale.module(445): db_query('SELECT s.lid, t...', Array)
#4 /var/lib/drupaltestbot/sites/default/files/checkout/core/includes/bootstrap.inc(1380): locale('1 pass?@count p...', '', 'fr')
#5 /var/lib/drupaltestbot/sites/default/files/checkout/core/includes/common.inc(1779): t('1 pass?@count p...', Array, Array)
#6 /var/lib/drupaltestbot/sites/default/files/checkout/core/modules/simpletest/simpletest.module(107): format_plural(37, '1 pass', '@count passes')
#7 /var/lib/drupaltestbot/sites/default/files/checkout/core/scripts/run-tests.sh(375): _simpletest_format_summary_line(Array)
#8 /var/lib/drupaltestbot/sites/default/files/checkout/core/scripts/run-tests.sh(22): simpletest_script_run_one_test('1', 'PathMonolingual...')
#9 {main}FATAL DrupalMatchPathTestCase: test runner returned a non-zero error code (1).
Drupal match path 41 passes, 0 fails, and 0 exceptions

It is not related to the test, so not sure whether it would actually makes tests fail like Clemens explained.

jthorson’s picture

jthorson commented

Yup ... This is the same error which is popping up in #261023, which is the source of the original report.

It also looks like the total test count (35,324) may be significantly lower than the current D8 head (35,372), so this issue could be masking out any failures ... on the other hand, we may also have simply added ~50 tests in the 12 hours between the runs.

jthorson’s picture

jthorson commented
Title: (jthorson) Locale related Exception in Review log of test. » Locale related Exception in Review log of test.
Priority: Normal » Critical

This same PDOException just appeared in a D8 HEAD run, which for some reason still passed with 35,384 passes.

OpenID helper functions 45 passes, 0 fails, and 0 exceptions
exception 'PDOException' with message 'SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.locales_source' doesn't exist' in /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Statement.php:58
Stack trace:
#0 /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Statement.php(58): PDOStatement->execute(Array)
#1 /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Connection.php(506): Drupal\Core\Database\Statement->execute(Array, Array)
#2 /var/lib/drupaltestbot/sites/default/files/checkout/core/includes/database.inc(215): Drupal\Core\Database\Connection->query('SELECT s.lid, t...', Array, Array)
#3 /var/lib/drupaltestbot/sites/default/files/checkout/core/modules/locale/locale.module(444): db_query('SELECT s.lid, t...', Array)
#4 /var/lib/drupaltestbot/sites/default/files/checkout/core/includes/bootstrap.inc(1380): locale('1 pass?@count p...', '', 'fr')
#5 /var/lib/drupaltestbot/sites/default/files/checkout/core/includes/common.inc(1779): t('1 pass?@count p...', Array, Array)
#6 /var/lib/drupaltestbot/sites/default/files/checkout/core/modules/simpletest/simpletest.module(107): format_plural(37, '1 pass', '@count passes')
#7 /var/lib/drupaltestbot/sites/default/files/checkout/core/scripts/run-tests.sh(375): _simpletest_format_summary_line(Array)
#8 /var/lib/drupaltestbot/sites/default/files/checkout/core/scripts/run-tests.sh(22): simpletest_script_run_one_test('1', 'PathMonolingual...')
#9 {main}Path alias functionality 132 passes, 0 fails, and 0 exceptions
FATAL PathTaxonomyTermTestCase: test runner returned a non-zero error code (1).
Drupal match path 41 passes, 0 fails, and 0 exceptions
jthorson’s picture

jthorson commented

Note: The PDO Exception is actually unrelated to the other errors. It's been around for a while. Suspect this was introduced in #1497230: Use Dependency Injection to handle object definitions .

A potential resolution for your other failures has been flagged in the original issue.

jthorson’s picture

jthorson commented
Status: Active » Fixed

Based on the comments in #1189184: OOP & PSR-0-ify gettext .po file parsing and generation, and a solution for the 'red herring' exception in #1497230: Use Dependency Injection to handle object definitions , I believe this particular issue is resolved ... feel free to bounce it back if I've missed something.

Status: Fixed » Closed (fixed)

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