Skip to content

Commit

Permalink
pts-core: Fix for not writing null test result file titles
Browse files Browse the repository at this point in the history
  • Loading branch information
michaellarabel committed Apr 5, 2014
1 parent fe773db commit 3d512ef
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CHANGE-LOG
Expand Up @@ -2,6 +2,8 @@ PHORONIX TEST SUITE CHANGE-LOG

Phoronix Test Suite (Git)

- pts-core: Fix for not writing null test result file titles

Phoronix Test Suite 5.0.1-Plavsk
2 April 2014

Expand Down
10 changes: 9 additions & 1 deletion pts-core/objects/pts_merge.php
Expand Up @@ -88,7 +88,15 @@ public static function merge_test_results_process(&$result_file_writer, &$files_

if(!isset($pass_attributes['only_render_results_xml']) && ($result_file_count = count($result_files)) > 0)
{
$result_file_writer->add_result_file_meta_data($result_files[($result_file_count - 1)]);
for($i = ($result_file_count - 1); $i >= 0; $i--)
{
$ret = $result_file_writer->add_result_file_meta_data($result_files[$i]);

if($ret)
{
break;
}
}
}

foreach($result_files as $i => &$result_file)
Expand Down
10 changes: 9 additions & 1 deletion pts-core/objects/pts_result_file_writer.php
Expand Up @@ -131,7 +131,15 @@ public function add_test_notes($test_notes, $json = null)
}
public function add_result_file_meta_data(&$object, $reference_id = null, $title = null, $description = null)
{
$this->xml_writer->addXmlNode('PhoronixTestSuite/Generated/Title', $title != null ? $title : $object->get_title());
$title = !empty($title) ? $title : $object->get_title();

if($title == null)
{
trigger_error('No title supplied for result file meta-data.', E_USER_WARNING);
return false;
}

$this->xml_writer->addXmlNode('PhoronixTestSuite/Generated/Title', $title);
$this->xml_writer->addXmlNode('PhoronixTestSuite/Generated/LastModified', date('Y-m-d H:i:s'));
$this->xml_writer->addXmlNode('PhoronixTestSuite/Generated/TestClient', pts_title(true));
$this->xml_writer->addXmlNode('PhoronixTestSuite/Generated/Description', $description != null ? $description : $object->get_description());
Expand Down

0 comments on commit 3d512ef

Please sign in to comment.