<table><tr><td style="">dfaure added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22687">View Revision</a></tr></table><br /><div><div><p>Wow, good catch, thanks for being the pre-commit CI ! ;-)</p>

<p>Amazingly, it's not because this commit introduces a regression.<br />
It's because the removed unittest method changes what sees the next test method. It used to delete the local sycoca file, which the next test method was making unreadable (but no file = not really a test for unreadable). Now that we don't delete the file anymore, it is indeed unreadable, and the code failed to handle that properly.<br />
<a href="https://phabricator.kde.org/D22759" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D22759</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R309 KService</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22687">https://phabricator.kde.org/D22687</a></div></div><br /><div><strong>To: </strong>dfaure, sitter<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns<br /></div>