D15825: move testing of common tags of test files to a new data-driven test for taglibextractor

Stefan BrĂ¼ns noreply at phabricator.kde.org
Mon Oct 8 19:49:08 BST 2018


bruns added a comment.


  In D15825#338218 <https://phabricator.kde.org/D15825#338218>, @astippich wrote:
  
  > In D15825#334087 <https://phabricator.kde.org/D15825#334087>, @bruns wrote:
  >
  > > There are some more common tags with identical values, i.e. AlbumArtist, Composer and Lyrics - any reason you kept these?
  >
  >
  > These are the tags directly supported by taglib (e.g. tag->artist() ) which don't require manual reading of tags. These are only the supported tags for the mime types tested in D15833 <https://phabricator.kde.org/D15833>
  
  
  Sorry, I don't get it yet ...
  
  Is there anything preventing this to work:
  
    SimpleExtractionResult result(testFilePath(fileName), mimeType);
    plugin.extract(&result);
    QCOMPARE(result.properties().value(Property::AlbumArtist), QVariant(QStringLiteral("Album Artist")));
  
  As far as I can see, this is identical code for all tests, and supported by all formats.
  
  Or are these not supported by some of the four //new// formats added in D15833 <https://phabricator.kde.org/D15833>?

REPOSITORY
  R286 KFileMetaData

REVISION DETAIL
  https://phabricator.kde.org/D15825

To: astippich, bruns, svuorela
Cc: 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/20181008/e7c0e91a/attachment.html>


More information about the Kde-frameworks-devel mailing list