These two unit tests are obscure, they are testing old and new way at once which makes the user wonder what is it actually testing.

Unit testing should not branch in this way, old way and new way should have their own proper tests and not leave it up to random occurrence which way is being tested. The code comment in the first test mentions:

        // QSafeFileReader will not use QSafeByteArray, so the exception is different
        // than when using QSafeReasourceReader (and old way).

so this branching should be removed and new unit tests added for both branches of code if needed.

Also TestRail documentation needs to be fixed as a part of this bug fix.