Review Request 126005: Began to port tests

Olivier Churlaud olivier at churlaud.com
Mon Nov 9 18:06:51 UTC 2015



> On nov. 9, 2015, 1:12 après-midi, Olivier Churlaud wrote:
> > src/core-impl/collections/db/sql/SqlScanResultProcessor.cpp, line 401
> > <https://git.reviewboard.kde.org/r/126005/diff/1/?file=415876#file415876line401>
> >
> >     This line doesn't concern the tests but the sql issue (see mail *Re: [KF5Port] Next-bug: mysql ?* of 07/11/2015 20:15)

I removed this to pass the tests, but now we have the problem again with sql...


- Olivier


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126005/#review88183
-----------------------------------------------------------


On nov. 9, 2015, 7:05 après-midi, Olivier Churlaud wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126005/
> -----------------------------------------------------------
> 
> (Updated nov. 9, 2015, 7:05 après-midi)
> 
> 
> Review request for Amarok.
> 
> 
> Repository: amarok
> 
> 
> Description
> -------
> 
> I port some of the tests.. I would like your feedback on all this.
> 
> 
> Diffs
> -----
> 
>   src/amarokurls/AmarokUrl.cpp e64a1ed 
>   src/browsers/BrowserMessageArea.cpp 13e6dd3 
>   src/core-impl/meta/cue/CueFileSupport.cpp f39b7eb 
>   src/core-impl/meta/file/File.cpp ac37887 
>   src/core-impl/meta/timecode/TimecodeTrackProvider.cpp 2c6fd52 
>   src/core-impl/playlists/types/file/PlaylistFileLoaderJob.cpp a6b448b 
>   src/core-impl/playlists/types/file/asx/ASXPlaylist.cpp 6691a20 
>   src/core-impl/playlists/types/file/xspf/XSPFPlaylist.cpp 8330c3d 
>   src/core/support/Amarok.cpp 1d83542 
>   tests/core-impl/meta/file/TestMetaFileTrack.cpp b1ae0a6 
>   tests/core-impl/playlists/types/file/asx/TestASXPlaylist.cpp ab97410 
>   tests/core-impl/playlists/types/file/m3u/TestM3UPlaylist.cpp 41ffa6c 
>   tests/core-impl/playlists/types/file/pls/TestPLSPlaylist.cpp ccf9465 
>   tests/core-impl/playlists/types/file/xspf/TestXSPFPlaylist.cpp d865213 
>   tests/core/collections/support/TestTrackForUrlWorker.h 8f6c4aa 
>   tests/core/collections/support/TestTrackForUrlWorker.cpp 7d9bebf 
>   tests/core/meta/TestMetaTrack.h 0859988 
>   tests/core/meta/TestMetaTrack.cpp 0f5ee16 
>   tests/dynamic/TestDynamicModel.cpp 43664f5 
>   tests/timecode/TestTimecodeTrackProvider.cpp 82f66bb 
> 
> Diff: https://git.reviewboard.kde.org/r/126005/diff/
> 
> 
> Testing
> -------
> 
> compiled and path some tests:
> 
> 16 - testtrackforurlworker (Failed)
> 18 - testmetatrack (Failed)
> 24 - testplaylistobserver (Failed)
> 31 - testsqlscanmanager (Failed)
> 37 - testcuefilesupport (Failed)
> 38 - testmetafiletrack (Failed)
> 40 - testasxplaylist (Failed)
> 41 - testm3uplaylist (Failed)
> 42 - testplsplaylist (Failed)
> 43 - testxspfplaylist (Failed)
> 45 - testtrackloader (Failed)
> 46 - testdynamicmodel (Failed)  << Corrected but memory leak now.
> 49 - testplaylistfileprovider (Failed)
> 
> 
> Thanks,
> 
> Olivier Churlaud
> 
>

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


More information about the Amarok-devel mailing list