A ModelSpy cannot work without a model, and there is no point
re-setting it, so it should be set in the ctor only.
Details
Details
- Reviewers
dvratil - Group Reviewers
KDE PIM - Commits
- R165:438f1aa09d45: Move non-optional ModelSpy dependency to ctor
Diff Detail
Diff Detail
- Repository
- R165 Akonadi
- Branch
- model-spy-nonoptional-dependency
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 4385 Build 4403: arc lint + arc unit
Comment Actions
I would say that the ModelSpy should even be created on the stack because the way it is now it leaks from one test run to another (this is only destroyed after all tests methods are executed).
Would you mind changing that, since you are touching the tests?
Comment Actions
I plan to, but when I first tried, all the tests blew up, big time. Moving all those objects to the stack was one of the things I had done in my first try.