sandsmark created this revision. sandsmark added reviewers: ngraham, Frameworks, broulik. sandsmark added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. sandsmark requested review of this revision.
REVISION SUMMARY Not entirely sure about this approach (and the stuff to get the test working), but I use kunitconversion in mangonel and the blocking behavior annoyed me a lot. A simpler approach would be to just try to check and download the cache immediately, but still force wait in the foreground, but the code (at least behavior) gets a lot more complex, especially with threading. TEST PLAN The tests pass, it gets the currencies fairly quickly but doesn't block the UI thread. REPOSITORY R292 KUnitConversion REVISION DETAIL https://phabricator.kde.org/D28770 AFFECTED FILES autotests/CMakeLists.txt autotests/convertertest.cpp autotests/convertertest.h autotests/valuetest.cpp autotests/valuetest.h src/currency.cpp src/currency_p.h To: sandsmark, ngraham, #frameworks, broulik Cc: broulik, ngraham, kde-frameworks-devel, #frameworks, LeGast00n, cblack, michaelh, bruns