Review Request 113277: GSoC 2013 - Advanced Importers - 3/4: Tests for importers framework and concrete importers

Edward Hades Toroshchin edward.hades at gmail.com
Thu Oct 17 14:27:17 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113277/#review41889
-----------------------------------------------------------

Ship it!


A minor note: I am sure that the binary MySQL files will not work across all the possible environments. They might stop working even from a minor version difference (5.1 vs 5.2), but they will definitely fail across platform boundaries (32 vs 64).

But I think we can handle this later.


Music
<http://git.reviewboard.kde.org/r/113277/#comment30571>

    Probably you should remove private data, like directory path and library ID.


- Edward Hades Toroshchin


On Oct. 16, 2013, 5:11 p.m., Konrad Zemek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113277/
> -----------------------------------------------------------
> 
> (Updated Oct. 16, 2013, 5:11 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Repository: amarok
> 
> 
> Description
> -------
> 
> This review contains importers tests.
> 
> The tests/importers/TestImporterBase.cpp contains common tests for concrete
> importers. The tests load a prepared database and read it through the importer.
> It's therefore important that the prepared database contain certain data.
> The tests/importers/files/testCollection directory contains very small mp3
> files that are *not* used as a part of the tests, and instead are meant for
> creating the test database: one would just import these files into a media
> player, copy the media player's database and have it instantly ready for
> the common tests.
> 
> 
> Diffs
> -----
> 
>   Music PRE-CREATION 
>   tests/importers/CMakeLists.txt PRE-CREATION 
>   tests/importers/ImporterMocks.h PRE-CREATION 
>   tests/importers/ImporterMocks.cpp PRE-CREATION 
>   tests/importers/TestAmarokImporter.h PRE-CREATION 
>   tests/importers/TestAmarokImporter.cpp PRE-CREATION 
>   tests/importers/TestBansheeImporter.h PRE-CREATION 
>   tests/importers/TestBansheeImporter.cpp PRE-CREATION 
>   tests/importers/TestClementineImporter.h PRE-CREATION 
>   tests/importers/TestClementineImporter.cpp PRE-CREATION 
>   tests/importers/TestFastForwardImporter.h PRE-CREATION 
>   tests/importers/TestFastForwardImporter.cpp PRE-CREATION 
>   tests/importers/TestITunesImporter.h PRE-CREATION 
>   tests/importers/TestITunesImporter.cpp PRE-CREATION 
>   tests/importers/TestImporterBase.h PRE-CREATION 
>   tests/importers/TestImporterBase.cpp PRE-CREATION 
>   tests/importers/TestImporterManager.h PRE-CREATION 
>   tests/importers/TestImporterManager.cpp PRE-CREATION 
>   tests/importers/TestImporterProvider.h PRE-CREATION 
>   tests/importers/TestImporterProvider.cpp PRE-CREATION 
>   tests/importers/TestRhythmboxImporter.h PRE-CREATION 
>   tests/importers/TestRhythmboxImporter.cpp PRE-CREATION 
>   tests/importers/TestSimpleImporterConfigWidget.h PRE-CREATION 
>   tests/importers/TestSimpleImporterConfigWidget.cpp PRE-CREATION 
>   tests/importers/files/banshee.db PRE-CREATION 
>   tests/importers/files/clementine.db PRE-CREATION 
>   tests/importers/files/collection.db PRE-CREATION 
>   tests/importers/files/illFormedLibrary.xml PRE-CREATION 
>   tests/importers/files/mysqle/amarok/admin.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/admin.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/admin.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/albums.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/albums.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/albums.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/amazon.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/amazon.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/artists.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/artists.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/artists.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/bookmark_groups.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/bookmark_groups.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/bookmarks.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/bookmarks.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/composers.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/composers.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/composers.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/db.opt PRE-CREATION 
>   tests/importers/files/mysqle/amarok/devices.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/devices.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/devices.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/directories.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/directories.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/directories.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/genres.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/genres.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/genres.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/images.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/images.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/labels.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/labels.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/labels.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/lyrics.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/lyrics.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/playlist_groups.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/playlist_groups.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/playlist_tracks.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/playlist_tracks.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/playlists.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/playlists.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/podcastchannels.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/podcastchannels.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/podcastepisodes.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/podcastepisodes.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics_permanent.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics_permanent.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics_tag.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/statistics_tag.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/tracks.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/tracks.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/tracks.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/urls.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/urls.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/urls.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/urls_labels.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/urls_labels.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/urls_labels.frm PRE-CREATION 
>   tests/importers/files/mysqle/amarok/years.MYD PRE-CREATION 
>   tests/importers/files/mysqle/amarok/years.MYI PRE-CREATION 
>   tests/importers/files/mysqle/amarok/years.frm PRE-CREATION 
>   tests/importers/files/mysqle/aria_log.00000001 PRE-CREATION 
>   tests/importers/files/mysqle/aria_log_control PRE-CREATION 
>   tests/importers/files/rhythmdb.xml PRE-CREATION 
>   tests/importers/files/testcollection/albumUnset.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveAlbum1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveAlbum2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveAlbum3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveArtist1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveArtist2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveArtist3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveComposer1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveComposer2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveComposer3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveTitle1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveTitle2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/caseSensitiveTitle3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/composerUnset.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/discUnset.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/multiWordAlbum.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/multiWordArtist.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/multiWordComposer.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/multiWordTitle.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics0.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics4.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics5.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics6.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics7.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics8.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatistics9.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet0.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet4.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet5.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet6.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet7.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet8.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/testStatisticsNotSet9.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trackUnset.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimAlbum1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimAlbum2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimAlbum3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimArtist1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimArtist2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimArtist3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimComposer1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimComposer2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimComposer3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimTitle1.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimTitle2.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/trimTitle3.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/utfAlbum.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/utfArtist.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/utfComposer.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/utfTitle.mp3 PRE-CREATION 
>   tests/importers/files/testcollection/yearUnset.mp3 PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/113277/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Konrad Zemek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20131017/17813bd9/attachment-0001.html>


More information about the Amarok-devel mailing list