Skip to content
Commit aeeb5d46 authored by rk flx's avatar rk flx
Browse files

Adapt PlaceTreeModelTest autotest to KIO change

Summary:
After e8d799c9 needed to account for additional entries in
`KFilePlacesModel`, in KIO's 7e1d2fb84546 the number of default items
returned by the model was changed again, in particular "This Month" and
"Last Month" were removed.

As those entries are not that relevant for Gwenview too, we simply
follow along, meaning apart from changing the number expected in the
test no further changes are necessary.

If in the future more changes are planned in KIO, it might be worth
thinking about removing the test altogether, but for now it serves as a
canary, so any effect on Gwenview can be investigated in a timely
fashion.

Test Plan:
`./tests/auto/placetreemodeltest testListPlaces` does not fail anymore
with KIO 5.45, still works fine with KIO 5.44 (make sure CMake picks up
the change before recompiling).

Reviewers: #gwenview, ngraham

Reviewed By: #gwenview, ngraham

Subscribers: ngraham, huoni

Differential Revision: https://phabricator.kde.org/D12101
parent 49ae27e8
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment