Review Request: SoK - Unit Test : core/meta/support/MetaKeys( AlbumKey )

[Matěj Laitl]

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



tests/core/meta/support/TestMetaAlbumKey.h
<http://git.reviewboard.kde.org/r/105497/#comment12395>

    Also please test AlbumKey( const AlbumPtr &album ) constructor.



tests/core/meta/support/TestMetaAlbumKey.cpp
<http://git.reviewboard.kde.org/r/105497/#comment12391>

    How big are the added files? They should be no more than a few kilobytes in order not to make Amarok source unnecessary big. You can even create tracks in test programatically, see TestSqlScanManager::createTrack()



tests/core/meta/support/TestMetaAlbumKey.cpp
<http://git.reviewboard.kde.org/r/105497/#comment12392>

    Hmm, spaces in Amaork source file names are rather to be avoided, please rename the directory.



tests/core/meta/support/TestMetaAlbumKey.cpp
<http://git.reviewboard.kde.org/r/105497/#comment12394>

    QCOMPARE? (1 more occurence)


- Matěj Laitl


On July 11, 2012, 8:44 p.m., Jasneet Bhatti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105497/
> -----------------------------------------------------------
> 
> (Updated July 11, 2012, 8:44 p.m.)
> 
> 
> Review request for Amarok, Matěj Laitl and Sven Krohlas.
> 
> 
> Description
> -------
> 
> Unit test for class AlbumKey of core/meta/support/MetaKeys
> 
> Created a couple of silent tracks( take negligible space ) for testing with the artist and album names comprehensively. Seemed like a smarter way than to create large mocks since no other functionality is to be tested for.
> 
> 
> Diffs
> -----
> 
>   tests/core/meta/CMakeLists.txt 3ae78c9 
>   tests/core/meta/support/CMakeLists.txt PRE-CREATION 
>   tests/core/meta/support/TestMetaAlbumKey.h PRE-CREATION 
>   tests/core/meta/support/TestMetaAlbumKey.cpp PRE-CREATION 
>   tests/data/audio/album PRE-CREATION 
>   tests/data/audio/album PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/105497/diff/
> 
> 
> Testing
> -------
> 
> Builds, links and runs fine.
> 
> Output of running test with -v2 flag:
> 
> ********* Start testing of TestMetaAlbumKey *********
> Config: Using QTest library 4.8.1, Qt 4.8.1
> INFO   : TestMetaAlbumKey::initTestCase() entering
> QSYSTEM: TestMetaAlbumKey::initTestCase() qttest(431)/kdecore (K*TimeZone*): No time zone information obtained from ktimezoned 
> PASS   : TestMetaAlbumKey::initTestCase()
> INFO   : TestMetaAlbumKey::testOperatorAssignment() entering
> INFO   : TestMetaAlbumKey::testOperatorAssignment() QVERIFY(!( albumKey1 == albumKey2 ))
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(55)]
> INFO   : TestMetaAlbumKey::testOperatorAssignment() QVERIFY(albumKey1 == tempAlbumKey)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(59)]
> INFO   : TestMetaAlbumKey::testOperatorAssignment() QVERIFY(!( albumKey1 == albumKey2 ))
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(64)]
> INFO   : TestMetaAlbumKey::testOperatorAssignment() QVERIFY(albumKey1 == tempAlbumKey)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(68)]
> PASS   : TestMetaAlbumKey::testOperatorAssignment()
> INFO   : TestMetaAlbumKey::testOperatorLessThan() entering
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(albumKey1 < albumKey2)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(79)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(!( albumKey1 < albumKey1 ))
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(82)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(albumKey2 < albumKey3)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(85)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(albumKey1 < albumKey3)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(88)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(albumKey4 < albumKey5)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(95)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(!( albumKey4 < albumKey4 ))
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(98)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(albumKey5 < albumKey6)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(101)]
> INFO   : TestMetaAlbumKey::testOperatorLessThan() QVERIFY(albumKey4 < albumKey6)
>    Loc: [/home/jasneet/amarok/tests/core/meta/support/TestMetaAlbumKey.cpp(104)]
> PASS   : TestMetaAlbumKey::testOperatorLessThan()
> INFO   : TestMetaAlbumKey::cleanupTestCase() entering
> PASS   : TestMetaAlbumKey::cleanupTestCase()
> Totals: 4 passed, 0 failed, 0 skipped
> ********* Finished testing of TestMetaAlbumKey *********
> 
> 
> Thanks,
> 
> Jasneet Bhatti
> 
>

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