Problem/Motivation

This issue is a follow up to #3211992: TestSiteApplicationTest::testInstallWithNonExistingFile() fails when another test creates database tables during the test run which only removed the broken code from one of the test methods but was meant to remove it from both. This issue is to remove it from TestSiteApplicationTest::testInstallWithNonExistingFile() and also from :: testInstallWithFileWithNoClass() because it can cause random failures.

Steps to reproduce

Proposed resolution

Remove the table counting from \Drupal\Tests\Scripts\TestSiteApplicationTest::testInstallWithNonExistingFile() and testInstallWithFileWithNoClass().

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3383230

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

hooroomoo created an issue. See original summary.

hooroomoo’s picture

Issue summary: View changes
hooroomoo’s picture

Status: Active » Needs review
smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Seems like a simple update.

hooroomoo’s picture

Issue summary: View changes
hooroomoo’s picture

Issue summary: View changes

  • lauriii committed a8e97817 on 11.x
    Issue #3383230 by hooroomoo: TestSiteApplicationTest::...

lauriii’s picture

Status: Reviewed & tested by the community » Fixed

I saw this randomly failing on couple issues this weekend. Not sure what's causing that but seems like this is just applying the fix from #3211992: TestSiteApplicationTest::testInstallWithNonExistingFile() fails when another test creates database tables during the test run to other test methods in the same class which seems fine.

Committed a8e9781 and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

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