bruns accepted this revision. bruns added a comment. This revision is now accepted and ready to land.
Thanks for the explanation. Looks good then, just update the summary. You could add another column to the tests, "hasFullImplementation", and do an Q_EXPECT_FAIL if not set. But thats for another patch ... REPOSITORY R286 KFileMetaData BRANCH refactor_taglib_extractor_tests REVISION DETAIL https://phabricator.kde.org/D15825 To: astippich, bruns, svuorela Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams