[rfc][icedtea-web] PluginAppletViewer unit tests with test extensions for mocking plugin input&output pipes

Jiri Vanek jvanek at redhat.com
Tue Apr 23 23:43:31 PDT 2013


On 04/23/2013 05:57 PM, Adam Domurad wrote:
> On 04/22/2013 05:17 AM, Jiri Vanek wrote:
>> On 04/19/2013 08:31 PM, Adam Domurad wrote:
>>> Hopefully more tasteful this time around :-)
>>
>>
>> Ok. Walk through and I think it can go in. Two nits:
>>
>> 1)Maybe some test to new test-extension classes would be worthy:(
>
> I wouldn't want to anger the gods of quality assurance, attached is my meagre offering:
tss :)

>
> 2013-04-XX Adam Domurad <adomurad at redhat.com>
>
> * tests/netx/unit/net/sourceforge/jnlp/AsyncCallTest.java: Unit tests for
> AsyncCall test extension.
>
>
>> 2) checking the stared Test utilities methods.. When seeing them separated ... well can be as it is, but (although I previously said it in reverse way) maybe they are rally woorthy to be in separate file in test extensions. But as you are confirming my previous suspicions then it is probably ok and do your best.
>>
>> J.
>>
>
> I believe if they are needed they can be separated afterwards.
>

yy
> OK to push everything + this new test ?

yy :)
Go on and push
>
> Happy hacking,
> -Adam




More information about the distro-pkg-dev mailing list