Closed Bug 1576893 Opened 6 years ago Closed 6 years ago

webrtc/RTCPeerConnection-iceConnectionState.https.html has an invalid test

Categories

(Core :: WebRTC: Signaling, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: bwc, Assigned: bwc)

References

Details

Attachments

(1 file)

The "Responder ICE connection state behaves as expected" test-case has been marked as intermittent on linux and android. Need to figure out why this is.

Type: defect → task

This looks like a bug in the test-case to me; it waits for ICE to complete on pc1, but not pc2, then asserts that pc2 has already reached ICE complete. The test case needs to wait for pc2 to reach ICE complete also.

Marking a defect because the test is buggy.

Type: task → defect
Summary: webrtc/RTCPeerConnection-iceConnectionState.https.html has an inconsistent test → webrtc/RTCPeerConnection-iceConnectionState.https.html has an invalid test

Looks like this change was backed out, but it will likely re-land. We'll have to wait for that to happen before landing this.

Pushed by csabou@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7134186c04b2 Wait for ICE to complete on pc2 before checking ICE state transitions. r=jib
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/18782 for changes under testing/web-platform/tests

The patch conflicted with the changes here: https://hg.mozilla.org/integration/autoland/rev/dcf08f0a3f22f0ead12e75e7bdda56c573dac71c so I rebased and landed it. Byron, if it's not ok with you I'll back it out.

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Upstream PR merged by moz-wptsync-bot
See Also: → 1578957
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: