[GitHub] [daffodil] stevedlawrence commented on a change in pull request #505: Update scala-library, scala-reflect to 2.12.13

[GitBox <gi...@apache.org>]

stevedlawrence commented on a change in pull request #505:
URL: https://github.com/apache/daffodil/pull/505#discussion_r623962021



##########
File path: daffodil-cli/src/it/resources/org/apache/daffodil/CLI/output/output9.txt
##########
@@ -1,4 +1,4 @@
-<base14:rabbitHole xmlns:b14="http://b14.com" xmlns:a14="http://a14.com" xmlns:base14="http://baseSchema.com">
+<base14:rabbitHole xmlns:a14="http://a14.com" xmlns:b14="http://b14.com" xmlns:base14="http://baseSchema.com">

Review comment:
       Most of our tests either use the TDML runner and so attributes are ignored, or they aren't complex enough to require multiple prefix mappings.
   
   This file is the expected output for one of the few tests where we expect the exact output from the Daffodil CLI. Since we've changed the orer of the namespaces to be consistent and repeatable, regardless of Scala version, this test was affected. So order here doesn't matter, but our test is looking for a specific order because it's looking for exact output.




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