<table><tr><td style="">acrouthamel added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16852">View Revision</a></tr></table><br /><div><div><p>I checked out the test log and see the following, yet I honestly can't figure out why it is off by one, since any line referencing hasDocumentsFolder is a ++ line for the count. Do I need to run this test on a new user account or something?</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">FAIL!  : PlacesItemModelTest::testDeletePlace() Compared lists differ at index 2.
   Actual   (places): "/home/andrew/Desktop"
   Expected (urls): "/home/andrew/Documents"
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(411)]
FAIL!  : PlacesItemModelTest::testPlaceItem(Places - Home) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testPlaceItem(Baloo - Documents) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testPlaceItem(Baloo - Today) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testPlaceItem(Devices - Floppy) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testTearDownDevice() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDefaultViewProperties(Places - Home) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDefaultViewProperties(Baloo - Documents) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDefaultViewProperties(Places - Audio) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDefaultViewProperties(Baloo - Today) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDefaultViewProperties(Devices - Floppy) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testClear() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testHideItem() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testSystemItems() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testEditBookmark() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testEditAfterCreation() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testEditMetadata() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testRefresh() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testIcons(Places - Home) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testIcons(Baloo - Documents) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testIcons(Baloo - Today) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testIcons(Devices - Floppy) Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDragAndDrop() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testHideDevices() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::testDuplicatedEntries() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]
FAIL!  : PlacesItemModelTest::renameAfterCreation() Compared values are not the same
   Actual   (m_model->count())    : 19
   Expected (m_expectedModelCount): 18
   Loc: [/home/andrew/kde/src/kde/applications/dolphin/src/tests/placesitemmodeltest.cpp(245)]</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>add-documents (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16852">https://phabricator.kde.org/D16852</a></div></div><br /><div><strong>To: </strong>acrouthamel, Frameworks, Dolphin, ngraham<br /><strong>Cc: </strong>davidc, elvisangelaccio, kde-frameworks-devel, ngraham, michaelh, bruns<br /></div>