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

Dan Vrátil dvratil at redhat.com
Mon Jul 29 17:19:05 BST 2013


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

(Updated July 29, 2013, 6:19 p.m.)


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 (updated)
-----

  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/CMakeLists.txt 295a666 
  resources/imap/tests/dummyresourcestate.h eb4b2ef 
  resources/imap/tests/dummyresourcestate.cpp d1e4e29 
  resources/imap/tests/testaddcollectiontask.cpp c8816d3 
  resources/imap/tests/testmoveitemstask.cpp PRE-CREATION 
  resources/imap/tests/testmoveitemtask.cpp 3bf67b2 
  resources/imap/tests/testremoveitemstask.cpp PRE-CREATION 
  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/


More information about the kde-pim mailing list