qtmobility/plugins/contacts/symbiansim/tsrc/tst_simcmasync/tst_simcmasync.cpp
changeset 4 90517678cc4f
parent 1 2b40d63a9c3d
child 11 06b8e2af4411
--- a/qtmobility/plugins/contacts/symbiansim/tsrc/tst_simcmasync/tst_simcmasync.cpp	Fri Apr 16 15:51:22 2010 +0300
+++ b/qtmobility/plugins/contacts/symbiansim/tsrc/tst_simcmasync/tst_simcmasync.cpp	Mon May 03 13:18:40 2010 +0300
@@ -119,14 +119,14 @@
 {
     // remove all contacts
     QList<QContactLocalId> ids = m_cm->contactIds();
-    m_cm->removeContacts(&ids, 0);   
+    m_cm->removeContacts(ids, 0);   
 }
 
 void tst_SimCMAsync::cleanup()
 {
     // remove all contacts
     QList<QContactLocalId> ids = m_cm->contactIds();
-    m_cm->removeContacts(&ids, 0);   
+    m_cm->removeContacts(ids, 0);   
 }
 
 void tst_SimCMAsync::initTestCase()
@@ -230,7 +230,7 @@
    
     // Remove all contacts
     QList<QContactLocalId> ids = m_cm->contactIds();
-    m_cm->removeContacts(&ids, 0);    
+    m_cm->removeContacts(ids, 0);    
     
     // Test fetching nothing
     stateSpy.clear();
@@ -290,7 +290,7 @@
     
     // Remove all contacts
     QList<QContactLocalId> ids = m_cm->contactIds();
-    m_cm->removeContacts(&ids, 0);    
+    m_cm->removeContacts(ids, 0);    
     
     // Start again
     stateSpy.clear();
@@ -385,7 +385,7 @@
     QVERIFY(c.details(QContactPhoneNumber::DefinitionName).count() == 0);
     QVERIFY(c.detail<QContactName>().customLabel() == c1.detail<QContactName>().customLabel());
     QVERIFY(m_cm->contactIds().count() == 2);
-    c = m_cm->contact(c1.localId(), QStringList());
+    c = m_cm->contact(c1.localId());
     QVERIFY(c.details(QContactPhoneNumber::DefinitionName).count() == 0);
     QVERIFY(c.detail<QContactName>().customLabel() == c1.detail<QContactName>().customLabel());
 }
@@ -439,14 +439,14 @@
     QVERIFY(stateSpy.count() == 1);
     QTRY_COMPARE(resultSpy.count(), 1);
     QVERIFY(req.state() == QContactAbstractRequest::FinishedState);
-#ifndef __WINS__
-    QWARN("This test fails in hardware!");
-    QWARN("In hardware removing SIM contacts which do not exist the etel server will return KErrNone instead of KErrNotFound");
-#endif
-    QVERIFY(req.error() == QContactManager::DoesNotExistError);
-    QVERIFY(req.errorMap().count() == 2);
-    QVERIFY(req.errorMap().value(0) == QContactManager::DoesNotExistError);
-    QVERIFY(req.errorMap().value(1) == QContactManager::DoesNotExistError);
+    // NOTE: In emulator removing a nonexisting contact will return DoesNotExistError
+    // and in hardware there will be no error.
+    QVERIFY(req.error() == QContactManager::DoesNotExistError || req.error() == QContactManager::NoError);
+    if (req.errorMap().count()) {
+        QVERIFY(req.errorMap().count() == 2);
+        QVERIFY(req.errorMap().value(0) == QContactManager::DoesNotExistError);
+        QVERIFY(req.errorMap().value(1) == QContactManager::DoesNotExistError);
+    }
 }
 
 void tst_SimCMAsync::detailDefinitionFetchReq()