Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-1257] KS XML DOM test fails for Android.

GitHub Issuen/a
TypeBug
PriorityLow
StatusClosed
ResolutionFixed
Resolution Date2011-04-17T01:55:40.000+0000
Affected Version/sn/a
Fix Version/sRelease 1.6.0 M06
ComponentsAndroid
Labelsandroid, defect, release-1.6.0
ReporterBill Dawson
AssigneeDon Thorp
Created2011-04-15T02:47:55.000+0000
Updated2011-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

  1. 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
       
  2. Brion Vibber 2011-04-15

    (I should say that's the first bit that fails, probably more similar miscounts follow. :)

  3. 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...

  4. Don Thorp 2011-04-15

    Verified on G1/1.6 and Nexus One/2.2.1 using build #e1cb22a

    drillbit tests passed on Monday.

JSON Source