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