co♻e: unittest framework for CODESYS

Hi,

Sadly at this moment, string(1024) is not supported, so technically it is not a Bug, but a feature ;-)
Currently a string with maximum length of 255 ( type T_MaxString) is supported only. I will put this idea on hold for the future as it might be a good idea to test larger strings. This idea has to wait for the next release beyond V1.1.0.0.

Hope this helps

Aliazzz

• status: unread --> pending

• Milestone: v1.1.0.0 --> [FUTURE]

On a second thought, adding a test for the stringlength might be a good idea to prevent the runtime from crashing.
The idea is to clip off any extra characters beyond 255. This way the assert stays true to its intent without causing larger content to crash the runtime

A future Extension possibility would be to add a new assert function to accept WSTRING or lengths beyond 255 chars.

Truncating is unfair, as an Assert value that differes only after byte 255 will return SUCCESS when it should return NOT_IMPLEMENTED. I would rather cfUnit crash my test PLC than just compare some of it.

Changing line 147 and 148 to

(* STRING <= STRING(255) *)
    IF UDINT_TO_INT(Expected.TypeClass) = IBaseLibrary.TypeClass.TYPE_STRING  AND (Expected.diSize <= SIZEOF(T_MaxString)) AND (Actual.diSize <= SIZEOF(T_MaxString)) THEN

Will cause it to fall through to the "compare byte by byte" stuff at the bottom (from line 213).
This is good, but will fall down when comparing strings that have not initialized the bytes after the 0x00 Null terminating character.

Thoughts for implementing STRING > 255

You already have the IECStringUtils library in the project.
So it should be OK to Use:

// Will return 0 if STRINGs are equal
IECStringUtils.Stu.StrCmpA(Expected.pValue, Actual.pValue);

If they are not equal, AssertEquals.IteratorCounter will already hold the Position where they first differ. It may be prudent to log:
'Strings first differ at character: ' TO_STRING(IteratorCounter)
And then only LOG ~50 characters before and ~50 characters after the change. Make sure to limit this to within the range of the STRING

Hi Ian,

Would you mind to give an update on the status of this Ticket?

Thank you!

1. TcUnit AssertEquals() still overwrites memory
2. cfUnit RC5 AssertEquals() still overwrites memory
3. Additionally, AssertEquals_WSTRING() and AssertEquals_STRING() happily accept strings over 255 in length, and just ignore any characters after the 255th. This might be obvious to the user though, as the function asks for two STRING(255)s.
4. I have a workaround WstringEquals (FUN) / StringEquals (FUN) with AssertEquals_BOOL here https://forge.codesys.com/u/i-campbell/comparestrings/ci/master/tree/CompareStrings.library

So I probably need to learn the TcUnit pull request technique, develop and submit a pull request in TcUnit?
or just submit a pull request to cfUnit?