TcUnit icon indicating copy to clipboard operation
TcUnit copied to clipboard

Variable "NumberOfTestsToAnalyse" is limited to the wrong boundaries

Open JoergWitt opened this issue 3 years ago • 2 comments

Describe the bug It is not possible to set the GVL variable MaxNumberOfTestSuites less than MaxNumberOfTestsForEachTestSuite. It causes compiler errors in FB_TestResults and FB_TestSuite.

The reason is the variable NumberOfTestsToAnalyse used in TcUnit. It is limited to: NumberOfTestsToAnalyse : UINT(1..GVL_Param_TcUnit.MaxNumberOfTestSuites);

The solution is as follows, if you agree my Expected behavior: NumberOfTestsToAnalyse : UINT(1..GVL_Param_TcUnit.MaxNumberOfTestsForEachTestSuite);

Expected behavior I think, that number of test suites and number of test in each test suite are independent from each other and so I can set them independently.

Screenshots NumbreOfTestsToAnalyse

JoergWitt avatar Jun 07 '22 09:06 JoergWitt

@JoergWitt thank you for this extremely well documented issue.

Thanks to your well documented issue it's crystal clear that this is a bug. Could you please create a pull request with the change, and then I will merge it in?

sagatowski avatar Jun 13 '22 14:06 sagatowski

Sure, will create the pull request.

JoergWitt avatar Jun 13 '22 14:06 JoergWitt

Issue solved. Thanks @JoergWitt.

sagatowski avatar Apr 29 '23 21:04 sagatowski