Closed doerk77 closed 1 year ago
Your output window looks weird, you get double outputs for some reason.
Please also check this FAQ: https://tcunit.org/frequently-asked-questions/?Display_FAQ=970
Note that version 1.2.1.0 of TcUnit doesn't exist. The latest release is 1.2.0.0.
Just tested with your sample project
As you can see there are clearly more than 4 tests throughout the various test suites. But only 4 tests get executed.
XML output states the same.
PS: I'm using the latest (unreleased) version from GitHub which tells 1.2.1.0.
I tested it with the latest release 1.2.0.0 and got the following:
Which means that something in a commit after the release has broken this.
@doerk77 Could you look into it and create a PR?
Make sure to read the contributing. Thanks.
Here are all the commits since the release of 1.2.0.0.
Also, if you could think of any good test to add to the TcUnit-Verifier to cover this case, that would be awesome!
@doerk77 seems reasonable. Good work! TcUnit always needs more contributors! Bad of me to not discover this during code review :/
Would be really good if the TcUnit-Runner would detect this failure, so we don't end up here again.
Question: Is there a reason why registering tests is only possible while the testsuite is running?
In general I'd allow it the whole time until the testsuite actually starts running. At this point in time the list should not change any more. That way it would even allow to register tests during FB_Init or in a method with attribute 'call_after_global_init_slot'.
Not that easy to write a test how it might be possible to detect this kind of issue though... Let me think about it.
Your output window looks weird, you get double outputs for some reason.
Seems to be an ADSLOGSTR / TwinCAT feature if you have 2 projects in a single solution.
Your output window looks weird, you get double outputs for some reason.
Seems to be an ADSLOGSTR / TwinCAT feature if you have 2 projects in a single solution.
FYI: Just checked with Beckhoff support. Inside the IDE there's an event listener instantiated per project on the PLC. Therefore the event from ADSLOGSTR is being written to the Error List window twice. However, as far as I can see this shouldn't cause any trouble. It's just a little annoying to see it pop up multiple times.
@doerk77 Yes, you are correct. I just remember that there already was an issue on this particular topic: https://github.com/tcunit/TcUnit/issues/92
Solved. Thanks for finding the problem @doerk77.
Describe the bug RUN_IN_SEQUENCE executes the tests on the first TestSuite, but the tests for the 2nd TestSuite are not being executed.
When RUN_IN_SEQUENCE calls AreAllTestsFinished(), it returns true (because no test has been registered yet) and therefore the tests never get executed. But the tests can only be registered in TEST_ORDERED during execution of the test suite.
Note: It all works fine if the test suites are being executed in parallel.
Screenshots Using TcUnit.RUN() and TEST()![image](https://user-images.githubusercontent.com/82865479/173577037-a689d2c4-b553-481c-b719-cdb269c6e6a4.png)
Using TcUnit.RUN_IN_SEQUENCE() and TEST_ORDERED()![image](https://user-images.githubusercontent.com/82865479/173577422-e3f7c7fd-b7d9-42d6-8fe6-15ebaa7e52e9.png)
Software versions v1.2.1.0