[TIMOB-18970] Android: Soasta Touch Ids not recognised on android widgets
| GitHub Issue | n/a |
|---|
| Type | Bug |
|---|
| Priority | High |
|---|
| Status | Closed |
|---|
| Resolution | Fixed |
|---|
| Resolution Date | 2015-06-30T18:58:24.000+0000 |
|---|
| Affected Version/s | n/a |
|---|
| Fix Version/s | Release 4.1.0, Release 5.0.0 |
|---|
| Components | Android |
|---|
| Labels | merge-4.1.0 |
|---|
| Reporter | Jamie Buckley |
|---|
| Assignee | Hieu Pham |
|---|
| Created | 2015-01-21T18:26:19.000+0000 |
|---|
| Updated | 2015-07-13T18:58:55.000+0000 |
|---|
Description
Problem Description
When assigning a touchtest id to a widget on android the touchtest id is not assigned however on iOS this works.
Reproduction Steps
Enable the attached sample with soasta, record a clip, when clicking on the top text field the id of the textfield is recorded, when clicking on the bottom textfield the id is also recorded, ignoring the touchtest id.
Expected Result
When clicking on the textfield the touchtest id of the widget should be recorded
Actual Result
The id of the top view of the widget is recorded.
Attachments
[~rtlechuga] Can you please follow up with SOASTA on this issue and provide an update?
[~jbuckley] Please add your comments to this ticket
Soasta engineering team is looking into this issue, no ETA yet. https://soasta.zendesk.com/requests/1207
From Soasta: {quote} Engg finally figured out what the problem is and found that the issue is not on our side. We expect touchTestId property to be set in TiViewProxy object which we then pull and set as the "touchTestId" value inside TouchTestDriver. String touchTestId = (String) proxy.getProperty("touchTestId"); For the Widget object, this property is not set in TiViewProxy object. But we still go ahead and use the "id" property for the "touchTestId" value which is a part of the auto-touchtestid generation feature in TouchTestDriver. So, AppC needs to figure out why touchTestId property does not get set in TiViewProxy object when used with the Widget object. {quote}
@Hieu, can you please review this with [~ewieber]?
master PR: https://github.com/appcelerator/titanium_mobile/pull/6943
[~cng][~hpham][~emerriman] Any concerns with pulling this into 4.1.0? It seems okay to me
[~ingo] No problems here
[~hpham] can you please do a backport?
backport PR: https://github.com/appcelerator/titanium_mobile/pull/6949
Verified fixed, using: MacOS 10.11 Studio 4.1.0.201507071122 Ti SDK 4.1.0.GA Appc NPM 4.1.0 Appc CLI 4.1.0 Ti CLI 4.0.1c Alloy 1.6.2 Node v0.10.37 Java 1.7.0_45 preproduction Able to detect and use Widget TTID locators in SOASTA recording and playback.