[TIMOB-1257] KS XML DOM test fails for Android.
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | Low |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2011-04-17T01:55:40.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 1.6.0 M06 |
Components | Android |
Labels | android, defect, release-1.6.0 |
Reporter | Bill Dawson |
Assignee | Don Thorp |
Created | 2011-04-15T02:47:55.000+0000 |
Updated | 2011-04-17T01:55:40.000+0000 |
Description
Second part of the test (result should be true, is false). Reason not known at this time.
Comments
- Brion Vibber 2011-04-15
The tests make assumptions about whitespace handling; this is the bit that fails:
testResult = testResult && (elements!=null && elements.length==3);
elements.length returns 7, which makes sense:
' <node id="node 1">'+ ' <node id="node 2">'+ ' <node id="node 3"/>'+ ' </node>'+ ' <node id="node 4">'+ ' <node id="node 5"/>'+ ' </node>'+ ' <node id="node 6">'+ ' <node id="node 7"/>'+ ' </node>'+ ' </node>'+
This gives 7 child nodes when including the whitespace:
whitespace element whitespace element whitespace element whitespace
whereas the 3 if the whitespace is stripped out during parsing:
element element elmeent
- Brion Vibber 2011-04-15
(I should say that's the first bit that fails, probably more similar miscounts follow. :)
- Opie Cyrus 2011-04-15
(from [a2079f4872f27847a7ff777993472aa81e123afb]) [#1257 state:fixed-in-qa] Remove XML DOM test from KS
Removed the XML DOM test from KS as there is an updated XML DOM test in drillbit and is no longer needed in KS.
https://github.com/appcelerator/titanium_mobile/commit/a2079f4872f27847a7ff777993472aa81e123afb"> https://github.com/appcelerator/titanium_mobile/commit/a2079f4872f2... - Don Thorp 2011-04-15
Verified on G1/1.6 and Nexus One/2.2.1 using build #e1cb22a
drillbit tests passed on Monday.