mirror of
https://github.com/ARM-software/workload-automation.git
synced 2025-04-12 22:00:51 +01:00
manual: Fixed trying to logcat on non linux devices
This commit is contained in:
parent
c96590b713
commit
68fefe8532
@ -59,7 +59,7 @@ class ManualWorkload(Workload):
|
|||||||
Parameter('view', default='SurfaceView',
|
Parameter('view', default='SurfaceView',
|
||||||
description="""Specifies the View of the workload. This enables instruments that require a
|
description="""Specifies the View of the workload. This enables instruments that require a
|
||||||
View to be specified, such as the ``fps`` instrument."""),
|
View to be specified, such as the ``fps`` instrument."""),
|
||||||
Parameter('enable_logcat', kind=boolean, default=True,
|
Parameter('enable_logcat', kind=boolean,
|
||||||
description='If ``True``, ``manual`` workload will collect logcat as part of the results.'),
|
description='If ``True``, ``manual`` workload will collect logcat as part of the results.'),
|
||||||
]
|
]
|
||||||
|
|
||||||
@ -103,3 +103,7 @@ class ManualWorkload(Workload):
|
|||||||
if not self.user_triggered and not self.duration:
|
if not self.user_triggered and not self.duration:
|
||||||
raise ConfigError('Either user_triggered must be ``True`` or duration must be > 0.')
|
raise ConfigError('Either user_triggered must be ``True`` or duration must be > 0.')
|
||||||
|
|
||||||
|
if self.enable_logcat is None:
|
||||||
|
self.enable_logcat = self.device.platform == "android"
|
||||||
|
elif self.enable_logcat and self.device.platform != "android":
|
||||||
|
raise ConfigError("The `enable_logcat` parameter can only be used on Android devices")
|
||||||
|
Loading…
x
Reference in New Issue
Block a user