Looking at the implementation of QContactMemoryEngine and the expectations of tst_QContactManager::relationships()°°° I find the API dox of QContactManagerEngine::saveRelationships(...)° and QContactRelationshipSaveRequest°° to be way incomplete:

• does not mention that second contactId could have an empty managerUri and by this mean to also be from the local manager
• does not mention that if the second contactId has an empty managerUri that one should be set to the one of the local manager in the passed relationship object (i.e. the passed relationship(s) should be updated with that)

Please add to the API dox about this behaviour, so it is officially specified. Until now qtcontacts-tracker had a implementation not matching this, so tst_QContactManager::relationships() fails for it.

° http://doc.qt.nokia.com/qtmobility-1.2/qcontactmanagerengine.html#saveRelationship
°° http://doc.qt.nokia.com/qtmobility-1.2/qcontactrelationshipsaverequest.html
°°° https://www.qt.gitorious.org/qt-mobility/contacts/blobs/master/tests/auto/qcontactmanager/tst_qcontactmanager.cpp#line3288