stevedlawrence commented on a change in pull request #99: Fix missing checks 
for minOccurs after zero occurrences.
URL: https://github.com/apache/incubator-daffodil/pull/99#discussion_r209913905
 
 

 ##########
 File path: 
daffodil-test/src/test/scala/org/apache/daffodil/section16/array_optional_elem/TestArrayOptionalElemNew.scala
 ##########
 @@ -18,40 +18,28 @@
 package org.apache.daffodil.section16.array_optional_elem
 
 import org.apache.daffodil.util._
-import org.apache.daffodil.tdml.DFDLTestSuite
+import org.apache.daffodil.tdml.Runner
 import org.junit.AfterClass
+import org.junit.Test
 
 object TestArrayOptionalElemNew {
 
 Review comment:
   We don't really use the scala-new thing anymore. I think it's reasonable to 
put this in TestArrayOptionalElem.scala. These *New.scala files should probably 
be deleted with their content moved to the not New files at some point. Maybe 
delete this file while you're at it?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to