olabusayoT commented on code in PR #1509: URL: https://github.com/apache/daffodil/pull/1509#discussion_r2216511921
########## daffodil-core/src/test/scala/org/apache/daffodil/core/dsom/TestMiddleEndAttributes.scala: ########## @@ -179,7 +179,10 @@ class TestMiddleEndAttributes { val mems = seq.groupMembers val Seq(t1: Term) = mems val e1ref = t1.asInstanceOf[ElementRef] - val Some(nes: LocalSequence) = e1ref.optLexicalParent + val nes = e1ref.optLexicalParent match { + case Some(nes: LocalSequence) => nes + case _ => fail(); null + } Review Comment: No, I'd discussed option with Steve before he left and unchecked (which is the recommended way) didn't seem like the best fit -- 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. To unsubscribe, e-mail: commits-unsubscr...@daffodil.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org