[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [test-API PATCH 0/4]Add feature to check case file before running



On 04/12/2012 07:03 PM, Guannan Ren wrote:
> On 04/12/2012 09:43 PM, Guannan Ren wrote:
>> On 04/12/2012 07:53 PM, Peter Krempa wrote:
>>> I don't think pushing this series without a review was a good idea.
>>> You actualy broke all of the tests in the repos/ as you didn't do the
>>> modifications to the parameter checking algorithm in a way that
>>> didn't require modification of the tests, neither did you change the
>>> tests to cope with the new code. The result is now:
>>>
>>> exception.TestCaseError: 'required_params or optional_params not
>>> found in interface:destroy'
>>>
>>> or similar for every test case.
>>>
>>> Peter
>>
>>
>>     Yes, sorry about this.
>>     There is some new feature and cleanup work on my hand, I don't know
>>     the exact time to get review.
>>     The error is generated by framework.  the part job of framework is
>> done.
>>     The cleanup work on testcase is ongoing,  I am sure that I will
>> finish the work today.
>>
> 
>         The cleanup is done and pushed.
>         I only wan to send framework code here, the cleaning code in
> testcases is huge
>         and mechanical, maybe nobody likes seeing it :)
>         Sorry about the intact commit again.
> 
>         Guannan Ren

I was writing next email about this when the internet at my place (and
thus VPN) started disconnecting me.
I had a thought in mind how we can keep all the tests working without
any change. However if this is now fixed then the thought is not needed
anymore.
Next time if there is some major change like this, then I'd like to keep
it that way. In case we don't rewrite it from scratch of course :)

Martin


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]