D5757: add requestToggleKeepAbove/below

[Martin Flöser]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170509/acc9bfbc/attachment.html>