Skip to content

Commit

Permalink
[test] Fix FakeYDL signatures
Browse files Browse the repository at this point in the history
Authored by: coletdjnz
  • Loading branch information
pukkandan committed Jun 21, 2022
1 parent 95032f3 commit f0500bd
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 13 deletions.
14 changes: 7 additions & 7 deletions test/helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ def try_rm(filename):
raise


def report_warning(message):
def report_warning(message, *args, **kwargs):
'''
Print the message to stderr, it will be prefixed with 'WARNING:'
If stderr is a tty file the 'WARNING:' will be colored
Expand All @@ -67,10 +67,10 @@ def __init__(self, override=None):
super().__init__(params, auto_init=False)
self.result = []

def to_screen(self, s, skip_eol=None):
def to_screen(self, s, *args, **kwargs):
print(s)

def trouble(self, s, tb=None):
def trouble(self, s, *args, **kwargs):
raise Exception(s)

def download(self, x):
Expand All @@ -80,10 +80,10 @@ def expect_warning(self, regex):
# Silence an expected warning matching a regex
old_report_warning = self.report_warning

def report_warning(self, message):
def report_warning(self, message, *args, **kwargs):
if re.match(regex, message):
return
old_report_warning(message)
old_report_warning(message, *args, **kwargs)
self.report_warning = types.MethodType(report_warning, self)


Expand Down Expand Up @@ -301,9 +301,9 @@ def assertEqual(self, got, expected, msg=None):
def expect_warnings(ydl, warnings_re):
real_warning = ydl.report_warning

def _report_warning(w):
def _report_warning(w, *args, **kwargs):
if not any(re.search(w_re, w) for w_re in warnings_re):
real_warning(w)
real_warning(w, *args, **kwargs)

ydl.report_warning = _report_warning

Expand Down
2 changes: 1 addition & 1 deletion test/test_YoutubeDL.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ def __init__(self, *args, **kwargs):
def process_info(self, info_dict):
self.downloaded_info_dicts.append(info_dict.copy())

def to_screen(self, msg):
def to_screen(self, msg, *args, **kwargs):
self.msgs.append(msg)

def dl(self, *args, **kwargs):
Expand Down
8 changes: 4 additions & 4 deletions test/test_cookies.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,16 @@


class Logger:
def debug(self, message):
def debug(self, message, *args, **kwargs):
print(f'[verbose] {message}')

def info(self, message):
def info(self, message, *args, **kwargs):
print(message)

def warning(self, message, only_once=False):
def warning(self, message, *args, **kwargs):
self.error(message)

def error(self, message):
def error(self, message, *args, **kwargs):
raise Exception(message)


Expand Down
2 changes: 1 addition & 1 deletion test/test_download.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ def __init__(self, *args, **kwargs):
self.processed_info_dicts = []
super().__init__(*args, **kwargs)

def report_warning(self, message):
def report_warning(self, message, *args, **kwargs):
# Don't accept warnings during tests
raise ExtractorError(message)

Expand Down

0 comments on commit f0500bd

Please sign in to comment.