[wp-trac] [WordPress Trac] #32360: Add oEmbed provider unit tests

[WordPress Trac]

#32360: Add oEmbed provider unit tests
-------------------------+------------------
 Reporter:  johnbillion  |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  4.4
Component:  Embeds       |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:
-------------------------+------------------
Changes (by johnbillion):

 * keywords:  needs-patch => has-patch
 * component:  Build/Test Tools => Embeds
 * milestone:  Future Release => 4.4


Comment:

 [attachment:32360.patch] introduces tests for the responses returned by
 each of our oEmbed providers. The tests which actually request the oEmbed
 for each of the test URLs is in the `external-oembed` group which doesn't
 run by default (these can be run with the `--group=oembed` or `--group
 =external-oembed` CLI option).

 Currently missing from the tests:

  * Test URL(s) for `poll.fm`.
  * Test URL(s) for `video214.com`.
  * Test URL(s) for `gi*.photobucket.com` (*cough* #28379 *cough*).

 Currently failing:

  * `scribd.com` is missing the `width` attribute.
  * `polldaddy.com` is missing the `width` attribute.
  * `meetup.com` returns an incorrect `content-type` header.
  * `animoto.com` returns an incorrect `content-type` header.
  * `tumblr.com` doesn't respect the `maxwidth` parameter for its `width`
 attribute.
  * `cloudup.com` doesn't respect the `maxwidth` parameter for its
 `thumbnail_width` attribute.
  * `reverbnation.com` doesn't adhere to the oEmbed specification (see
 #33207).

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32360#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform