dfaure added inline comments. INLINE COMMENTS
> kfileplacesmodeltest.cpp:966 > QTest::newRow("Places - Trash") << m_places->index(2, 0) > - << QStringLiteral("user-trash-full"); > + << QStringLiteral("user-trash"); > QTest::newRow("Remote - Network") << m_places->index(3, 0) Why this change? It created a regression in the CI. Please debug fully. If this depends on your real-user's trash being full or not, then something's broken ;) QStandardPaths' test mode should make it look in a different config dir than ~/.config (where trashrc stores this information). http://ci-logs.kde.flaska.net/0b/0bc40d7c8b71af2c3861f5b39a4ebee91b046371/rebuilddep/rebuilddep-kf5-qt57-clang-el7/27e1666/shell_output.log REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D8366 To: franckarrecot, renatoo, ervin, mlaurent Cc: dfaure, anthonyfieroni, ervin, ngraham, mlaurent, #frameworks