Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SoundCloud] Fix extraction of tracks like count #1024

Merged
merged 2 commits into from
Jan 29, 2023

Conversation

AudricV
Copy link
Member

@AudricV AudricV commented Jan 29, 2023

  • I carefully read the contribution guidelines and agree to them.
  • I have tested the API against NewPipe. As SoundCloud tests still pass with my non-breaking API changes, everything should work fine in the app.

This PR fixes the extraction of SoundCloud likes. It uses the current property to get like counts and not the previous one (see #1017).

The corresponding stream like count tests have been updated to assert a value close to the real one.

Fixes #1017.

SoundCloud is using likes_count to return the like count of a track, like it
was the case before they switched to favoritings_count.
…tractorTest test classes

As like count is now returned by the extractor, we need to assert a positive
minimum like count, which is close to the actual value, in order to avoid test
failures due to lower like counts than the ones excepted.
@AudricV AudricV added bug soundcloud service, https://soundcloud.com/ labels Jan 29, 2023
@Stypox Stypox merged commit 999fb7f into TeamNewPipe:dev Jan 29, 2023
@AudricV AudricV deleted the snd_fix-tracks-like-count branch January 29, 2023 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug soundcloud service, https://soundcloud.com/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SoundCloud] Tracks like count extraction is broken
2 participants