Re: [Jsynthlib-devel] test refactor branch

[Chris W. <chr...@ch...>]

Martin Tarenskeen said on 29/12/11 09:43:
>
>
> On Thu, 29 Dec 2011, Chris Wareham wrote:
>
>> I've commited a change to the AbstractLibraryFrame and Patch classes in
>> order to handle conversion from core.Patch to org.jsynthlib.core.Patch.
>> The patches seem to load and are being identified as Yamaha patches,
>> but not specifically TX81Z ones. I'll test with the trunk version of
>> JSynthLib to see if this is the same behaviour there, and if not I'll
>> work out why tomorrow.
>
> I tested your latest version, and on my system the patches are loaded
> and identified as TX81Z patches perfectly.
>

I'd forgtotten to add the TX81Z driver via the preferences panel - once 
I'd done that the library patcehs were identified as TX81Z patches 
rather than just generic Yamaha ones. It would be nice if JSL loaded the 
correct driver on demand, something for a wish list perhaps.

OK, seeing as it doesn't look like I've broken support for existing 
synths, I'll now try and complete support for the Cheetah MS6. As 
Frankie Fisher mentioned in another mail, it's quite difficult to test 
without a supported device! Not sure whether this is something that 
could be mocked with JMock or Mockito, but it would be very useful.

Chris