[Webtest] Possible issue with selectform

[Dierk Koenig]

Hi Koorosh,

(hey, cool name, what language is that?)

thank you very much for the bug resolution.

We need to include a test for this into the selftests and apply 
your patch to the codebase.

cheers
Mittie

> -----Original Message-----
> From: webtest-admin@lists.canoo.com
> [mailto:webtest-admin@lists.canoo.com]On Behalf Of Koorosh Vakhshoori
> Sent: Donnerstag, 6. Januar 2005 21:02
> To: webtest@gate2.canoo.com
> Subject: [Webtest] Possible issue with selectform
> 
> 
> Hi,
>    I have been using Webtest for a while and it is a great tool 
> and thanks 
> for all people who are involved.
>    Currently I am migrating my test scripts to latest version 
> which support 
> htmlunit, build 582. I started using <selectform> step in one of my test 
> scripts and started getting the following error message:
> 
> Attribute name must be set, Step: SelectForm ...
> 
> The line it was referring to is:
> 
> <selecform index="2"/>
> 
> Based on the documentation:
> 
> http://webtest.canoo.com/webtest/manual/syntax.html#new_selectform
> 
> Either name or index is required. When looking at the file 
> SelectForm.java 
> in method verifyParameters(), it requires the 'name' attribute 
> only. I went 
> ahead and made the change to my local version and the test past 
> fine. Am I 
> missing something here? Here is my change:
> 
> protected void verifyParameters()
> {
> //Koorosh: Fixed the check so it would work either name or index.
> if ((getName() == null) && (getIndex() == null))
> {
> throw new StepExecutionException("Attribute name/index must be 
> set", this);
> }
> }
> 
> 
> 
> Thanks,
> Koorosh
> 
> _______________________________________________
> WebTest mailing list
> WebTest@lists.canoo.com
> http://lists.canoo.com/mailman/listinfo/webtest
>