This adds support for Documents detection when included as a default
as implemented in D16852. This is a work-in-progress and currently
does not succeed.
Assistance is welcome.
Details
Details
- Reviewers
acrouthamel - Group Reviewers
Dolphin - Commits
- R318:1b985d1ef400: Add Documents shortcut detection
Run ctest within the dolphin build directory.
Diff Detail
Diff Detail
- Repository
- R318 Dolphin
- Branch
- add-documents (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 5061 Build 5079: arc lint + arc unit
Comment Actions
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. Looking for help to fix this.
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)]
Comment Actions
The patch looks fine and works for me: the upatched test fails, the patched one passes.
Are you sure you actually have installed kio with D16852 applied?