The use of mt_rand(0, 127) to create two different uid's means that we have occasional random on line 196. entity_test_mul: Two entities loaded by uid without caring about property translatability.

The failure occurs when the two randomly generate uid's are the same.

CommentFileSizeAuthor
drupal8.random-fail-entity-translation.patch732 bytesalexpott
drupal8.random-fail-entity-translation.always-fail.patch1.03 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Component: jsonld.module » entity system
Issue tags: +Random test failure

Fixing tags

xjm’s picture

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

Nice find!

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

That must be like the fifth random test failure issue in the entity translation tests... Nice find!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Die, randomosity, die. Great find.

Committed and pushed to 8.x. Thanks!

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