jena-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dam...@apache.org
Subject svn commit: r1328373 - /incubator/jena/Jena2/jena/trunk/src/test/java/com/hp/hpl/jena/xmloutput/TestXMLFeatures.java
Date Fri, 20 Apr 2012 13:37:46 GMT
Author: damian
Date: Fri Apr 20 13:37:46 2012
New Revision: 1328373

URL: http://svn.apache.org/viewvc?rev=1328373&view=rev
Log:
Couple more tests for JENA-24

Modified:
    incubator/jena/Jena2/jena/trunk/src/test/java/com/hp/hpl/jena/xmloutput/TestXMLFeatures.java

Modified: incubator/jena/Jena2/jena/trunk/src/test/java/com/hp/hpl/jena/xmloutput/TestXMLFeatures.java
URL: http://svn.apache.org/viewvc/incubator/jena/Jena2/jena/trunk/src/test/java/com/hp/hpl/jena/xmloutput/TestXMLFeatures.java?rev=1328373&r1=1328372&r2=1328373&view=diff
==============================================================================
--- incubator/jena/Jena2/jena/trunk/src/test/java/com/hp/hpl/jena/xmloutput/TestXMLFeatures.java
(original)
+++ incubator/jena/Jena2/jena/trunk/src/test/java/com/hp/hpl/jena/xmloutput/TestXMLFeatures.java
Fri Apr 20 13:37:46 2012
@@ -238,18 +238,30 @@ public class TestXMLFeatures extends XML
         
         // JENA-24
         public void testDisallowedXMLNamespace() throws IOException {
-		// xml and xmlns, if present, must be bound to correct namespaces
-		check(file1, null, Change.setPrefix("j.0", "http://www.w3.org/XML/1998/namespace"));
-                check(file1, null, Change.setPrefix("xml", "http://www.w3.org/XML/1998/namespace"));
-                // Not sure about this case. It's bad, but it ought to round trip
-                //check(file1, null, Change.setPrefix("xml", "http://example.org/#"));
+		// xml, if present, must be bound to correct namespaces
+                
+                // fine, though ill-advised
+		check(file1, null, Change.setPrefix("xml", "http://www.w3.org/XML/1998/namespace"));
+                
+                // bad, but not fatal now -- we probably ought to raise an warning
+                check(file1, null, Change.setPrefix("notxml", "http://www.w3.org/XML/1998/namespace"));
+                
+                // bad, will warn
+                check(file1, null, null, null, true, Change.setPrefix("xml", "http://example.org/#"));
 	}
         
         // JENA-24
         public void testDisallowedXMLNSNamespace() throws IOException {
-		// xml and xmlns, if present, must be bound to correct namespaces
-		check(file1, null, Change.setPrefix("j.0", "http://www.w3.org/2000/xmlns/"));
-                check(file1, null, Change.setPrefix("xmlns", "http://www.w3.org/2000/xmlns/"));
+		// xmlns, if present, must be bound to correct namespace
+                
+                // fine, though ill-advised
+		check(file1, null, Change.setPrefix("xmlns", "http://www.w3.org/2000/xmlns/"));
+                
+                // bad, but not fatal now -- we probably ought to raise an warning
+                check(file1, null, Change.setPrefix("notxmlns", "http://www.w3.org/2000/xmlns/"));
+                
+                // bad, will warn
+                check(file1, null, null, null, true, Change.setPrefix("xmlns", "http://example.org/#"));
 	}
         
 	public void testDuplicateNamespace() throws IOException {



Mime
View raw message