From 0684b4cc11fbfc2f0eadeeadda303378394f8c05 Mon Sep 17 00:00:00 2001
From: Dominique Hazael-Massieux <dom@w3.org>
Date: Wed, 13 Dec 2023 00:59:50 +0000
Subject: [PATCH] Bug 1864845 [wpt PR 43171] - Add test for w3c/webrtc-pc#2687,
 a=testonly

Automatic update from web-platform-tests
Add test for w3c/webrtc-pc#2687 (#43171)

* Add test for w3c/webrtc-pc#2687

Check that a new data channel to an unbundling endpoint makes the caller connectionState go through 'connecting'

* Fix lint
--

wpt-commits: ade2b4b05020a2a1d4c0504f343571fb86d12668
wpt-pr: 43171
---
 ...CPeerConnection-connectionState.https.html | 44 +++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/testing/web-platform/tests/webrtc/RTCPeerConnection-connectionState.https.html b/testing/web-platform/tests/webrtc/RTCPeerConnection-connectionState.https.html
index d7716a1d4da19..b3884e4314d5a 100644
--- a/testing/web-platform/tests/webrtc/RTCPeerConnection-connectionState.https.html
+++ b/testing/web-platform/tests/webrtc/RTCPeerConnection-connectionState.https.html
@@ -271,6 +271,50 @@
     assert_array_equals(states, ['connecting', 'connected']);
   }, 'connectionState transitions to connected via connecting');
 
+
+  // Make the callee act as if not bundle-aware
+  async function exchangeOfferAnswerUnbundled(caller, callee) {
+    const offer = await caller.createOffer();
+    const sdp = offer.sdp.replace('BUNDLE', 'SOMETHING')
+          .replace(/rtp-hdrext:sdes/g, 'rtp-hdrext:something')
+          .replace(/a=ssrc:/g, 'a=notssrc');
+    await caller.setLocalDescription(offer);
+    await callee.setRemoteDescription({type: 'offer', sdp});
+
+    await exchangeAnswer(caller, callee);
+  }
+
+  promise_test(async t => {
+    const pc1 = new RTCPeerConnection({bundlePolicy: 'max-compat'});
+    t.add_cleanup(() => pc1.close());
+    const pc2 = new RTCPeerConnection();
+    t.add_cleanup(() => pc2.close());
+    const stream = await getNoiseStream({ audio: true });
+    t.add_cleanup(() => stream.getTracks().forEach(track => track.stop()));
+    stream.getTracks().forEach(track => pc1.addTrack(track, stream));
+    exchangeIceCandidates(pc1, pc2);
+    exchangeOfferAnswerUnbundled(pc1, pc2);
+    await listenToConnected(pc1);
+
+    // https://github.com/w3c/webrtc-pc/issues/2678#issuecomment-948554126
+    let had_intermediary_connecting = false
+    let channel;
+    const onConnectionStateChange = t.step_func(() => {
+      const {connectionState, iceConnectionState} = pc1;
+      if (connectionState === 'connecting') {
+        had_intermediary_connecting = true;
+      }
+    });
+
+    pc1.addEventListener('connectionstatechange', onConnectionStateChange);
+    channel = pc1.createDataChannel('test');
+    await exchangeOfferAnswer(pc1, pc2);
+    await listenToConnected(pc1);
+
+    assert_true(had_intermediary_connecting, "state should re-pass connecting before reaching connected");
+  }, 'when adding a datachannel to an existing unbundled connected PC, it should go through a connecting state');
+
+
   promise_test(async t => {
     const pc1 = new RTCPeerConnection();
     t.add_cleanup(() => pc1.close());