cullmann added a comment.
I the unit test is separated, no longer all things will be tested with the invalid definitions. In D19532#426268 <https://phabricator.kde.org/D19532#426268>, @dhaumann wrote: > Could you change the unit test please to not mix with the other ones? > > - add a new void testBrokenDefinition() > - use a local Repository repo; > - add custom search oath to this repo > - and then check only the broken file. > > This would be much cleaner and better to understand & maintain. Maybe even put broken.xml into autotest/input/syntax/broken/ Hmm, I disagree: then not all functions are tested with a broken file in the repo. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D19532 To: davispuh, cullmann, dhaumann, vandenoever, vkrause Cc: kwrite-devel, kde-frameworks-devel, gennad, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann