D13813: make this test work again with new uds implementation
Jaime Torres Amate
noreply at phabricator.kde.org
Sat Jun 30 16:40:36 BST 2018
jtamate created this revision.
jtamate added reviewers: Dolphin, Frameworks.
Restricted Application added a project: Dolphin.
Restricted Application added a subscriber: kfm-devel.
jtamate requested review of this revision.
REVISION SUMMARY
With the new uds implementation, when some data is meant to replace an old one, it has to use replace() instead of insert() to avoid an assert of data already exists.
TEST PLAN
before: crash in the uds assert
after: passes the test
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D13813
AFFECTED FILES
src/tests/kfileitemmodeltest.cpp
To: jtamate, #dolphin, #frameworks
Cc: kfm-devel, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180630/5d7cfbc5/attachment.htm>
More information about the kfm-devel
mailing list