I may need to add some sort of "#if KIO_VERSION < QT_VERSION_CHECK(5,60,0)" all over the place.
I have tried it but I haven't found similar patterns to base upon.
If I don't, it will make dolphin tests depends on kio master/5.60, unless this is tolerable.
I don't know the expectancies on tests.
Your call @elvisangelaccio.

In D21789#479548, @meven wrote:

I may need to add some sort of "#if KIO_VERSION < QT_VERSION_CHECK(5,60,0)" all over the place.
I have tried it but I haven't found similar patterns to base upon.
If I don't, it will make dolphin tests depends on kio master/5.60, unless this is tolerable.
I don't know the expectancies on tests.
Your call @elvisangelaccio.

Probably makes sense to bump the Frameworks dependency version instead (safe since we're still in master development), but yeah, it's @elvisangelaccio's call.

In D21789#480922, @elvisangelaccio wrote:

I'd avoid to bump the KF5 version only because of a test.

What we can do instead is to add the following check in src/tests/CMakeLists.txt :

if (KIO_VERSION VERSION_GREATER_EQUAL "5.60.0")
    ecm_add_test(placesitemmodeltest.cpp ...)
endif()

This is done, will wait for approval here before merging in KIO D15739.