diff -r 6638e7f4bd8f -r 58332560b319 telephonyserverplugins/ctsydispatchlayer/test/dispatchertests/dispatchsrc/cctsycallcontrolmultipartyfu.cpp --- a/telephonyserverplugins/ctsydispatchlayer/test/dispatchertests/dispatchsrc/cctsycallcontrolmultipartyfu.cpp Thu May 06 15:10:38 2010 +0100 +++ b/telephonyserverplugins/ctsydispatchlayer/test/dispatchertests/dispatchsrc/cctsycallcontrolmultipartyfu.cpp Wed Jun 02 16:33:50 2010 +0100 @@ -146,6 +146,10 @@ User::WaitForRequest(hangupStatus); ASSERT_EQUALS(KErrNone, hangupStatus.Int()); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -227,6 +231,10 @@ ASSERT_EQUALS(KErrNone, mobileCall2.GetMobileCallStatus(callStatus)); ASSERT_EQUALS(RMobileCall::EStatusConnected, callStatus); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -307,6 +315,10 @@ ASSERT_EQUALS(KErrNone, mobileCall2.GetMobileCallStatus(callStatus)); ASSERT_EQUALS(RMobileCall::EStatusConnected, callStatus); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -395,6 +407,10 @@ ASSERT_EQUALS(KErrNone, notifyConfStatus.Int()); ASSERT_EQUALS(RMobileConferenceCall::EConferenceIdle, confStatus); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -646,6 +662,10 @@ ASSERT_EQUALS(KErrNone, confCall.GetConferenceStatus(confStatus)); ASSERT_EQUALS(RMobileConferenceCall::EConferenceHold, confStatus); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -742,6 +762,10 @@ User::WaitForRequest(addCallStatus); ASSERT_EQUALS(KErrGeneral, addCallStatus.Int()); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -870,6 +894,10 @@ ASSERT_EQUALS(KErrNone, mobileCall2.GetMobileCallStatus(callStatus)); ASSERT_EQUALS(RMobileCall::EStatusConnected, callStatus); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. mobileCall.Close(); User::After(50000); mobileCall2.Close(); @@ -1012,6 +1040,10 @@ ASSERT_EQUALS(KErrNone, mobileCall3.GetMobileCallStatus(callStatus)); ASSERT_EQUALS(RMobileCall::EStatusHold, callStatus); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. mobileCall.Close(); User::After(50000); mobileCall2.Close(); @@ -1180,6 +1212,10 @@ User::WaitForRequest(sendStatus); ASSERT_EQUALS(KErrNone, sendStatus.Int()); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close(); @@ -1307,6 +1343,10 @@ // Clean up when subsessions are closed DriverCloseTwoRemotePartyConferenceAndCallSubsessionsL(callId1, callId2, callIds); +// NOTE: Adopting User::After(50000) in here is because of test harness failure in the SMP (naviengine). +// The ETel server is closing a tel object by an Active Object (CAsyncOneShot), there can be out of ordered closes in SMP. +// Since the MockLtsy is expecting exact order of test procedure, this can be regarded as an error but it is not a part of intended testing. +// Therefore this waiting function calls are required to close the tel objects in perfect order. confCall.Close(); User::After(50000); mobileCall.Close();