Acquia Search - 6.x-3.x

Test status
StatusResult
Last receivedFri, 09/21/2012 - 15:41:29
Last requestedFri, 09/21/2012 - 15:42:02
Last testedFri, 09/21/2012 - 15:42:32
Test count4
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier6.x-3.x
ProjectAcquia Search
LinkProject information
SummaryFAILED: [[Coder]]: [Code review] 12 minor(s), 0 critical(s), and 5 normal(s); [[SimpleTest]]: [MySQL] 5 pass(es), 1 fail(s), and 4 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
5 pass(es), 1 fail(s), and 4 exception(s)

Non-pass

Test namePassFailException
ExpandAcquiaSearchWebTestCase114
MessageGroupFilenameLineFunctionStatus
The test did not complete due to a fatal error.Completion checkacquia_search.test184AcquiaSearchWebTestCase->testEnvironment()
Undefined index: contentNoticeinstall.inc316drupal_install_modules()
Undefined index: contentNoticeinstall.inc86drupal_get_installed_schema_version()
Trying to get property of non-objectNoticemodule.inc316module_enable()
require_once(SolrPhpClient/Apache/Solr/Service.php): failed to open stream: No such file or directoryWarningDrupal_Apache_Solr_Service.php2require_once()

All

Test namePassFailException
LeafAcquiaSearchUnitTestCase400
ExpandAcquiaSearchWebTestCase114
MessageGroupFilenameLineFunctionStatus
The test did not complete due to a fatal error.Completion checkacquia_search.test184AcquiaSearchWebTestCase->testEnvironment()
Undefined index: contentNoticeinstall.inc316drupal_install_modules()
Undefined index: contentNoticeinstall.inc86drupal_get_installed_schema_version()
Trying to get property of non-objectNoticemodule.inc316module_enable()
require_once(SolrPhpClient/Apache/Solr/Service.php): failed to open stream: No such file or directoryWarningDrupal_Apache_Solr_Service.php2require_once()

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
12 minor(s), 0 critical(s), and 5 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandacquia_search.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheracquia_search.install-1N/A
ExpandAcquia_Search_Service.php103
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)OtherAcquia_Search_Service.php-1N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().OtherAcquia_Search_Service.php17N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherAcquia_Search_Service.php19N/A
missing space after commaOtherAcquia_Search_Service.php32N/A
Expandtests/acquia_search.test001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otheracquia_search.test230N/A

All

Test nameMinorCriticalNormal
Expandacquia_search.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheracquia_search.install-1N/A
Expandacquia_search.module1100
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otheracquia_search.module54N/A
Format should be * Implementation of hook_foo().Otheracquia_search.module92N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtheracquia_search.module273N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheracquia_search.module437N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otheracquia_search.module484N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtheracquia_search.module484N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheracquia_search.module502N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheracquia_search.module520N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheracquia_search.module530N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheracquia_search.module535N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheracquia_search.module536N/A
ExpandAcquia_Search_Service.php103
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)OtherAcquia_Search_Service.php-1N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().OtherAcquia_Search_Service.php17N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherAcquia_Search_Service.php19N/A
missing space after commaOtherAcquia_Search_Service.php32N/A
Leafdrush/acquia_search.drush.inc000
Expandtests/acquia_search.test001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otheracquia_search.test230N/A