daffodil-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [daffodil] mbeckerle commented on a change in pull request #560: Verify that we get errors if the files contain a DOCTYPE
Date Mon, 17 May 2021 22:24:36 GMT

mbeckerle commented on a change in pull request #560:
URL: https://github.com/apache/daffodil/pull/560#discussion_r633905668



##########
File path: daffodil-core/src/test/scala/org/apache/daffodil/processor/TestSAXUtils.scala
##########
@@ -54,10 +55,12 @@ object TestSAXUtils {
   lazy val testInfosetString: String = expectedInfoset.toString()
   lazy val testData: String = "910"
 
+  lazy val loader = new DaffodilXMLLoader()
 
-  lazy val qualifiedWithNestedSchemasFile: File = new File(
-    Misc.getRequiredResource("/test/example_nested_namespaces_qualified.dfdl.xsd"))
-  lazy val qualifiedWithNestedSchemasElem: Elem = scala.xml.XML.loadFile(qualifiedWithNestedSchemasFile)
+  lazy val qualifiedWithNestedSchemasFile =
+    Misc.getRequiredResource("/test/example_nested_namespaces_qualified.dfdl.xsd")
+  lazy val qualifiedWithNestedSchemasElem: Elem =
+    loader.load(URISchemaSource(qualifiedWithNestedSchemasFile), None).asInstanceOf[Elem]

Review comment:
       @olabusayoT  Please note this is what I changed, that eliminated the bad behavior.

   
   It remains a mystery to me why calling scala.xml.XML.loadFile(...) was resulting in bad
behavior, but only when sbt was run from the command line. 
   I even commented out all but one test in the TestSAXParseAPI.scala file, but still running
sbt "testOnly *TestSAXParseAPI" would run just that one test, and fail with the b02:a namespace
prefix problem. 
   
   Changing this to use DaffodilXMLLoader.load fixed the issue. I am not sure why. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message