Review Request 127103: Add autotests for IconItem

David Rosca nowrep at
Wed Feb 17 17:37:08 UTC 2016

This is an automatically generated e-mail. To reply, visit:

Review request for Plasma.

Repository: plasma-framework


Add some tests for IconItem.
Also found one issue in animation when quickly changing source, first frame will be rendered with wrong icon.


  autotests/CMakeLists.txt 258adb8 
  autotests/data/icons/hicolor/22x22/apps/bug359388.svg PRE-CREATION 
  autotests/data/view.qml PRE-CREATION 
  autotests/iconitemtest.h PRE-CREATION 
  autotests/iconitemtest.cpp PRE-CREATION 
  src/declarativeimports/core/iconitem.cpp 085f284 



Tests pass localy, I used only icons that are shipped with plasma-framework breeze theme so it should work on CI too.

There is also test that checks if animation is not running after the item was hidden. This needs, so this should land only after that.


David Rosca

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the Plasma-devel mailing list