mbeckerle commented on a change in pull request #521:
URL: https://github.com/apache/daffodil/pull/521#discussion_r608608989



##########
File path: 
daffodil-test/src/test/resources/org/apache/daffodil/section05/simple_types/SimpleTypes.tdml
##########
@@ -5935,12 +5935,13 @@
 -->
   
   <tdml:parserTestCase name="dateCalendarDaysInFirstWeek5" root="date20"
-    model="dateTimeSchema" description="Section 13 Simple Types - 
calendarDaysInFirstWeek - DFDL-13-143R">
+    model="dateTimeSchema" description="Section 13 Simple Types - 
calendarDaysInFirstWeek - DFDL-13-143R"
+    roundTrip="twoPass">

Review comment:
       Just insert comment to the effect of what you just said as for why this 
is two pass. I think this is a good policy for twoPass tests, ... always 
comment on why they have to be two pass. 




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


Reply via email to