-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Iqiyi: test regression #9912
Comments
ping @yan12125 |
test_Iqiyi_2 is a VIP-only video and I haven't fixed it yet. |
Ah, so it's in fact failing (it's downloading the ad as said in the other bug ?) |
Yep it's falling. Unlike non-VIP videos, this one now returns nothing. |
There's another regression, probably introduced by 2644e91. See https://travis-ci.org/anisse/youtube-dl/jobs/142021660 Is it another VIP video? |
Apparently Iqiyi serves different files to Travis CI and my machine. |
Fixed by 3a212ed. |
Please follow the guide below
x
into all the boxes [ ] relevant to your issue (like that [x])Make sure you are using the latest version: run
youtube-dl --version
and ensure your version is 2016.06.25. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.Before submitting an issue make sure you have:
What is the purpose of your issue?
test_Iqiyi_2 test had a recently introduced regression:
https://travis-ci.org/anisse/youtube-dl/builds/140328781
This is the new test infrastructure discussed in #8496 that detected it (it has an auto-merge to trigger builds).
The text was updated successfully, but these errors were encountered: