Issue: #1734642-72: Move breakpoint module into core, failing test at #1734642-67: Move breakpoint module into core

Code in test

  module_uninstall(array('breakpoint_theme_test'));
  $this->assertFalse(entity_load('breakpoint_group', 'module_test'), 'Breakpoint group is removed if module is uninstalled.');

Comments

rfay’s picture

This particular test is still on its inaugural voyage (hasn't completed first test yet, nor should it have). We'll see what happens.

attiks’s picture

http://qa.drupal.org/pifr/test/352933 is the failing one (comment 67)

the other should be ok, testbot on the branch

attiks’s picture

http://qa.drupal.org/pifr/test/353068 failed as well, but on something completely different Drupal\system\Tests\Common\CascadingStylesheetsTest, I'll retest it.

attiks’s picture

FYI
http://qa.drupal.org/pifr/test/352933 failed (has the code included)
http://qa.drupal.org/pifr/test/353068 is green (without the module_uninstall)

rfay’s picture

Status: Active » Fixed

The problem is a dpm() left in the code.

Fatal error: Call to undefined function dpm() in /var/lib/drupaltestbot/sites/default/files/checkout/core/modules/breakpoint/breakpoint.module on line 130
FATAL Drupal\breakpoint\Tests\BreakpointThemeTest: test runner returned a non-zero error code (255).

Please open the "review log" section when solving these things; when running locally in case of a testbot situation, try to run without any of the non-dependency contrib modules installed.

rfay’s picture

BTW, you *can* use debug() in the code, since it's core. It works pretty nicely too, and can give hints as to what's going on on the testbot in some cases.

attiks’s picture

Mea culpa

Status: Fixed » Closed (fixed)

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