mbeckerle commented on code in PR #928:
URL: https://github.com/apache/daffodil/pull/928#discussion_r1084415823


##########
daffodil-test/src/test/resources/org/apache/daffodil/section13/text_number_props/TextNumberProps.tdml:
##########
@@ -1520,49 +1520,31 @@
     </tdml:infoset>
 
   </tdml:parserTestCase>
-  
-  <!--
-     Test Name: textStandardDecimalSeparator10
-        Schema: textNumberPattern
-          Root: tnp26
-       Purpose: This test demonstrates that textStandardDecimalSeparator can 
be a whitespace-separated list of
-                single characters that appear (individually) in the data as 
the decimal separator
-  -->

Review Comment:
   I believe these block comments, which just restate what is obvious from the 
test itself, serve no purpose. 
   
   I will change the test names however, to better reflect the specific thing 
they are testing. 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to