Regarding DAFFODIL-1300, I’m hunting down usage of “new DFDLTestSuite” and
replacing them with usage of the “Runner” object. In order to help identify
where the DFDLTestSuite was being invoked, I un-commented:
daffodil-tdml-lib/src/main/scala/org/apache/daffodil/tdml/TDMLRunner.scala line
198
@deprecated("2016-12-30", "Use Runner(...) instead."
This revealed that the Runner object itself invokes “new DFDLTestSuite” when
the runOneTest function is called. This seems to defeat the purpose of this
task. I’m hoping for some guidance or opinions on when, if ever, it’s
appropriate to invoke the DFDLTestSuite constructor.
Related to this – I believe I’ll need to create more constructors for the
Runner object in order to preserve the argument combinations currently invoked
by some of the new DFDLTestSuite calls – in particular when compileAllTopLevel
must be provided as true when paired with an xml node instead of a file name.
Is deprecating the DFDLTestSuite constructor still appropriate, or since this
ticket was written have we changed or strategy such that it shouldn’t be called
directly, but will need to be used by the Runner object?
[A picture containing object, clock Description automatically generated]
Ian Carlson | Software Engineer
[Owl Cyber Defense]
W [email protected]<https://owlcyberdefense.com/>
Connect with us!
Facebook<https://www.facebook.com/owlcyberdefense/> |
LinkedIn<https://www.linkedin.com/company/owlcyberdefense/> |
Twitter<https://twitter.com/owlcyberdefense>
[Find us at our next event. Click
Here.]<https://owlcyberdefense.com/resources/events/?utm_source=owl-cyber-defense&utm_medium=email&utm_content=banner&utm_campaign=2020-events>
The information contained in this transmission is for the personal and
confidential use of the individual or entity to which it is addressed.
If the reader is not the intended recipient, you are hereby notified that any
review, dissemination, or copying of this communication is strictly prohibited.
If you have received this transmission in error, please notify the sender
immediately