Unable to locate element by link text in XHTML documents

RESOLVED FIXED in Firefox 49

Status

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ato, Assigned: ato)

Tracking

Version 3
mozilla50
Points:
---

Firefox Tracking Flags

(firefox49 fixed, firefox50 fixed)

Details

Attachments

(5 attachments)

(Assignee)

Comment 1

2 years ago
Created attachment 8767421 [details]
Bug 1283999 - Rewrite and complement element retrieval tests;

Many tests for the plural Find Element, Find Element From Element,
and FInd Elements From Element commands were missing.

Review commit: https://reviewboard.mozilla.org/r/61910/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61910/
Attachment #8767421 - Flags: review?(dburns)
Attachment #8767422 - Flags: review?(dburns)
Attachment #8767423 - Flags: review?(dburns)
Attachment #8767424 - Flags: review?(dburns)
Attachment #8767425 - Flags: review?(dburns)
(Assignee)

Comment 2

2 years ago
Created attachment 8767422 [details]
Bug 1283999 - Rename element retrieval tests to match spec;

The test_findelement.py name is misleading because it covers tests for
commands Find Element, Find Elements, Find Element From Element, Find
Elements From Element, and Get Active Element.

Review commit: https://reviewboard.mozilla.org/r/61912/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61912/
(Assignee)

Comment 3

2 years ago
Created attachment 8767423 [details]
Bug 1283999 - Run element retrieval tests on XHTML documents;

Review commit: https://reviewboard.mozilla.org/r/61914/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61914/
(Assignee)

Comment 4

2 years ago
Created attachment 8767424 [details]
Bug 1283999 - Fix lookup of hyperlinks in XHTML documents;

Lower-case "a" matches hyperlinks in XHTML documents as well as HTML
documents. Upper-case "A" only matches HTML documents.

The patch also refactors link text- and partial link text lookup into
distinct functions, so that there is no more worry about variable scoping
in match blocks.

Review commit: https://reviewboard.mozilla.org/r/61916/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61916/
(Assignee)

Comment 5

2 years ago
Created attachment 8767425 [details]
Bug 1283999 - Publish XPath find element functions;

Publish the findByXPath and findAllByXPath functions to match the style
of the recently introduces findByLinkText and findByPartialLinkText
functions.

Do note that this patch rearranges the input arguments these functions
take.  Possibly we do not have to send the root element as input at all.

Review commit: https://reviewboard.mozilla.org/r/61918/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61918/
(Assignee)

Updated

2 years ago
Assignee: nobody → ato
Status: NEW → ASSIGNED
Comment on attachment 8767421 [details]
Bug 1283999 - Rewrite and complement element retrieval tests;

https://reviewboard.mozilla.org/r/61910/#review58830
Attachment #8767421 - Flags: review?(dburns) → review+
Comment on attachment 8767422 [details]
Bug 1283999 - Rename element retrieval tests to match spec;

https://reviewboard.mozilla.org/r/61912/#review58832
Attachment #8767422 - Flags: review?(dburns) → review+
Attachment #8767423 - Flags: review?(dburns) → review+
Attachment #8767424 - Flags: review?(dburns) → review+
Comment on attachment 8767425 [details]
Bug 1283999 - Publish XPath find element functions;

https://reviewboard.mozilla.org/r/61918/#review58838
Attachment #8767425 - Flags: review?(dburns) → review+

Comment 11

2 years ago
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5118b19b4aa7
Rewrite and complement element retrieval tests; r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/61aa05b0a291
Rename element retrieval tests to match spec; r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/88db59b4e611
Run element retrieval tests on XHTML documents; r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/50de407be97c
Fix lookup of hyperlinks in XHTML documents; r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/a55102c5ea8c
Publish XPath find element functions; r=automatedtester
You need to log in before you can comment on or make changes to this bug.