admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Disable P2PTestConductor.LocalP2PTestDtlsBundleInIceRestart under msan

Browse Source 
The test is flaky.

BUG=webrtc:6811
TBR=deadbeef@webrtc.org

Review-Url: https://codereview.webrtc.org/2546913003
Cr-Commit-Position: refs/heads/master@{#15382}
This commit is contained in:
henrik.lundin 2016-12-02 02:59:32 -08:00 committed by Commit bot
parent 5c71166dff
commit a28a1b9db6
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
webrtc/api/peerconnection_unittest.cc
View File

@ -1721,7 +1721,16 @@ TEST_F(P2PTestConductor, LocalP2PTestDtlsTransferCallee) {
// This test sets up a non-bundle call and apply bundle during ICE restart. When
// bundle is in effect in the restart, the channel can successfully reset its
// DTLS-SRTP context.
TEST_F(P2PTestConductor, LocalP2PTestDtlsBundleInIceRestart) {
#if defined(MEMORY_SANITIZER)
// Fails under MemorySanitizer:
// See https://bugs.chromium.org/p/webrtc/issues/detail?id=6811
#define MAYBE_LocalP2PTestDtlsBundleInIceRestart \
DISABLED_LocalP2PTestDtlsBundleInIceRestart
#else
#define MAYBE_LocalP2PTestDtlsBundleInIceRestart \
LocalP2PTestDtlsBundleInIceRestart
#endif
TEST_F(P2PTestConductor, MAYBE_LocalP2PTestDtlsBundleInIceRestart) {
MAYBE_SKIP_TEST(rtc::SSLStreamAdapter::HaveDtlsSrtp);
FakeConstraints setup_constraints;
setup_constraints.AddMandatory(MediaConstraintsInterface::kEnableDtlsSrtp,