-
-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Test Regression due to Make _search_nextjs_data non fatal #9763
Comments
can you elaborate on this? and how are you running the tests? what command and from which directory? |
Using API to get tarball from commit: https://github.com/yt-dlp/yt-dlp/tarball/ff38a011d57b763f3a69bebd25a5dc9044a717ce
|
Theoretically, tests should be run using We change the test to |
Perfect. Tests working again with that change. Issue can be closed when merged in master. |
Closes #9763 Authored by: bashonly, seproDev, Grub4K Co-authored-by: bashonly <88596187+bashonly@users.noreply.github.com> Co-authored-by: sepro <4618135+seproDev@users.noreply.github.com>
DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE
Checklist
Provide a description that is worded well enough to be understood
Test regression due to 3ee1194:
=================================== FAILURES ===================================
__________________ TestInfoExtractor.test_search_nextjs_data ___________________
self = <test.test_InfoExtractor.TestInfoExtractor testMethod=test_search_nextjs_data>
E AssertionError: DeprecationWarning not raised
test/test_InfoExtractor.py:1915: AssertionError
----------------------------- Captured stderr call -----------------------------
WARNING: [Dummy] unable to extract next.js data; please report this issue on https://github.com/yt
Provide verbose output that clearly demonstrates the problem
yt-dlp -vU <your command line>
)'verbose': True
toYoutubeDL
params instead[debug] Command-line config
) and insert it belowComplete Verbose Output
The text was updated successfully, but these errors were encountered: