amber icon indicating copy to clipboard operation
amber copied to clipboard

Amber does not allow Not Supported to be returned as a test result

Open MarcinKantochAMD opened this issue 6 years ago • 3 comments
trafficstars

The function AmberTestInstance::iterate() allows only one of two states to be returned - Pass or Fail. Amber framework does check support of various device features, image format support etc. but if the device returns Not Supported the result is turned by Amber to Fail. I would advice to extend the Result class to store also Not Supported state.

MarcinKantochAMD avatar Sep 11 '19 12:09 MarcinKantochAMD

How does the rest of CTS support this? Is it considered a failure, success or is there a third state we can set in CTS to log this?

Instead of changing the Result class, we should add a check into the Recipe (or somewhere) to verify that the script is fully supported before executing. Then we only call execute if the script is fully supported.

dj2 avatar Sep 11 '19 12:09 dj2

CTS has 3 possible return states - Pass, Fail or Not Supported. Before the test executes it queries the device to check if the formats, operations etc used by the test are supported. If not, it stops execution and returns Not Supported.

MarcinKantochAMD avatar Sep 11 '19 13:09 MarcinKantochAMD

@dneto0

dj2 avatar Sep 19 '19 13:09 dj2