graesslin added a comment.
In https://phabricator.kde.org/D5757#108115, @mart wrote: > In https://phabricator.kde.org/D5757#108070, @graesslin wrote: > > > Please also extend the test in autotests/client/test_wayland_windowmanagement.cpp > > > isn't it covered by no, that is a test for the Model. The test for the actual interface is in test_wayland_windowmanagement. That test goes more into the detail of how the interface works. Things we cannot test with the test for the model which only can cover one specific branch. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D5757 To: mart, #plasma, hein, graesslin, #plasma_on_wayland Cc: graesslin, hein, plasma-devel, #frameworks, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas