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

New regression: FranceTVInfoIE #15577

Open
anisse opened this issue Feb 12, 2018 · 0 comments
Open

New regression: FranceTVInfoIE #15577

anisse opened this issue Feb 12, 2018 · 0 comments

Comments

@anisse
Copy link
Contributor

anisse commented Feb 12, 2018

Since 99892e9 , there's a new test failing in francetvinfo ie. It's not a big issue, just the description that needs to be updated. Previously the description was fetched in dailymotion's description, now it's from the webpage meta description tags. Here they differ, and I don't know which one is better.

ping @dstftw


FAIL: test_FranceTVInfo_1 (test.test_download.TestDownload) [Dailymotion]:
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/anisse/dev/youtube-dl/test/test_download.py", line 210, in test_template
    expect_info_dict(self, tc_res_dict, tc.get('info_dict', {}))
  File "/home/anisse/dev/youtube-dl/test/helper.py", line 177, in expect_info_dict
    expect_dict(self, got_dict, expected_dict)
  File "/home/anisse/dev/youtube-dl/test/helper.py", line 173, in expect_dict
    expect_value(self, got, expected, info_field)
  File "/home/anisse/dev/youtube-dl/test/helper.py", line 167, in expect_value
    'Invalid value for field %s, expected %r, got %r' % (field, expected, got))
AssertionError: Invalid value for field description, expected u'Au lendemain de la victoire du "oui" au r\xe9f\xe9rendum sur l\'a\xe9roport de Notre-Dame-des-Landes, l\'ancienne ministre \xe9cologiste est l\'invit\xe9e de Patrick Cohen. Plus d\'info : https://www.franceinter.fr/emissions/le-7-9/le-7-9-27-juin-2016', got u'Regardez NDDL, r\xe9f\xe9rendum, Brexit : C\xe9cile Duflot r\xe9pond \xe0 Patrick Cohen par France Inter sur Dailymotion'
-------------------- >> begin captured stdout << ---------------------
[francetvinfo.fr] video-sur-france-inter-cecile-duflot-denonce-le-regard-meprisant-de-patrick-cohen_1520091: Downloading webpage
[download] Downloading playlist: None
[francetvinfo.fr] playlist None: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[dailymotion] x4iiko0: Downloading webpage
[dailymotion] x4iiko0: Downloading m3u8 information
[dailymotion] x4iiko0: Downloading m3u8 information
[dailymotion] x4iiko0: Downloading m3u8 information
[dailymotion] x4iiko0: Downloading m3u8 information
[dailymotion] x4iiko0: Downloading m3u8 information
[info] Writing video description metadata as JSON to: test_FranceTVInfo_1_x4iiko0.info.json
[debug] Invoking downloader on u'https://www.dailymotion.com/cdn/H264-848x480/video/x4iiko0.mp4?auth=1518419634-2562-c21jdgzm-ef380cbd6ab7c6c3eca8c1aeaa4aba34'
[download] Destination: test_FranceTVInfo_1_x4iiko0.mp4
[download] 100% of 10.00KiB in 00:0097MiB/s ETA 00:21known ETA
[download] Finished downloading playlist: None

--------------------- >> end captured stdout << ----------------------
    """Fail immediately, with the given message."""
>>  raise self.failureException(u'Invalid value for field description, expected u\'Au lendemain de la victoire du "oui" au r\\xe9f\\xe9rendum sur l\\\'a\\xe9roport de Notre-Dame-des-Landes, l\\\'ancienne ministre \\xe9cologiste est l\\\'invit\\xe9e de Patrick Cohen. Plus d\\\'info : https://www.franceinter.fr/emissions/le-7-9/le-7-9-27-juin-2016\', got u\'Regardez NDDL, r\\xe9f\\xe9rendum, Brexit : C\\xe9cile Duflot r\\xe9pond \\xe0 Patrick Cohen par France Inter sur Dailymotion\'')
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant