Closed Fisherman166 closed 6 years ago
@Fisherman166 thank you for this PR :) However I've disabled the AV tests in my latest PR (still not merged), can this cause problems during merge? :/
I doubt this will merge conflict. It will likely add my code into the commented out section.
It did cause a merge conflict, but I've fixed it manually (on mobile via github web editor - wow!)
Some basic testing for RtlAnsiStringToUnicodeString has been added. It does not check for any errors related to running out of memory if the function is requested to allocate a pool of memory for the converted string.
Also, the AvSetDisplayMode test fails and causes an exception. The test is skipped for the time being. I plan to add the ability to specify a list of tests to skip at some point assuming the config.txt file works correctly now.