Problem/Motivation

While testing a patch from #3208259: Migrating from Media Entity to Media in core creates a new source field where one already existed I've discovered
that soundcloud shared music URL in SoundcloudEmbedFormatterTest is broken.

Check the https://soundcloud.com/winguy/billie-jean-remix-ft link.

This track was not found. Maybe it has been removed

Steps to reproduce

Just run test locally with an old url:

php core/scripts/run-tests.sh --verbose --module media_entity_soundcloud --sqlite /tmp/drupal/test.sqlite --url http://localhost

Proposed resolution

Replace music URL with (for example) https://soundcloud.com/grouchnz/michael-jackson-billie-jeangrouch-rmx.
I've tried to find a good alternative URL with lot's of likes and shares to assume that it is stable enought :)

Remaining tasks

- Create patch.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

vlad.dancer created an issue. See original summary.

vlad.dancer’s picture

Added the patch.

The last submitted patch, 2: media_entity_soundcloud-dummy-test-change-3228191-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Status: Needs review » Needs work
vlad.dancer’s picture

Ops, previously added a wrong patch.
Here is a good one.

vlad.dancer’s picture

Status: Needs work » Needs review
rajeshreeputra’s picture

this is fixed with the latest release.

rajeshreeputra’s picture

Status: Needs review » Fixed
rajeshreeputra’s picture

Version: 8.x-2.0 » 3.x-dev
rajeshreeputra’s picture

Status: Fixed » Needs work
rajeshreeputra’s picture

Title: testSoundcloudEmbedFormatter is now broken due to wrong music URL » testSoundcloudEmbedFormatter is now broken due to wrong default theme

rajeshreeputra’s picture

Status: Needs work » Fixed

default theme in test set to 'claro' as stable is removed from D10

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.