tcunit / TcUnit

An unit testing framework for Beckhoff's TwinCAT 3
Other
258 stars 72 forks source link

Variable "NumberOfTestsToAnalyse" is limited to the wrong boundaries #194

Closed JoergWitt closed 1 year ago

JoergWitt commented 2 years ago

This pull request solves #187.

Correction of the definition of variable NumberOfTestsToAnalyse in the two blocks FB_TestResults (one time) and FB_TestSuite (five times). A wrong limit was used. Old: NumberOfTestsToAnalyse : UINT(1..GVL_Param_TcUnit.MaxNumberOfTestSuites); New: NumberOfTestsToAnalyse : UINT(1..GVL_Param_TcUnit.MaxNumberOfTestsForEachTestSuite);

sagatowski commented 1 year ago

Sorry for late merge. Thanks for your contribution @JoergWitt!