Review Request 117475: kmimeassociationstest: do not use KDE apps in test
Alex Merry
alex.merry at kde.org
Fri Apr 25 10:03:23 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117475/
-----------------------------------------------------------
(Updated April 25, 2014, 10:03 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and David Faure.
Repository: kservice
Description
-------
kmimeassociationstest: do not use KDE apps in test
Looking for applications with a kde4- prefix will not be useful in the
long term, as those applications will disappear as people move away from
kdelibs4. However, simply stripping the kde4- prefix also does not work
because you can set a mime type assocation for "kwrite.desktop", and if
the KF5-based KWrite (which has the desktop ID "kwrite.desktop") is not
installed but the kdelibs4-based one (which has the desktop ID
"kde4-kwrite.desktop") is installed, the mime type will be associated
with kde4-kwrite.desktop, and the test will fail.
To avoid this issue, we avoid using KDE applications in the tests.
The prefixing stuff is explicitly tested with some fake application
desktop files. We still try adding and removing associations for real
applications (that may or may not be installed) to test cascading.
Diffs
-----
autotests/kmimeassociationstest.cpp d7b3ac29ca7292c0250286b241f20891c988bab6
Diff: https://git.reviewboard.kde.org/r/117475/diff/
Testing
-------
Test still passes.
Thanks,
Alex Merry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140425/24d3e353/attachment.html>
More information about the Kde-frameworks-devel
mailing list