D15714: add a string suffix to test data and use for unicode testing of taglibwriter
Stefan Brüns
noreply at phabricator.kde.org
Sun Sep 23 19:51:34 BST 2018
bruns added inline comments.
INLINE COMMENTS
> svuorela wrote in taglibwritertest.cpp:66
> I suggest checking that the last bytes of all these extracted things is the actual actual utf8 bytes, so that if someone compiles or saves this file in a broken encoding that the testing doesn't fail.
?
The file/tags are written inside this test.
> svuorela wrote in taglibwritertest.cpp:75
> Given stringSuffix is the same for all tests, is it needed to have as a test data thing ?
My idea here was to check each format twice, once with a simple latin1/ascii string (stringsuffix = "") and a second time with some unicode chars (e.g. "€", probably some more from other code blocks).
This allows to differentiate if only unicode tags are broken.
REPOSITORY
R286 KFileMetaData
REVISION DETAIL
https://phabricator.kde.org/D15714
To: astippich, mgallien, bruns
Cc: svuorela, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180923/7835ad0a/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list