Skip to content

Commit

Permalink
[test:download] Test for expected_exception
Browse files Browse the repository at this point in the history
Authored by: at-wat

Co-authored-by: Atsushi Watanabe <atsushi.w@ieee.org>
  • Loading branch information
bashonly and at-wat committed Sep 21, 2023
1 parent 568f080 commit 661c9a1
Showing 1 changed file with 20 additions and 0 deletions.
20 changes: 20 additions & 0 deletions test/test_download.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
DownloadError,
ExtractorError,
UnavailableVideoError,
YoutubeDLError,
format_bytes,
join_nonempty,
)
Expand Down Expand Up @@ -100,6 +101,8 @@ def print_skipping(reason):
print_skipping('IE marked as not _WORKING')

for tc in test_cases:
if tc.get('expected_exception'):
continue
info_dict = tc.get('info_dict', {})
params = tc.get('params', {})
if not info_dict.get('id'):
Expand Down Expand Up @@ -139,6 +142,17 @@ def get_tc_filename(tc):

res_dict = None

def match_exception(err):
expected_exception = test_case.get('expected_exception')
if not expected_exception:
return False
if err.__class__.__name__ == expected_exception:
return True
for exc in err.exc_info:
if exc.__class__.__name__ == expected_exception:
return True
return False

def try_rm_tcs_files(tcs=None):
if tcs is None:
tcs = test_cases
Expand All @@ -161,6 +175,8 @@ def try_rm_tcs_files(tcs=None):
except (DownloadError, ExtractorError) as err:
# Check if the exception is not a network related one
if not isinstance(err.exc_info[1], (TransportError, UnavailableVideoError)) or (isinstance(err.exc_info[1], HTTPError) and err.exc_info[1].status == 503):
if match_exception(err):
return
err.msg = f'{getattr(err, "msg", err)} ({tname})'
raise

Expand All @@ -171,6 +187,10 @@ def try_rm_tcs_files(tcs=None):
print(f'Retrying: {try_num} failed tries\n\n##########\n\n')

try_num += 1
except YoutubeDLError as err:
if match_exception(err):
return
raise
else:
break

Expand Down

0 comments on commit 661c9a1

Please sign in to comment.