assertTableColumn() in uuid.test causes assertion to fail incorrectly as $record->field property does not exist. When running the tests the property is $record->Field (capital F). I don't know if this is different in different versions of mysql/php/drupal/pdo/etc, so safest would be to check both properties i.e.

function assertTableColumn($table, $column, $message) {
    $result = db_query("SHOW COLUMNS FROM {$table}");
    $exists = FALSE;
    foreach ($result as $record) {
      $field_property = isset($record->Field) ? "Field" : "field";
      if ($record->$field_property == $column) {
        $exists = TRUE;
        break;
      }
    }
    $this->assertTrue($exists, $message);
  }
CommentFileSizeAuthor
#1 uuid-fix-assertTableColumn-2199617.patch484 bytesandyrigby
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andyrigby’s picture

andyrigby’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: uuid-fix-assertTableColumn-2199617.patch, failed testing.