Colin,

Here are some comments on that test.

1) instead of looping through the results array to count it, just do this:

Object[] results = handler...
assertEquals(5, results.length);

Everything else looked ok.

jesus

On Sun, Oct 5, 2008 at 4:42 AM, Coe, Colin C. (Unix Engineer)
<[EMAIL PROTECTED]> wrote:
> Hi all
>
> Attached is a patch to implement the JUnit tests for the 
> kickstart.profile.get/setCustomOptions API calls.
>
> Comments/criticisms welcome.
>
> CC
>
> NOTICE: This email and any attachments are confidential.
> They may contain legally privileged information or
> copyright material. You must not read, copy, use or
> disclose them without authorisation. If you are not an
> intended recipient, please contact us at once by return
> email and then delete both messages and all attachments.
>
> _______________________________________________
> Spacewalk-devel mailing list
> [email protected]
> https://www.redhat.com/mailman/listinfo/spacewalk-devel
>

_______________________________________________
Spacewalk-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to