[MGNLTEST-14] ContentApp#selectRowByPath method name and signature may be misleading Created: 10/Jan/20 Updated: 09/Nov/21 Resolved: 09/Nov/21 |
|
| Status: | Closed |
| Project: | Magnolia Test Framework |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Federico Grilli | Assignee: | Unassigned |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | QA&Testing, ease-of-usage, ui-test-fwk | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Epic Link: | core-TF-features-bugs-improvements | ||||||||||||||||
| Story Points: | 1 | ||||||||||||||||
| Description |
|
ContentApp#selectRowByPath(String absolutePath) may trick some devs into thinking that absolutePath is a JCR path whereas this is not necessarily the case. Path here actually represents labels next to Grid expander elements that need to be expanded in order to get to the element to select. |
| Comments |
| Comment by Christoph Meier [ 13/Jan/20 ] |
|
The currently misleading javadoc will already be improved in the context of |
| Comment by Maxime Michel [ 04/Oct/21 ] |
|
Reopening after we unearthed an occurence (see linked ticket) where this is actually an issue. |
| Comment by Christoph Meier [ 09/Nov/21 ] |
|
This one has been tackled. Rico added a now very precise javadoc explaining what the method really does. Closing it now. |