Remove dependency to AkonadiNotes and KMime
ClosedPublic

Authored by ervin on Tue, Jan 8, 4:27 PM.

Details

Summary

Keep the withNoteContent method in GenCollection and some of its use
though, it's useful in tests to simulate having collections containing
something else than tasks.

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin requested review of this revision.Tue, Jan 8, 4:27 PM
ervin created this revision.
dfaure added inline comments.Tue, Jan 8, 9:54 PM
tests/testlib/gencollection.cpp
100

that makes it QStringLiteral, I thought this was a crash risk.

ervin updated this revision to Diff 49045.Wed, Jan 9, 6:34 AM

Addresses David's comment.

Note it was likely harmless in this context since it's passed to a method
expecting a QString right away. But better safe than sorry indeed, we
don't know how that code or QString API might evolve in the future.

dfaure accepted this revision.Wed, Jan 9, 7:52 AM
This revision is now accepted and ready to land.Wed, Jan 9, 7:52 AM
This revision was automatically updated to reflect the committed changes.