ModuleHandlerTest::testUninstallProfileDependency()

    $dependency = 'dblog';
    $non_dependency = 'ban';

    // Omissis.

    // Uninstall the profile module that is not a dependent.
    $result = $this->moduleInstaller()->uninstall([$non_dependency]);
    $this->assertTrue($result, 'ModuleInstaller::uninstall() returns TRUE.');
    $this->assertFalse($this->moduleHandler()->moduleExists($non_dependency));
    $this->assertEquals(drupal_get_installed_schema_version($non_dependency), SCHEMA_UNINSTALLED, "$dependency module was uninstalled.");

The test code uninstall the ban module ($non_dependency), but the message says the dblog module was uninstalled ($dependency), which is not true.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kiamlaluno created an issue. See original summary.

apaderno’s picture

Status: Active » Needs review
FileSize
1.49 KB
TR’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

apaderno’s picture

Issue summary: View changes
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed 0a3f9f1a60 to 8.7.x and 2d9d447386 to 8.6.x. Thanks!

  • alexpott committed 0a3f9f1 on 8.7.x
    Issue #3014772 by kiamlaluno: ModuleHandlerTest::...

  • alexpott committed 2d9d447 on 8.6.x
    Issue #3014772 by kiamlaluno: ModuleHandlerTest::...

Status: Fixed » Closed (fixed)

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