From #2831944-227: Implement media source plugin for remote video via oEmbed:

YouTube videos are by far the most common use case for oEmbed, but we don't have any test fixtures for YouTube videos. Instead, we used Vimeo. We should open a follow-up to either test YouTube fixtures in addition to Vimeo, or just remove the Vimeo fixtures entirely in favor of YouTube.

It's debatable whether or not we should remove the Vimeo fixtures from the oEmbed tests, but it's indisputable that the top use case for oEmbed is YouTube videos. So we should at least add a YouTube fixture or two and test those.

Comments

phenaproxima created an issue. See original summary.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Title: [PP-1] Test oEmbed with a YouTube fixture » Test oEmbed with a YouTube fixture
Status: Postponed » Active

The main issue has landed, so let's take a look at this.

Version: 8.6.x-dev » 8.7.x-dev

Drupal 8.6.0-alpha1 will be released the week of July 16, 2018, which means new developments and disruptive changes should now be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.7.x-dev » 8.8.x-dev

Drupal 8.7.0-alpha1 will be released the week of March 11, 2019, which means new developments and disruptive changes should now be targeted against the 8.8.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Closed (outdated)

We test youtube in both \Drupal\Tests\media\FunctionalJavascript\MediaStandardProfileTest and \Drupal\Tests\media_library\FunctionalJavascript\MediaLibraryTest::testWidgetOEmbed