Problem/Motivation

Some test entities has the same base table so they share their tables, which is not something you are supposed to do because each entity can have different basefields.

Proposed resolution

Fix the base tables for test entities.

Remaining tasks

Fix the fails and review the patch.

User interface changes

None

API changes

None

Data model changes

None

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jibran created an issue. See original summary.

jibran’s picture

jibran’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: don_t_allow_test-2751363-2.patch, failed testing.

jibran’s picture

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Thanks. This can have pretty nasty side effects in tests because it creates tables incorrectly and then you get weird exceptions.

Berdir’s picture

I can actually reproduce this problem locally with \Drupal\Tests\system\Kernel\Extension\ModuleHandlerTest::testUninstallContentDependency, not sure why it doesn't happen for testbot?

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: don_t_allow_test-2751363-5.patch, failed testing.

jibran’s picture

Status: Needs work » Reviewed & tested by the community

Unrelated fail.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: don_t_allow_test-2751363-5.patch, failed testing.

jibran’s picture

Status: Needs work » Reviewed & tested by the community

Another unrelated fail.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: don_t_allow_test-2751363-5.patch, failed testing.

jibran’s picture

Status: Needs work » Reviewed & tested by the community
chx’s picture

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: don_t_allow_test-2751363-5.patch, failed testing.

jibran’s picture

Status: Needs work » Reviewed & tested by the community

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: don_t_allow_test-2751363-5.patch, failed testing.

Berdir’s picture

NodeTypeTranslationTest failed. This somehow causes *a lot* of random fails. Wondering if the change has anything to do with it..

jibran’s picture

Status: Needs work » Reviewed & tested by the community

NodeTypeTranslationTest failed for the first time.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed 4dd3e6a5066142135cc15956035339ee89296be6 to 8.2.x and 83cc368 to 8.1.x. Thanks!

Let's hope this isn't a random fail engine.

  • alexpott committed 4dd3e6a on 8.2.x
    Issue #2751363 by jibran: Don't allow test entities to share base table
    

  • alexpott committed 83cc368 on 8.1.x
    Issue #2751363 by jibran: Don't allow test entities to share base table...
jibran’s picture

Thanks @alexpott for committing this.

Status: Fixed » Closed (fixed)

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