Fix overflow of assert array limits

tcunit / TcUnit

An unit testing framework for Beckhoff's TwinCAT 3

Other

258 stars 72 forks source link

Fix overflow of assert array limits #193

Closed doerk77 closed 9 months ago

doerk77 commented 2 years ago

Checking assertion array limits in TcUnit (issue #190).

Added tests to TcUnit-Verifier
Added code to detect failing tests in TcUnit-Verfier_DotNet.
Added additional command line option to use Visual Studio or XAE shell in TcUnit-Verfier_DotNet. Default DTE set to XAE shell -> Might require update to Jenkins call
Updated formatting to log output of TcUnit-Verfier_DotNet

sagatowski commented 2 years ago

Thanks for this PR. Please read the contributing, especially the part about "Use a Consistent Coding Style". As VS2013 is the default editor for this, the option run in in the XAE shell (which is so far based on VS2017) should be an option (default should be as it is right now).

Does this also solve https://github.com/tcunit/TcUnit/issues/189?

sagatowski commented 9 months ago

@doerk77 I manually copied the stuff over and commited everything in 4a48bdd.

Thanks for this, I'll add you to the contributors for the release. Appreciate your work!