Problem/Motivation

Drupal\KernelTests\Core\Entity\CreateSampleEntityTest::testSampleValueContentEntity can randomly fail.

There was 1 failure:

1) Drupal\KernelTests\Core\Entity\CreateSampleEntityTest::testSampleValueContentEntity
Failed asserting that actual size 1 matches expected size 0.

/var/www/html/core/tests/Drupal/KernelTests/Core/Entity/CreateSampleEntityTest.php:81

https://www.drupal.org/pift-ci-job/1425347
https://www.drupal.org/pift-ci-job/1425317
https://www.drupal.org/pift-ci-job/1422878

77  // Create sample entities without bundles.
78  else {
79    $entity = $this->entityTypeManager->getStorage($entity_type_id)->createWithSampleValues(FALSE, $values);
80    $violations = $entity->validate();
81    $this->assertCount(0, $violations);
82    if ($label) {
83      $this->assertEquals($title, $entity->label());
84    }
}
CommentFileSizeAuthor
#9 3086001-9.patch737 bytesalexpott
#9 3086001-9.test.patch2.96 KBalexpott
#7 3086001-6.test-only.patch3.41 KBalexpott
#4 3086001-4-test-only-repeat-500.patch3.01 KBkrzysztof domański
#3 3086001-3-test-only-repeat-500.patch3.01 KBkrzysztof domański
#2 3086001-test-only-repeat-500.patch2.23 KBkrzysztof domański
#2 3086001-test-only-repeat-100.patch2.23 KBkrzysztof domański

Comments

krzysztof domański’s picture

krzysztof domański
Location Poland
commented

Krzysztof Domański created an issue. See original summary.

krzysztof domański’s picture

krzysztof domański
Location Poland
commented
StatusFileSize
new 3086001-test-only-repeat-100.patch2.23 KB
new 3086001-test-only-repeat-500.patch2.23 KB
krzysztof domański’s picture

krzysztof domański
Location Poland
commented
StatusFileSize
new 3086001-3-test-only-repeat-500.patch3.01 KB

Debugging...

           $entity = $this->entityTypeManager->getStorage($entity_type_id)->createWithSampleValues(FALSE, $values);
           $violations = $entity->validate();
+          if (count($violations) > 0) {
+            var_dump($values);
+          }
           $this->assertCount(0, $violations);

Sometimes the $values ​​are an empty array.

Testing Drupal\KernelTests\Core\Entity\CreateSampleEntityTest
F                                                                   1 / 1 (100%)array(0) {
}


Time: 2.81 seconds, Memory: 4.00MB

There was 1 failure:

1) Drupal\KernelTests\Core\Entity\CreateSampleEntityTest::testSampleValueContentEntity
Failed asserting that actual size 1 matches expected size 0.
krzysztof domański’s picture

krzysztof domański
Location Poland
commented
StatusFileSize
new 3086001-4-test-only-repeat-500.patch3.01 KB

Debugging $entity_type_id.

           $entity = $this->entityTypeManager->getStorage($entity_type_id)->createWithSampleValues(FALSE, $values);
           $violations = $entity->validate();
+          if (count($violations) > 0) {
+            var_dump($entity_type_id);
+          }
           $this->assertCount(0, $violations);

This is "path_alias".

Testing Drupal\KernelTests\Core\Entity\CreateSampleEntityTest
F                                                                   1 / 1 (100%)string(10) "path_alias"


Time: 1.89 seconds, Memory: 4.00MB

There was 1 failure:

1) Drupal\KernelTests\Core\Entity\CreateSampleEntityTest::testSampleValueContentEntity
Failed asserting that actual size 1 matches expected size 0.
plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented

Another instance: https://www.drupal.org/pift-ci-job/1428851

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Priority: Normal » Critical

Random test failures are usually critical.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 3086001-6.test-only.patch3.41 KB

Let's see what the violation actually is.

Status: Needs review » Needs work

The last submitted patch, 7: 3086001-6.test-only.patch, failed testing. View results

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Needs review
StatusFileSize
new 3086001-9.test.patch2.96 KB
new 3086001-9.patch737 bytes

Here's a fix. It's happening because occasionally we generate a string 255 chars long and then add a character to it :)

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Needs review » Reviewed & tested by the community

Looks good!

  • catch committed 42034e7 on 8.8.x 
    Issue #3086001 by Krzysztof Domański, alexpott: CreateSampleEntityTest::...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed b541e05 and pushed to 8.8.x. Thanks!

Status: Fixed » Closed (fixed)

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

xjm’s picture

xjm
she/her
Primary language English
commented
Parent issue: #2829040: [meta] Known intermittent, random, and environment-specific test failures »