Github user mbeckerle commented on a diff in the pull request:
https://github.com/apache/incubator-daffodil/pull/5#discussion_r150876868
--- Diff:
daffodil-lib/src/main/scala/edu/illinois/ncsa/daffodil/schema/annotation/props/PropertyScoping.scala
---
@@ -209,8 +209,10 @@ trait FindPropertyMixin extends PropTypes {
* For unit testing convenience
*/
final def verifyPropValue(key: String, value: String): Boolean = {
- findPropertyOption(key) match {
- case Found(`value`, _, _, _) => true
+ val prop = findPropertyOption(key)
+ val valueLC = value.toLowerCase
+ prop match {
+ case Found(v, _, _, _) if (v.toLowerCase == valueLC) => true
--- End diff --
Removed case insensitivity. Only one test had to be changed to UTF-8 from
"utf-8".
Moving this is problematic, as tests depend on this being available on DSOM
objects so as to examine specific object for properties. Might be convenient to
have when debugging as well. So I'll leave it on FindPropertyMixin for now.
---
