Author: scottbw
Date: Wed Dec  8 14:47:37 2010
New Revision: 1043440

URL: http://svn.apache.org/viewvc?rev=1043440&view=rev
Log:
Fixed an i18n test - turns out the test was correct, but how we evaluated it 
wasn't. All we have to do is make sure the widget is parsed OK, as if it treats 
the required attribute incorrectly it will trigger us trying to load a spurious 
unsupported feature and throwing an exception.

Modified:
    
incubator/wookie/trunk/parser/java/src-test/org/apache/wookie/w3c/test/I18n.java

Modified: 
incubator/wookie/trunk/parser/java/src-test/org/apache/wookie/w3c/test/I18n.java
URL: 
http://svn.apache.org/viewvc/incubator/wookie/trunk/parser/java/src-test/org/apache/wookie/w3c/test/I18n.java?rev=1043440&r1=1043439&r2=1043440&view=diff
==============================================================================
--- 
incubator/wookie/trunk/parser/java/src-test/org/apache/wookie/w3c/test/I18n.java
 (original)
+++ 
incubator/wookie/trunk/parser/java/src-test/org/apache/wookie/w3c/test/I18n.java
 Wed Dec  8 14:47:37 2010
@@ -370,16 +370,15 @@ public class I18n extends ConformanceTes
                        assertEquals(pass_value, feature.getName());
                }       
        }
+       
+       // Note that to pass this we only have to not throw an exception for an 
invalid widget.
        @Test
-       @Ignore // test in error
        public void feature30(){
-               boolean pass_value=false;
                String id="030";
                String[] tests = {"lro","rlo","rtl","ltr"};
                for (String dir: tests){
                        W3CWidget widget = getWidget(id,dir);
-                       IFeatureEntity feature = widget.getFeatures().get(0);
-                       assertEquals(pass_value, feature.isRequired());
+                       assertEquals(0, widget.getFeatures().size());
                }       
        }
        


Reply via email to