Apachesolr causes all simpletests to fail

That is odd since the test-bot in drupal.org runs perfectly... I've tested it again and it seems to work. Will see if I can manage to test it through drush.
I would need more info from you, have you tested it through the UI? Does that work?

The same problem.

$ drush test-run ViewsCacheTest
<h1>Additional uncaught exception thrown while handling exception.</h1><h2>Original</h2><p>PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table &amp;#039;aat_clean.simpletest297647node_type&amp;#039; doesn&amp;#039;t exist: SELECT nt.*
FROM 
{node_type} nt
WHERE  (disabled = :db_condition_placeholder_0) 
ORDER BY nt.type ASC; Array
(
    [:db_condition_placeholder_0] =&amp;gt; 0
)
 in _node_types_build() (line 703 of /Users/kenorb/Sites/AAT/docroot/modules/node/node.module).</p><h2>Additional</h2><p>PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table &amp;#039;aat_clean.simpletest297647watchdog&amp;#039; doesn&amp;#039;t exist: INSERT INTO {watchdog} (uid, type, message, variables, severity, link, location, referer, hostname, timestamp) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4, :db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7, :db_insert_placeholder_8, :db_insert_placeholder_9); Array
(
    [:db_insert_placeholder_0] =&amp;gt; 0
    [:db_insert_placeholder_1] =&amp;gt; php
    [:db_insert_placeholder_2] =&amp;gt; %type: !message in %function (line %line of %file).
    [:db_insert_placeholder_3] =&amp;gt; a:6:{s:5:&amp;quot;%type&amp;quot;;s:12:&amp;quot;PDOException&amp;quot;;s:8:&amp;quot;!message&amp;quot;;s:282:&amp;quot;SQLSTATE[42S02]: Base table or view not found: 1146 Table &amp;amp;#039;aat_clean.simpletest297647node_type&amp;amp;#039; doesn&amp;amp;#039;t exist: SELECT nt.*
FROM 
{node_type} nt
WHERE  (disabled = :db_condition_placeholder_0) 
ORDER BY nt.type ASC; Array
(
    [:db_condition_placeholder_0] =&amp;amp;gt; 0
)
&amp;quot;;s:9:&amp;quot;%function&amp;quot;;s:19:&amp;quot;_node_types_build()&amp;quot;;s:5:&amp;quot;%file&amp;quot;;s:56:&amp;quot;/Users/kenorb/Sites/AAT/docroot/modules/node/node.module&amp;quot;;s:5:&amp;quot;%line&amp;quot;;i:703;s:14:&amp;quot;severity_level&amp;quot;;i:3;}
    [:db_insert_placeholder_4] =&amp;gt; 3
    [:db_insert_placeholder_5] =&amp;gt; 
    [:db_insert_placeholder_6] =&amp;gt; http://default/index.php
    [:db_insert_placeholder_7] =&amp;gt; 
    [:db_insert_placeholder_8] =&amp;gt; 127.0.0.1
    [:db_insert_placeholder_9] =&amp;gt; 1336664545
)
 in dblog_watchdog() (line 154 of /Users/kenorb/Sites/AAT/docroot/modules/dblog/dblog.module).</p><hr />Drush command terminated abnormally due to an unrecoverable error.                                                                                                                             [error]

Disabling temporary apachesolr, the error disappears.

I just ran into the same problem. I dug a little and it looks like this could be the same issue as #1410256: apachesolr_hook_info() calls entity_get_info() which can throw an error when installing using a profile.

For a quick temporary fix it is possible to wrap the call to entity_get_info() in a try-catch block

Line 2172 apachesolr.module

/**
 * Implements hook_hook_info_alter().
 */
function apachesolr_hook_info_alter(&$hooks) {
  // we can't do this in apachesolr_hook_info() until http://drupal.org/node/1415278 is fixed
  try {
    foreach (entity_get_info() as $entity_type => $info) {
      $hooks['apachesolr_index_document_build_' . $entity_type] = array(
        'group' => 'apachesolr',
      );
    }
  } catch (Exception $e) {

  }
}

Do we really need this code at all?

If we delete it the only downside is that these hooks are not declared to be part of the "group", which is not a well-used feature anyhow.

Normally hooks do not need to be explicitly defined. However, by declaring a hook explicitly, a module may define a "group" for it. Modules that implement a hook may then place their implementation in either $module.module or in $module.$group.inc. If the hook is located in $module.$group.inc, then that file will be automatically loaded when needed. In general, hooks that are rarely invoked and/or are very large should be placed in a separate include file, while hooks that are very short or very frequently called should be left in the main module file so that they are always available

We do not use the groups feature so I agree in removing it

6.x has this, maybe doesn't need to be changed?

  $hooks['apachesolr_index_document_build'] = array(
    'group' => 'apachesolr',
  );
  $hooks['apachesolr_index_document_node_build'] = array(
    'group' => 'apachesolr',
  );

Right, closing this.

If this might be problematic for you in the D6, please reopen this issue

Sorry I had to reopen this but the current hook_hook_info() implementation creates an error in the trigger module:

hooks = array(
    'apachesolr_field_mappings' => array(
      'group' => 'apachesolr',
    ),
    'apachesolr_field_mappings_alter' => array(
      'group' => 'apachesolr',
    ),
    'apachesolr_query_prepare' => array(
      'group' => 'apachesolr',
    ),
    'apachesolr_query_alter' => array(
      'group' => 'apachesolr',
    ),
    'apachesolr_search_result_alter' => array(
      'group' => 'apachesolr',
    ),
    'apachesolr_environment_delete' => array(
      'group' => 'apachesolr',
    )
  );

  $hooks['apachesolr_index_document_build'] = array(
    'group' => 'apachesolr',
  );
  $hooks['apachesolr_index_document_node_build'] = array(
    'group' => 'apachesolr',
  );

The $hooks content should be a multidimensional array. However, currently it returns a string. Which causes a foreach error in /modules/trigger/trigger.module line 173. The $ops tries to process a foreach loop on 'group', but group is a string.

Forgot to reopen. I agree with comment #9. Let's not use groups in D6 and just remove it.

Same goes for facetapi, facetapi_hook_info() btw

Can you show us to the api page where it states that this is wrongly defined? Also, please provide us with a patch that removes the thing. Makes it easier for us to commit the change and test if it is ok :)

For hook_hook_info() please refer to both: http://drupal.org/node/375833 and http://api.drupal.org D6 hook_hook_info

Patch to remove hook_hook_info attached.

Same solution crossposted to facetapi issue list: #1701214: Remove backport of hook_hook_info()

Committed to 6.x-3.x. Thanks!