[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH v5 3/6] iotests: Add casenotrun rep
From: |
Andrey Shinkevich |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH v5 3/6] iotests: Add casenotrun report to bash tests |
Date: |
Sun, 25 Aug 2019 13:03:50 +0000 |
On 16/08/2019 23:33, Cleber Rosa wrote:
> On Thu, Aug 15, 2019 at 08:44:11PM -0400, John Snow wrote:
>>
>>
>> On 7/19/19 12:30 PM, Andrey Shinkevich wrote:
>>> The new function _casenotrun() is to be invoked if a test case cannot
>>> be run for some reason. The user will be notified by a message passed
>>> to the function.
>>>
>>
>> Oh, I assume this is a sub-test granularity; if we need to skip
>> individual items.
>>
>> I'm good with this, but we should CC Cleber Rosa, who has struggled
>> against this in the past, too.
>>
>
> The discussion I was involved in was not that much about skipping
> tests per se, but about how to determine if a test should be skipped
> or not. At that time, we proposed an integration with the build
> system, but the downside (and the reason for not pushing it forward)
> was the requirement to run the iotest outside of a build tree.
>
>>> Suggested-by: Kevin Wolf <address@hidden>
>>> Signed-off-by: Andrey Shinkevich <address@hidden>
>>> ---
>>> tests/qemu-iotests/common.rc | 7 +++++++
>>> 1 file changed, 7 insertions(+)
>>>
>>> diff --git a/tests/qemu-iotests/common.rc b/tests/qemu-iotests/common.rc
>>> index 6e461a1..1089050 100644
>>> --- a/tests/qemu-iotests/common.rc
>>> +++ b/tests/qemu-iotests/common.rc
>>> @@ -428,6 +428,13 @@ _notrun()
>>> exit
>>> }
>>>
>>> +# bail out, setting up .casenotrun file
>>> +#
>>> +_casenotrun()
>>> +{
>>> + echo " [case not run] $*" >>"$OUTPUT_DIR/$seq.casenotrun"
>>> +}
>>> +
>>> # just plain bail out
>>> #
>>> _fail()
>>>
>>
>> seems fine to me otherwise.
>>
>> Reviewed-by: John Snow <address@hidden>
>
> Yeah, this also LGTM.
>
> Reviewed-by: Cleber Rosa <address@hidden>
>
Thank you very much for your reviews. Please note that the function
_casenotrun() works as a notifier only as it was done for the Python
based iotests. It is a caller responsibility to skip running a
particular test with all relevant logic. I will supply the comment in v6
and will keep your 'Reviewed-by' if there are no objections ))
Andrey
--
With the best regards,
Andrey Shinkevich