D16780: Fix the way how we quit kactivitymanagerd from the unit test

Martin Flöser <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
graesslin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in activities_test.cpp:36
> unused

Are you sure?

  /home/martin/src/kde/workspace/kwin/autotests/integration/activities_test.cpp: In member function ‘void KWin::ActivitiesTest::cleanupTestCase()’:
  /home/martin/src/kde/workspace/kwin/autotests/integration/activities_test.cpp:87:32: error: invalid use of incomplete type ‘class QDBusPendingCall’
           QStringLiteral("quit")));
                                  ^
  In file included from /usr/include/x86_64-linux-gnu/qt5/QtDBus/QDBusConnection:1:0,
                   from /home/martin/src/kde/workspace/kwin/autotests/integration/activities_test.cpp:34:
  /usr/include/x86_64-linux-gnu/qt5/QtDBus/qdbusconnection.h:71:7: note: forward declaration of ‘class QDBusPendingCall’
   class QDBusPendingCall;
         ^~~~~~~~~~~~~~~~
  autotests/integration/CMakeFiles/testActivities.dir/build.make:62: recipe for target 'autotests/integration/CMakeFiles/testActivities.dir/activities_test.cpp.o' failed
  make[2]: *** [autotests/integration/CMakeFiles/testActivities.dir/activities_test.cpp.o] Error 1
  CMakeFiles/Makefile2:10944: recipe for target 'autotests/integration/CMakeFiles/testActivities.dir/all' failed
  make[1]: *** [autotests/integration/CMakeFiles/testActivities.dir/all] Error 2
  Makefile:140: recipe for target 'all' failed
  make: *** [all] Error 2
  *** Failure: Exit code 2 ***

asyncCall returns a QDBusPendingCall and without the include it doesn't compile.

REPOSITORY
  R108 KWin

BRANCH
  kill-activitymanager

REVISION DETAIL
  https://phabricator.kde.org/D16780

To: graesslin, #kwin, davidedmundson
Cc: davidedmundson, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.