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

Reply via email to