Properly cleanup in notificationmanagertest
ClosedPublic

Authored by dfaure on May 10 2019, 3:42 PM.

Details

Summary

The call to "quit" is symmetrical with

QMetaObject::invokeMethod(&manager, "init", Qt::DirectConnection);

on line 56 of the test (due to NoThread mode).

But for the call to quit() to work, the base class has to not assert.

Test Plan

bin/notificationmanagertest in an ASAN build

Diff Detail

Repository
R165 Akonadi
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dfaure requested review of this revision.May 10 2019, 3:42 PM
dfaure created this revision.
dvratil accepted this revision.May 10 2019, 3:51 PM
This revision is now accepted and ready to land.May 10 2019, 3:51 PM
This revision was automatically updated to reflect the committed changes.
Restricted Application added a project: KDE PIM. · View Herald TranscriptMay 10 2019, 4:13 PM