Noticed this in the log output:

Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[HY000]: General error: 1366 Incorrect integer value: '' for column 'line' at row 3: INSERT INTO {simpletest} (test_id, test_class, status, message, message_group, function, line, file) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4, :db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7), (:db_insert_placeholder_8, :db_insert_placeholder_9, :db_insert_placeholder_10, :db_insert_placeholder_11, :db_insert_placeholder_12, :db_insert_placeholder_13, :db_insert_placeholder_14, :db_insert_placeholder_15), (:db_insert_placeholder_16, :db_insert_placeholder_17, :db_insert_placeholder_18, :db_insert_placeholder_19, :db_insert_placeholder_20, :db_insert_placeholder_21, :db_insert_placeholder_22, :db_insert_placeholder_23), (:db_insert_placeholder_24, :db_insert_placeholder_25, :db_insert_placeholder_26, :db_insert_placeholder_27, :db_insert_placeholder_28, :db_insert_placeholder_29, :db_insert_placeholder_30, :db_insert_placeholder_31); Array
(
    [:db_insert_placeholder_0] => 828
    [:db_insert_placeholder_1] => Drupal\Tests\Component\PhpStorage\FileStorageTest
    [:db_insert_placeholder_2] => pass
    [:db_insert_placeholder_3] =>
    [:db_insert_placeholder_4] => Other
    [:db_insert_placeholder_5] => Drupal\Tests\Component\PhpStorage\FileStorageTest->testCRUD()
    [:db_insert_placeholder_6] => 42
    [:db_insert_placeholder_7] => /var/lib/drupaltestbot/sites/default/files/checkout/core/tests/Drupal/Tests/Component/PhpStorage/FileStorageTest.php
    [:db_insert_placeholder_8] => 828
    [:db_insert_placeholder_9] => Drupal\Tests\Component\PhpStorage\FileStorageTest
    [:db_insert_placeholder_10] => pass
    [:db_insert_placeholder_11] =>
    [:db_insert_placeholder_12] => Other
    [:db_insert_placeholder_13] => Drupal\Tests\Component\PhpStorage\FileStorageTest->testReadOnly()
    [:db_insert_placeholder_14] => 51
    [:db_insert_placeholder_15] => /var/lib/drupaltestbot/sites/default/files/checkout/core/tests/Drupal/Tests/Component/PhpStorage/FileStorageTest.php
    [:db_insert_placeholder_16] => 828
    [:db_insert_placeholder_17] =>
    [:db_insert_placeholder_18] => pass
    [:db_insert_placeholder_19] =>
    [:db_insert_placeholder_20] => Other
    [:db_insert_placeholder_21] => ->Drupal\Tests\Component\Utility\StringTest::testCheckPlain()
    [:db_insert_placeholder_22] =>
    [:db_insert_placeholder_23] =>
    [:db_insert_placeholder_24] => 828
    [:db_insert_placeholder_25] =>
    [:db_insert_placeholder_26] => pass
    [:db_insert_placeholder_27] =>
    [:db_insert_placeholder_28] => Other
    [:db_insert_placeholder_29] => ->Drupal\Tests\Component\Utility\StringTest::testFormat()
    [:db_insert_placeholder_30] =>
    [:db_insert_placeholder_31] =>
)
 in Drupal\Core\Database\Connection->query() (line 554 of /var/lib/drupaltestbot/sites/default/files/checkout/core/lib/Drupal/Core/Database/Connection.php).].
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ParisLiakos’s picture

Issue tags: +PHPUnit

tagging

Berdir’s picture

Sorry for not providing more information.

This happens on every single test run, you can e.g. see it here: http://qa.drupal.org/8.x-status and it can easily be reproduced locally when running one of the mentioned tests there:

php core/scripts/run-tests.sh --class "Drupal\Tests\Component\PhpStorage\FileStorageTest"

ParisLiakos’s picture

Status: Active » Needs review
FileSize
1005 bytes

seems we have the wrong typehint there
also fixed some spaces there

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

This looks good. the error is no longer displayed.

This must be my first RTBC from mobile. ;)

I don't think we can test this and it should not block this from getting committed.

ParisLiakos’s picture

testing tests ftw:P
lets get it in

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Weird. How come PIFT wasn't blowing up constantly on this..?

In any event, committed and pushed to 8.x. thanks!

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