Sounds like the side effect of changing the separator from child to descendant. Maybe I can add the group locating concept here to the separator to put constraints on the separator so that Tellurium knows which one to select. Will let you know the result.
On May 7, 10:16 pm, Jian Fang <[email protected]> wrote: > Seems the new changes on List do not work well for the getListSize() method. > I will look into it and try to fix it as soon as possible. In the meanwhile, > please > switch back to use the XPath version of getListSize, i.e., > getListSizeByXPath() > until I fix the problem there. > > Thanks, > > Jian > > On Thu, May 7, 2009 at 8:49 PM, rohanh <[email protected]> wrote: > > > John, > > > Am still having some problems with Lists. I think I must be doing > > something wrong but I cant figure it out.. > > > Take for example the following html fragment > > > <div id=main> > > ... > > <div class=myBox> > > <div id=heading>Personal Details</div> > > <div> > > <div>Address</div> > > <div>Somewhere 1, Somewhere 1, London, SW13EY</div> > > </div> > > <div> > > <div>Tel</div> > > <div>1111111111</div> > > </div> > > <div> > > <div>Email</div> > > <div><a href="mailto:test...@somewhere";>test...@somwhere</a></ > > div> > > </div> > > <div> > > <div>Mobile</div> > > <div>1111111111</div> > > </div> > > </div> > > </div> > > > And the model. (NOTE: I am not sure I have used the position attribute > > correctly below) > > > ui.Container(uid: "MainPanel", clocator: [tag: "div", id: "main"]){ > > List(uid: "UserProfileDetails" ,clocator: [tag: "div" , > > class: "myBox"], separator: "div") { > > TextBox(uid:"1" ,clocator: [tag: "div"]) > > Container(uid: "all" ,clocator: [tag: "div"]) { > > TextBox(uid: "Name" ,clocator: [tag: "div", > > position: "1"]) > > TextBox(uid: "Value" ,clocator: [tag: "div", > > position: "2"]) > > } > > } > > } > > > Anyway, before the recent change to List.groovy, getListSize > > ("MainPanel.UserProfileDetails") would have reported "5" > > After, the change, it will report "13" as separator "div" will match > > all descendant divs. That's ok I though > > ,as I can remove the separator and use direct: "true" on the List > > content objects, like the following: > > > ui.Container(uid: "MainPanel", clocator: [tag: "div", id: "main"]){ > > List(uid: "UserProfileDetails" ,clocator: [tag: "div" , > > class: "myBox"]) { > > TextBox(uid:"1" ,clocator: [tag: "div" , > > direct: "true"]) > > Container(uid: "all" ,clocator: [tag: "div" , > > direct: "true"]) { > > TextBox(uid: "Name" ,clocator: [tag: "div" , > > direct: "true", position: "1"]) > > TextBox(uid: "Value" ,clocator: [tag: "div" , > > direct: "true", position: "2"]) > > } > > } > > } > > > I would have expected getListSize("MainPanel.UserProfileDetails") to > > return "5" now > > > But instead the logs show: > > > 01:10:31.563 INFO - Command request: getListSize[jquery=#main > > div.myBox, div,div] on session ec03ffac789e4013a4adadc82f38a7eb > > 01:10:31.586 INFO - Got result: OK,13 on session > > ec03ffac789e4013a4adadc82f38a7eb > > 01:10:31.618 INFO - Command request: getText[jquery=#main div.myBox > > div:eq(1) > div > div:eq(0), ] on session > > ec03ffac789e4013a4adadc82f38a7eb > > 01:10:31.655 INFO - Got result: ERROR: Element jquery=#main div.myBox > > div:eq(1) > div > div:eq(0) not found on session > > ec03ffac789e4013a4adadc82f38a7eb > > > It looks to me like the direct: "true" attribute is being ignored for > > list contents. > > Also I am curious about the generated jquery for getText > > ("MainPanel.UserProfileDetails[2].Name")...it has more div children > > that I would have expected.. > > > The correct jquery string should be: #main div.myBox > div:eq(1) > > > div:eq(0) > > > Am I doing something wrong? > > > Thanks > > Rohan > > > On May 6, 3:05 am, John <[email protected]> wrote: > > > Rohanh, > > > > Good question. The direct children came from the old mode that we use > > > header + object's xpath + trailer. > > > But this mode is really not flexible because it has hard-coded xpath > > > there. As a result, it is a bit > > > depreciated now. As long as all the List elements on the same sub-tree > > > related to its parent, you are fine to > > > use not direct children there. As a matter of fact, if you look at the > > > generated locator for List without separator, > > > they are descendants instead of direct children. > > > > I will apply your change there and commit the code soon. > > > > Thanks, > > > > Jian > > > > On May 5, 6:48 pm, rohanh <[email protected]> wrote: > > > > > I have the following html structure which I wish to represent as a > > > > List of repeating groups: > > > > > <div id="main"> > > > > <span>User blah</span> > > > > > <form method="post"> > > > > <div id="cols"> > > > > <div id="col1"> > > > > <table> > > > > <tr> > > > > ... > > > > content > > > > ... > > > > </tr> > > > > </table> > > > > <table> > > > > <tr> > > > > ... > > > > content > > > > ... > > > > </tr> > > > > </table> > > > > </div> > > > > <div id="col2"> > > > > <table> > > > > <tr> > > > > ... > > > > content > > > > ... > > > > </tr> > > > > </table> > > > > <table> > > > > <tr> > > > > ... > > > > content > > > > ... > > > > </tr> > > > > </table> > > > > </div> > > > > </div> > > > > ... > > > > </form> > > > > </div> > > > > > The "content" is the same for each table row ( a link and a checkbox, > > > > plus some other stuff I dont care about ). I have no control of the > > > > html I am testing so at this stage I cannot change it to be more > > > > suitable for tellurium. And I dont really care about the layout or > > > > positioning of the elements I wish to test, all I want is to able to > > > > easily traverse a list and click the links or checkboxes. I though I > > > > would be able to do something like the following: > > > > > ui.Container(uid: "MainContent", clocator: [tag: "div", id: "main"], > > > > group: "true") { > > > > TextBox(uid: "User" , clocator: [tag: "span"]) > > > > Form(uid: "NetworkSubscriptionForm" , clocator: [method: > > > > "post"]) { > > > > List(uid: "List" , clocator: [tag: "div", id: > > > > "cols"], separator: "tr") { > > > > Container(uid: "all" ) { > > > > CheckBox(uid: "Subscribe", clocator: [tag: > > > > "input, type: "checkbox"] ) > > > > UrlLink(uid: "Name" , clocator: [href: "^/ > > > > networks/"]) > > > > } > > > > } > > > > SubmitButton(uid: "Submit" , clocator: [name: > > > > "send_submit"]) > > > > } > > > > > } > > > > > However, it seems the Abstract List Containers "separator" is expected > > > > to be a direct descendant of the List clocator so this did not work.. > > > > Is there a technical reason for forcing the separator to be a direct > > > > descendant? > > > > > To test a theory, I changed the following code so that the direct > > > > descendant restriction was lifted for the separator. It gave the > > > > results I was after. This would allow a more flexible referencing of > > > > items in tables and repeating patterns with a common ancestor. It > > > > should still allow the intended scenarios for lists to work ( I > > > > think ) Can you see any problems with this approach? > > > > > Index: core/src/main/groovy/org/tellurium/object/List.groovy > > > > =================================================================== > > > > --- core/src/main/groovy/org/tellurium/object/List.groovy > > (revision > > > > 1596) > > > > +++ core/src/main/groovy/org/tellurium/object/List.groovy > > (working > > > > copy) > > > > @@ -129,7 +129,7 @@ > > > > if (separator == null || separator.trim().size() == 0) > > > > return deriveListSelector(index) > > > > > - return " > " + separator + ":eq(${index-1})" > > > > + return " " + separator + ":eq(${index-1})" > > > > } > > > > > int getListSize(Closure c) { --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "tellurium-users" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/tellurium-users?hl=en -~----------~----~----~----~------~----~------~--~---
