diff options
author | jiayl@webrtc.org <jiayl@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> | 2014-01-29 19:44:40 +0000 |
---|---|---|
committer | jiayl@webrtc.org <jiayl@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> | 2014-01-29 19:44:40 +0000 |
commit | d51e05ddea8c27cfc8f1e33fd4ef068534280bd5 (patch) | |
tree | 6497b770a89346464b5cf65151f350ab051a6ae3 /app | |
parent | 13a42bcb8b909faef285ec36b59740e5696a2295 (diff) | |
download | talk-d51e05ddea8c27cfc8f1e33fd4ef068534280bd5.tar.gz |
Disable a test assert which fails due to usrsctp not cleaned up in SctpDataEngine.cc
BUG=2749
R=fischman@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/7739005
git-svn-id: http://webrtc.googlecode.com/svn/trunk/talk@5460 4adac7df-926f-26a2-2b94-8c16560cd09d
Diffstat (limited to 'app')
-rw-r--r-- | app/webrtc/javatests/src/org/webrtc/PeerConnectionTest.java | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/app/webrtc/javatests/src/org/webrtc/PeerConnectionTest.java b/app/webrtc/javatests/src/org/webrtc/PeerConnectionTest.java index 03be05c..e81f75a 100644 --- a/app/webrtc/javatests/src/org/webrtc/PeerConnectionTest.java +++ b/app/webrtc/javatests/src/org/webrtc/PeerConnectionTest.java @@ -496,7 +496,7 @@ public class PeerConnectionTest extends TestCase { public void testCompleteSession() throws Exception { CountDownLatch testDone = new CountDownLatch(1); System.gc(); // Encourage any GC-related threads to start up. - TreeSet<String> threadsBeforeTest = allThreads(); + //TreeSet<String> threadsBeforeTest = allThreads(); PeerConnectionFactory factory = new PeerConnectionFactory(); // Uncomment to get ALL WebRTC tracing and SENSITIVE libjingle logging. @@ -718,8 +718,12 @@ public class PeerConnectionTest extends TestCase { videoSource.dispose(); factory.dispose(); System.gc(); - TreeSet<String> threadsAfterTest = allThreads(); - assertEquals(threadsBeforeTest, threadsAfterTest); + + // TODO(ldixon): the usrsctp threads are not cleaned up (issue 2749) and + // caused the assert to fail. We should reenable the assert once issue 2749 + // is fixed. + //TreeSet<String> threadsAfterTest = allThreads(); + //assertEquals(threadsBeforeTest, threadsAfterTest); Thread.sleep(100); } |