[Chandler-dev] Re: [commits] (jeffrey) [14956] - Fix bug 9821, remove makes mine-collection items disappear instead

[Mikeal Rogers]

The original CATS, 0.1, used QAUITestAppLib.

The new CATS 0.2 that Dan and I wrote uses ChandlerTestLib. We moved  
it and renamed it when we moved everything else over because the old  
name was terrible.

At the time we moved it there was no functional difference between  
ChandlerTestLib and QAUITestAppLib except removing some of the calls  
to the old logger and old framework. Since all the current functional  
tests, minus those that use record/playback, use ChandlerTestLib I  
imagine that most additions and fixes have only made their way in to  
ChandlerTestLib.

Last time i checked the performance tests were using CATS 0.1 so  
fixes that Heikki needs must have made it in to QAUITestAppLib somehow.

As I see it we have two options going forward;

- We leave things the way they are and keep making sure that fixes/ 
additions make their way in to QAUITestAppLib when necessary. This  
has the lowest impact on Preview and we're still hoping to move all  
of the tests over to CATS 0.3 (recorded/playback) in the future, and  
0.3 doesn't use either library.

or

- We do the work now to make ChandlerTestLib check which framework  
it's in and call the necessary logger calls. This is a little risky  
as it might alter the timing of the performance test cases ( I don't  
know how much the two libraries differ at this point). It's also a  
lot of throwaway code once we succeed in moving everything over to  
CATS 0.3

Yes, it's annoying we still have 3 functional test frameworks.
No, we won't be able to move all the tests over to CATS 0.3 before  
preview.

-Mikeal

On Jul 9, 2007, at 11:30 AM, Aparna Kadakia wrote:

> Mikeal,
> Do you remember why we have 2 libraries here?
>
> Aparna
>
> Begin forwarded message:
>
>> From: Heikki Toivonen <heikki at osafoundation.org>
>> Date: July 9, 2007 11:08:43 AM PDT
>> Cc: OSAF Development <dev at osafoundation.org>
>> Subject: Re: [Chandler-dev] Re: [commits] (jeffrey) [14956] - Fix  
>> bug 9821, remove makes mine-collection items disappear instead
>>
>> Jeffrey Harris wrote:
>>>> I think you need the same change in QAUITestAppLib.py
>>>
>>> I don't think I understand the circumstances in which changes  
>>> need to
>>> propagate from ChandlerTestLib to QAUITestAppLib.  If they're  
>>> supposed
>>> to be the same, why do we have two libraries?
>>
>> Strictly speaking, if it is a new thing that is used by functional  
>> tests
>> only, then it does not need to go into QAUITestAppLib.py. However,  
>> since
>> QAUITestAppLib.py and ChandlerTestLib.py are meant to be as  
>> identical as
>> possible, any discrepancies will likely cause problems down the line.
>>
>> Ideally we'd have just one file, but so far nobody has done the  
>> work to
>> make it so.
>>
>> -- 
>>   Heikki Toivonen
>>
>>
>> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>>
>> Open Source Applications Foundation "chandler-dev" mailing list
>> http://lists.osafoundation.org/mailman/listinfo/chandler-dev
>