----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105389/ -----------------------------------------------------------
(Updated July 11, 2012, 7:31 p.m.) Review request for Amarok, Matěj Laitl and Sven Krohlas. Changes ------- Fixed typo Description ------- Added unit test for core/collections/support/TrackForUrlWorker Just the one slot completeJob() to test. Tested for both KUrl and QString types of urls. Diffs (updated) ----- tests/core/collections/CMakeLists.txt b01b655 tests/core/collections/support/CMakeLists.txt PRE-CREATION tests/core/collections/support/TestTrackForUrlWorker.h PRE-CREATION tests/core/collections/support/TestTrackForUrlWorker.cpp PRE-CREATION tests/mocks/MockTrackForUrlWorker.h PRE-CREATION tests/mocks/MockTrackForUrlWorker.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/105389/diff/ Testing ------- Builds and runs fine. Thanks, Jasneet Bhatti
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
