Disable breadcrumbs - 7.x-1.x

Test status
StatusResult
Last receivedSat, 03/03/2012 - 22:58:36
Last requestedSat, 03/03/2012 - 22:59:03
Last testedSat, 03/03/2012 - 22:59:43
Test count4
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier7.x-1.x
ProjectDisable breadcrumbs
LinkProject information
SummaryFAILED: [[Coder]]: [Code review] 35 minor(s), 0 critical(s), and 21 normal(s); [[SimpleTest]]: [MySQL] 30 pass(es), 7 fail(s), and 5 exception(s).
Details

MySQL

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a Drupal installation failure
  8. Detect a test run failure
  9. Detect a failing test
    Review the test failures below then if necessary run them locally until the problem has been fixed.
  10. Complete test suite with all tests passing
30 pass(es), 7 fail(s), and 5 exception(s)

Non-pass

Test namePassFailException
ExpandDisable breadcrumbs (DisableBreadcrumbsTestCase) [Disable breadcrumbs]3075
MessageGroupFilenameLineFunctionStatus
Undefined variable: editorNoticedisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Argument 1 passed to DrupalWebTestCase::drupalLogin() must be an instance of stdClass, null given, called in /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/disable_breadcrumbs/disable_breadcrumbs.test on line 48 and definedRecoverable errordrupal_web_test_case.php1208DrupalWebTestCase->drupalLogin()
Trying to get property of non-objectNoticedrupal_web_test_case.php1214DrupalWebTestCase->drupalLogin()
Trying to get property of non-objectNoticedrupal_web_test_case.php1215DrupalWebTestCase->drupalLogin()
Failed to set field name to Otherdisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Failed to set field pass to Otherdisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Found the requested form fields at userOtherdisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Trying to get property of non-objectNoticedrupal_web_test_case.php1221DrupalWebTestCase->drupalLogin()
User successfully logged in.User logindisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Failed to set field disable_breadcrumb to 1Otherdisable_breadcrumbs.test61DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Found the Save buttonOtherdisable_breadcrumbs.test61DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Found the requested form fields at node/1Otherdisable_breadcrumbs.test61DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()

All

Test namePassFailException
ExpandDisable breadcrumbs (DisableBreadcrumbsTestCase) [Disable breadcrumbs]3075
MessageGroupFilenameLineFunctionStatus
Undefined variable: editorNoticedisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Argument 1 passed to DrupalWebTestCase::drupalLogin() must be an instance of stdClass, null given, called in /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/disable_breadcrumbs/disable_breadcrumbs.test on line 48 and definedRecoverable errordrupal_web_test_case.php1208DrupalWebTestCase->drupalLogin()
Trying to get property of non-objectNoticedrupal_web_test_case.php1214DrupalWebTestCase->drupalLogin()
Trying to get property of non-objectNoticedrupal_web_test_case.php1215DrupalWebTestCase->drupalLogin()
Failed to set field name to Otherdisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Failed to set field pass to Otherdisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Found the requested form fields at userOtherdisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Trying to get property of non-objectNoticedrupal_web_test_case.php1221DrupalWebTestCase->drupalLogin()
User successfully logged in.User logindisable_breadcrumbs.test48DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Failed to set field disable_breadcrumb to 1Otherdisable_breadcrumbs.test61DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Found the Save buttonOtherdisable_breadcrumbs.test61DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()
Found the requested form fields at node/1Otherdisable_breadcrumbs.test61DisableBreadcrumbsTestCase->testDisableNodeBreadcrumbs()

Code review

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a review run failure
  8. Detect coder flags
    Ensure that your code follows the Drupal standard and passes a coder review.
  9. Complete coder review with all passed
35 minor(s), 0 critical(s), and 21 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanddisable_breadcrumbs.admin.inc906
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc4N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc20N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc22N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc38N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc62N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdisable_breadcrumbs.admin.inc69N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc133N/A
Use the matching Drupal theme functions, not raw HTML.Otherdisable_breadcrumbs.admin.inc153N/A
Function summaries should be one line only. (Drupal Docs)Otherdisable_breadcrumbs.admin.inc174N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc175N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.admin.inc176N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc178N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.admin.inc179N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc181N/A
Remove the empty commented line in your function documentation.Otherdisable_breadcrumbs.admin.inc181N/A
Expanddisable_breadcrumbs.module19010
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherdisable_breadcrumbs.module7N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module16N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module17N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module18N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module19N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module30N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module101N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module125N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module175N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module196N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module197N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module225N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module238N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module246N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module260N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module263N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module265N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module268N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module290N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module293N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module311N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module317N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module323N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.module352N/A
Remove the empty commented line in your function documentation.Otherdisable_breadcrumbs.module354N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module419N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.module429N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.module433N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Otherdisable_breadcrumbs.module433N/A
Expanddisable_breadcrumbs.test704
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdisable_breadcrumbs.test20N/A
indent secondary line of comment one space Otherdisable_breadcrumbs.test26N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test27N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test34N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test39N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test40N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test49N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test56N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test58N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test60N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test62N/A
Expanddisable_breadcrumbs.views.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdisable_breadcrumbs.views.inc23N/A

All

Test nameMinorCriticalNormal
Expanddisable_breadcrumbs.admin.inc906
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc4N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc20N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc22N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc38N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc62N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdisable_breadcrumbs.admin.inc69N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc133N/A
Use the matching Drupal theme functions, not raw HTML.Otherdisable_breadcrumbs.admin.inc153N/A
Function summaries should be one line only. (Drupal Docs)Otherdisable_breadcrumbs.admin.inc174N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc175N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.admin.inc176N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc178N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.admin.inc179N/A
There should be no trailing spacesOtherdisable_breadcrumbs.admin.inc181N/A
Remove the empty commented line in your function documentation.Otherdisable_breadcrumbs.admin.inc181N/A
Leafdisable_breadcrumbs.install000
Expanddisable_breadcrumbs.module19010
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherdisable_breadcrumbs.module7N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module16N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module17N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module18N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module19N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module30N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module101N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module125N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module175N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module196N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module197N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module225N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module238N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module246N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module260N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module263N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module265N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module268N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module290N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.module293N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module311N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module317N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module323N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.module352N/A
Remove the empty commented line in your function documentation.Otherdisable_breadcrumbs.module354N/A
There should be no trailing spacesOtherdisable_breadcrumbs.module419N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.module429N/A
If you define a @param or @return, you should document it as well.Otherdisable_breadcrumbs.module433N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Otherdisable_breadcrumbs.module433N/A
Expanddisable_breadcrumbs.test704
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdisable_breadcrumbs.test20N/A
indent secondary line of comment one space Otherdisable_breadcrumbs.test26N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test27N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test34N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test39N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test40N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test49N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test56N/A
There should be no trailing spacesOtherdisable_breadcrumbs.test58N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test60N/A
Separate comments from comment syntax by a space.Otherdisable_breadcrumbs.test62N/A
Expanddisable_breadcrumbs.views.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdisable_breadcrumbs.views.inc23N/A