<table><tr><td style="">ahiemstra created this revision.<br />ahiemstra added a reviewer: Kirigami.<br />Herald added a project: Kirigami.<br />Herald added a subscriber: plasma-devel.<br />ahiemstra requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27521">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Apparently Kirigami's autotests have not run for a while. This<br />
patch fixes that and makes some other changes so the tests pass<br />
again and in the future it should be easier to add tests.</p>

<p>Changes:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Move add_subdirectory(autotests) to a location where BUILD_TESTING is defined so we actually can add the dir.</li>
<li class="remarkup-list-item">Add each test QML file as a separate test so reports are generated per test and tests can be run individually.</li>
<li class="remarkup-list-item">Add an autotest to test creation and rendering of Icon.</li>
<li class="remarkup-list-item">Fix most of the existing tests by making sure the window they rely on actually gets shown.</li>
<li class="remarkup-list-item">Fix PageRow test by changing it to matching current behaviour of PageRow.</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>With this, "make test" should work and pass.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>autotest_fixes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27521">https://phabricator.kde.org/D27521</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
autotests/CMakeLists.txt<br />
autotests/tst_icon.qml<br />
autotests/tst_keynavigation.qml<br />
autotests/tst_listskeynavigation.qml<br />
autotests/tst_pagerow.qml<br />
src/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>ahiemstra, Kirigami<br /><strong>Cc: </strong>plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart, hein<br /></div>