w3c / webrtc-pc

WebRTC 1.0 API
https://w3c.github.io/webrtc-pc/
Other
436 stars 115 forks source link

Document test updates associated with amendments #2910

Closed dontcallmedom closed 6 months ago

dontcallmedom commented 10 months ago

This is a first pass at associating amendments to the relevant test cases and updates made to WPT as part of the amendment documenting process.

I have not documented associated test updates for the following PRs: #2758 #2759 #2794 #2797 #2801 #2847 #2853 #2861 - most likely because no updates were brought, but it could be that I missed them too.

It would be useful if someone could review that the annotations made in amendments.json are correct. Namely that:

Merging this PR will also require that any new amendment documents its test status (either as "already-tested", "not-testable", or as an array of pointers to WPT pull requests).


Preview | Diff

henbos commented 10 months ago

There are WebIDL validation steps failing, can you take a look @dontcallmedom ?

dontcallmedom commented 10 months ago

I did - I think they're bogus (since this PR doesn't touch the document at all), but in any case, I can spend more time on them once there is approval at least with the principles and the data :)

henbos commented 10 months ago

Ack, @jan-ivar to take a closer look, but free free to merge if he is happy about doing so

henbos commented 9 months ago

PTAL @jan-ivar :)

youennf commented 6 months ago

If the tool requires to put a test status in the amendment, we might need a 'to-be-added-shortly' state, as implementation might not be yet available.