[extractor/audiomack] Updated URL regex, corrected invalid testcases,…

[abdullah-if]

… fixed bug

Co-authored-by: dirkf fieldhouse@gmx.net

Please follow the guide below

• You will be asked some questions, please read them carefully and answer honestly
• Put an x into all the boxes [ ] relevant to your pull request (like that [x])
• Use Preview tab to see how your pull request will actually look like

---

Before submitting a pull request make sure you have:

• Searched the bugtracker for similar pull requests
• Read adding new extractor tutorial
• Read youtube-dl coding conventions and adjusted the code to meet them
• Covered the code with tests (note that PRs without tests will be REJECTED)
• Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

• I am the original author of this code and I am willing to release it under Unlicense
• I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

• Bug fix
• Improvement
• New extractor
• New feature

---

Description of your pull request and other information

Resolve #29800, resolve #30290, contains following:

• Updated URL regex. Both new and old URL are compatible.
• Updated test case. Number of song in playlist was lesser than expected.
• Updated test case. Playlist didn't contained test song. Added another.
• Disabled test case. Link broken. Add another. See New test case for audiomack wrapper around soundcloud song #29808
• Fixed bug. Convert id from integer to string

[dirkf]

LGTM

[dirkf]

@freedom4JA wrote:

Is there also a way for playlists? played around a bit but don't worked on my side.

Indeed, I considered making an AudiomackPlaylistIE and had these findings:

• playlists work the same as songs and albums with .../playlist/uploader/name being redirected to .../uploader/playlist/name

• a static html playlist page only contains unpopulated playlist JSON, so an API would be needed

• an obvious possible API endpoint corresponding to those used for songs and albums (www.audiomack.com/api/music/url/playlist/uploader/name) just redirects to the web page

• the page itself uses api.audiomack.com/v1/playlist/uploader/name which needs OAuth parameters

• the API endpoints used by the extractor for songs and albums don't seem to be documented anywhere now (only the .../v1/... endpoints as above).

Unless perhaps @heyitsxavier (who added the original API calls) can advise, an AudiomackPlaylistIE seems a bit tricky.