upstream/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-05-28 04:34:11 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

correctly use getfullargspec

Browse source
This commit is contained in:
Erica Portnoy 2018-10-31 17:49:50 -07:00
parent 594fbe3ae8
commit a5e0e801be
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
certbot/tests/display/util_test.py
View file

@ -16,11 +16,6 @@ CHOICES = [("First", "Description1"), ("Second", "Description2")]
TAGS = ["tag1", "tag2", "tag3"]
TAGS_CHOICES = [("1", "tag1"), ("2", "tag2"), ("3", "tag3")]
if six.PY2:
getargspec = inspect.getargspec # pylint:disable=no-member
else:
getargspec = inspect.getfullargspec # pylint:disable=no-member
class InputWithTimeoutTest(unittest.TestCase):
"""Tests for certbot.display.util.input_with_timeout."""
@ -319,6 +314,10 @@ class FileOutputDisplayTest(unittest.TestCase):
# Every IDisplay method implemented by FileDisplay must take
# force_interactive to prevent workflow regressions.
for name in interfaces.IDisplay.names(): # pylint: disable=no-member
if six.PY2:
getargspec = inspect.getargspec
else:
getargspec = inspect.getfullargspec
arg_spec = getargspec(getattr(self.displayer, name))
self.assertTrue("force_interactive" in arg_spec.args)
@ -376,7 +375,10 @@ class NoninteractiveDisplayTest(unittest.TestCase):
for name in interfaces.IDisplay.names(): # pylint: disable=no-member
method = getattr(self.displayer, name)
# asserts method accepts arbitrary keyword arguments
self.assertFalse(getargspec(method).keywords is None)
if six.PY2:
self.assertFalse(inspect.getargspec(method).keywords is None)
else:
self.assertFalse(inspect.getfullargspec(method).varkw is None)
class SeparateListInputTest(unittest.TestCase):