[Kde-pim] Review Request 111755: Fix, update and re-enable IMAP resource unit-tests

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111755/
-----------------------------------------------------------

(Updated July 30, 2013, 10:58 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDEPIM and Kevin Ottens.


Description
-------

1) makes the tests compile
2) adds a new abstraction that allows us to move Akonadi::ItemFetchJob from RetrieveItemsTask to ImapResource and replace it by a dummy Job in unittests
3) fixes MoveItemsTask to pass tests
4) updates all unittests to pass (changed behavior in tasks)

For TestMoveItemsTask to pass, a patch in KIMAP is needed (will post in a new review)


Diffs
-----

  resources/imap/CMakeLists.txt 6498129 
  resources/imap/imapresource.h 83f6325 
  resources/imap/imapresource.cpp 0128eed 
  resources/imap/moveitemstask.cpp 5dc7021 
  resources/imap/resourcestate.h 87e9811 
  resources/imap/resourcestate.cpp 5d521a5 
  resources/imap/resourcestateinterface.h 7bfe5a3 
  resources/imap/resourcetask.h 4a121fb 
  resources/imap/resourcetask.cpp ce780cb 
  resources/imap/retrieveitemstask.h a3ba234 
  resources/imap/retrieveitemstask.cpp c82d465 
  resources/imap/tests/dummyresourcestate.h eb4b2ef 
  resources/imap/tests/dummyresourcestate.cpp d1e4e29 
  resources/imap/tests/testaddcollectiontask.cpp c8816d3 
  resources/imap/tests/testmoveitemtask.cpp 3bf67b2 
  resources/imap/tests/testremoveitemtask.cpp 0e61135 
  resources/imap/tests/testretrieveitemstask.cpp b542423 

Diff: http://git.reviewboard.kde.org/r/111755/diff/


Testing
-------

All tests pass


Thanks,

Dan Vrátil

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/