Skip to content
Permalink
Browse files

pts-core: Add RequiresDisplay test profile property to indicate if th…

…e test profile explicitly requires a monitor / display server to work
  • Loading branch information...
michaellarabel committed Sep 8, 2018
1 parent 2fc399d commit 3567d78b326402a3c33e521385e42c8f884aad37
@@ -1,5 +1,9 @@
PHORONIX TEST SUITE CHANGE-LOG

Phoronix Test Suite (Git)

pts-core: Add RequiresDisplay test profile property to indicate if the test profile explicitly requires a monitor / display server to work

Phoronix Test Suite 8.2.0 Milestone 4
7 September 2018

@@ -1700,7 +1700,7 @@ public static function test_profile_system_compatibility_check(&$test_profile, $
{
$valid_test_profile = false;
}
else if($test_type == 'Graphics' && !phodevi::is_display_server_active())
else if($test_profile->is_display_required() && !phodevi::is_display_server_active())
{
$report_errors && pts_client::$display->test_run_error('No display server was found, cannot run ' . $test_profile);
$valid_test_profile = false;
@@ -342,6 +342,10 @@ public function is_root_required()
{
return pts_strings::string_bool($this->xg('TestProfile/RequiresRoot', 'FALSE'));
}
public function is_display_required()
{
return pts_strings::string_bool($this->xg('TestProfile/RequiresDisplay', 'FALSE')) || $this->get_test_hardware_type() == 'Graphics';
}
public function allow_cache_share()
{
return pts_strings::string_bool($this->xg('TestSettings/Default/AllowCacheShare'));
@@ -282,6 +282,17 @@
<xs:documentation>RequiresRoot allows a value of TRUE or FALSE if the test requires root/administrator rights for successfully running the test. By default the value is assumed to FALSE. If TRUE, the Phoronix Test Suite will attempt to prompt the user at run-time for their root/sudo password to run the test with elevated permissions if the user is not current root/administrator.</xs:documentation>
</xs:annotation>
</xs:element>
<xs:element name="RequiresDisplay" type="Boolean" minOccurs="0" maxOccurs="1">
<xs:annotation>
<xs:appinfo>
<get>is_display_required</get>
<set></set>
<default>FALSE</default>
<flags>UNCOMMON</flags>
</xs:appinfo>
<xs:documentation>RequiresDisplay allows a value of TRUE or FALSE if the test requires a display server / monitor to be attached for the test to run, i.e. the test requires the ability to launch a GUI window or similar in order for the test to execute. By default the value is assumed to FALSE unless the TestType is set to 'Graphics' and then it is assumed to be TRUE for GPU tests. If TRUE, the Phoronix Test Suite will skip tests from execution if no display server is believed to be running.</xs:documentation>
</xs:annotation>
</xs:element>
<xs:element name="EnvironmentSize" type="PositiveDecimal" minOccurs="0" maxOccurs="1">
<xs:annotation>
<xs:appinfo>

0 comments on commit 3567d78

Please sign in to comment.
You can’t perform that action at this time.