Merge pull request #4440 from qutebrowser/pyup-scheduled-update-2018-11-26
Scheduled weekly dependency update for week 47
This commit is contained in:
Florian Bruhin
GitHub
commit
63fa65df65
@ -3,6 +3,6 @@
|
||||
appdirs==1.4.3
|
||||
packaging==18.0
|
||||
pyparsing==2.3.0
|
||||
setuptools==40.5.0
|
||||
setuptools==40.6.2
|
||||
six==1.11.0
|
||||
wheel==0.32.2
|
||||
wheel==0.32.3
|
||||
|
||||
@ -1,11 +1,11 @@
|
||||
# This file is automatically generated by scripts/dev/recompile_requirements.py
|
||||
|
||||
asn1crypto==0.24.0
|
||||
astroid==2.0.4
|
||||
astroid==2.1.0
|
||||
certifi==2018.10.15
|
||||
cffi==1.11.5
|
||||
chardet==3.0.4
|
||||
cryptography==2.4.1
|
||||
cryptography==2.4.2
|
||||
github3.py==1.2.0
|
||||
idna==2.7
|
||||
isort==4.3.4
|
||||
@ -13,7 +13,7 @@ jwcrypto==0.6.0
|
||||
lazy-object-proxy==1.3.1
|
||||
mccabe==0.6.1
|
||||
pycparser==2.19
|
||||
pylint==2.1.1
|
||||
pylint==2.2.2
|
||||
python-dateutil==2.7.5
|
||||
./scripts/dev/pylint_checkers
|
||||
requests==2.20.1
|
||||
|
||||
@ -6,13 +6,12 @@ backports.functools-lru-cache==1.5
|
||||
beautifulsoup4==4.6.3
|
||||
cheroot==6.5.2
|
||||
Click==7.0
|
||||
# colorama==0.4.0
|
||||
# colorama==0.4.1
|
||||
coverage==4.5.2
|
||||
EasyProcess==0.2.3
|
||||
fields==5.0.0
|
||||
Flask==1.0.2
|
||||
glob2==0.6
|
||||
hunter==2.0.2
|
||||
hunter==2.1.0
|
||||
hypothesis==3.82.1
|
||||
itsdangerous==1.1.0
|
||||
# Jinja2==2.10
|
||||
@ -24,7 +23,7 @@ parse-type==0.4.2
|
||||
pluggy==0.8.0
|
||||
py==1.7.0
|
||||
py-cpuinfo==4.0.0
|
||||
pytest==4.0.0
|
||||
pytest==4.0.1
|
||||
pytest-bdd==3.0.0
|
||||
pytest-benchmark==3.1.1
|
||||
pytest-cov==2.6.0
|
||||
|
||||
@ -193,7 +193,7 @@ def _init_icon():
|
||||
icon = QIcon()
|
||||
fallback_icon = QIcon()
|
||||
for size in [16, 24, 32, 48, 64, 96, 128, 256, 512]:
|
||||
filename = ':/icons/qutebrowser-{}x{}.png'.format(size, size)
|
||||
filename = ':/icons/qutebrowser-{size}x{size}.png'.format(size=size)
|
||||
pixmap = QPixmap(filename)
|
||||
if pixmap.isNull():
|
||||
log.init.warning("Failed to load {}".format(filename))
|
||||
|
||||
@ -64,8 +64,6 @@ class UnsupportedAttribute:
|
||||
supported with QtWebengine.
|
||||
"""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class UnsupportedOperationError(Exception):
|
||||
|
||||
|
||||
@ -49,8 +49,6 @@ class WebInspectorError(Exception):
|
||||
|
||||
"""Raised when the inspector could not be initialized."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class AbstractWebInspector(QWidget):
|
||||
|
||||
|
||||
@ -35,15 +35,11 @@ class ParseProxyError(Exception):
|
||||
|
||||
"""Error while parsing PAC result string."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class EvalProxyError(Exception):
|
||||
|
||||
"""Error while evaluating PAC script."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def _js_slot(*args):
|
||||
"""Wrap a methods as a JavaScript function.
|
||||
|
||||
@ -61,36 +61,26 @@ class Error(Exception):
|
||||
|
||||
"""Exception for generic errors on a qute:// page."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class NotFoundError(Error):
|
||||
|
||||
"""Raised when the given URL was not found."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class SchemeOSError(Error):
|
||||
|
||||
"""Raised when there was an OSError inside a handler."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class UrlInvalidError(Error):
|
||||
|
||||
"""Raised when an invalid URL was opened."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class RequestDeniedError(Error):
|
||||
|
||||
"""Raised when the request is forbidden."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class Redirect(Exception):
|
||||
|
||||
|
||||
@ -43,29 +43,21 @@ class Error(Exception):
|
||||
|
||||
"""Base class for all errors in this module."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class InvalidUrlError(Error):
|
||||
|
||||
"""Exception emitted when a URL is invalid."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class DoesNotExistError(Error):
|
||||
|
||||
"""Exception emitted when a given URL does not exist."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class AlreadyExistsError(Error):
|
||||
|
||||
"""Exception emitted when a given URL does already exist."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class UrlMarkManager(QObject):
|
||||
|
||||
|
||||
@ -34,15 +34,11 @@ class Error(Exception):
|
||||
|
||||
"""Base class for WebElement errors."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class OrphanedError(Error):
|
||||
|
||||
"""Raised when a webelement's parent has vanished."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def css_selector(group, url):
|
||||
"""Get a CSS selector for the given group/URL."""
|
||||
|
||||
@ -117,8 +117,7 @@ class DownloadItem(downloads.AbstractDownloadItem):
|
||||
def _get_open_filename(self):
|
||||
return self._filename
|
||||
|
||||
def _set_fileobj(self, fileobj, *,
|
||||
autoclose=True): # pylint: disable=unused-argument
|
||||
def _set_fileobj(self, fileobj, *, autoclose=True):
|
||||
raise downloads.UnsupportedOperationError
|
||||
|
||||
def _set_tempfile(self, fileobj):
|
||||
|
||||
@ -516,7 +516,6 @@ class _NoCloseBytesIO(io.BytesIO):
|
||||
|
||||
def close(self):
|
||||
"""Do nothing."""
|
||||
pass
|
||||
|
||||
def actual_close(self):
|
||||
"""Close the stream."""
|
||||
|
||||
@ -67,7 +67,6 @@ class FixedDataNetworkReply(QNetworkReply):
|
||||
@pyqtSlot()
|
||||
def abort(self):
|
||||
"""Abort the operation."""
|
||||
pass
|
||||
|
||||
def bytesAvailable(self):
|
||||
"""Determine the bytes available for being read.
|
||||
@ -123,7 +122,6 @@ class ErrorNetworkReply(QNetworkReply):
|
||||
|
||||
def abort(self):
|
||||
"""Do nothing since it's a fake reply."""
|
||||
pass
|
||||
|
||||
def bytesAvailable(self):
|
||||
"""We always have 0 bytes available."""
|
||||
@ -151,7 +149,6 @@ class RedirectNetworkReply(QNetworkReply):
|
||||
|
||||
def abort(self):
|
||||
"""Called when there's e.g. a redirection limit."""
|
||||
pass
|
||||
|
||||
def readData(self, _maxlen):
|
||||
return bytes()
|
||||
|
||||
@ -31,8 +31,6 @@ class IsNullError(webelem.Error):
|
||||
|
||||
"""Gets raised by WebKitElement if an element is null."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class WebKitElement(webelem.AbstractWebElement):
|
||||
|
||||
|
||||
@ -41,7 +41,6 @@ class WebHistoryInterface(QWebHistoryInterface):
|
||||
|
||||
def addHistoryEntry(self, url_string):
|
||||
"""Required for a QWebHistoryInterface impl, obsoleted by add_url."""
|
||||
pass
|
||||
|
||||
@functools.lru_cache(maxsize=32768)
|
||||
def historyContains(self, url_string):
|
||||
|
||||
@ -32,22 +32,16 @@ class CommandError(Error):
|
||||
|
||||
"""Raised when a command encounters an error while running."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class NoSuchCommandError(Error):
|
||||
|
||||
"""Raised when a command wasn't found."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class ArgumentTypeError(Error):
|
||||
|
||||
"""Raised when an argument had an invalid type."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class PrerequisitesError(Error):
|
||||
|
||||
@ -56,5 +50,3 @@ class PrerequisitesError(Error):
|
||||
This is raised for example when we're in the wrong mode while executing the
|
||||
command, or we need javascript enabled but don't have done so.
|
||||
"""
|
||||
|
||||
pass
|
||||
|
||||
@ -28,8 +28,6 @@ class Error(Exception):
|
||||
|
||||
"""Base exception for config-related errors."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class NoAutoconfigError(Error):
|
||||
|
||||
|
||||
@ -25,5 +25,3 @@ from qutebrowser.mainwindow.statusbar import textbase
|
||||
class KeyString(textbase.TextBase):
|
||||
|
||||
"""Keychain string displayed in the statusbar."""
|
||||
|
||||
pass
|
||||
|
||||
@ -284,7 +284,7 @@ class BackendImports:
|
||||
|
||||
def _try_import_backends():
|
||||
"""Check whether backends can be imported and return BackendImports."""
|
||||
# pylint: disable=unused-variable
|
||||
# pylint: disable=unused-import
|
||||
results = BackendImports()
|
||||
|
||||
try:
|
||||
|
||||
@ -29,15 +29,11 @@ class HistoryEmptyError(Exception):
|
||||
|
||||
"""Raised when the history is empty."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class HistoryEndReachedError(Exception):
|
||||
|
||||
"""Raised when the end of the history is reached."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class History(QObject):
|
||||
|
||||
|
||||
@ -201,7 +201,6 @@ class _CrashDialog(QDialog):
|
||||
|
||||
def _init_checkboxes(self):
|
||||
"""Initialize the checkboxes."""
|
||||
pass
|
||||
|
||||
def _init_buttons(self):
|
||||
"""Initialize the buttons."""
|
||||
@ -573,7 +572,6 @@ class ReportDialog(_CrashDialog):
|
||||
|
||||
def _init_info_text(self):
|
||||
"""We don't want an info text as the user wanted to report."""
|
||||
pass
|
||||
|
||||
def _get_error_type(self):
|
||||
return 'report'
|
||||
|
||||
@ -133,7 +133,7 @@ def init_faulthandler(fileobj=sys.__stderr__):
|
||||
def check_pyqt_core():
|
||||
"""Check if PyQt core is installed."""
|
||||
try:
|
||||
import PyQt5.QtCore # pylint: disable=unused-variable
|
||||
import PyQt5.QtCore # pylint: disable=unused-import
|
||||
except ImportError as e:
|
||||
text = _missing_str('PyQt5')
|
||||
text = text.replace('<b>', '')
|
||||
@ -187,9 +187,8 @@ def check_qt_version():
|
||||
|
||||
def check_ssl_support():
|
||||
"""Check if SSL support is available."""
|
||||
# pylint: disable=unused-variable
|
||||
try:
|
||||
from PyQt5.QtNetwork import QSslSocket
|
||||
from PyQt5.QtNetwork import QSslSocket # pylint: disable=unused-import
|
||||
except ImportError:
|
||||
_die("Fatal error: Your Qt is built without SSL support.")
|
||||
|
||||
|
||||
@ -72,8 +72,6 @@ class SqlEnvironmentError(SqlError):
|
||||
disk or I/O errors), where qutebrowser isn't to blame.
|
||||
"""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class SqlBugError(SqlError):
|
||||
|
||||
@ -82,8 +80,6 @@ class SqlBugError(SqlError):
|
||||
This is raised for errors resulting from a qutebrowser bug.
|
||||
"""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def raise_sqlite_error(msg, error):
|
||||
"""Raise either a SqlBugError or SqlEnvironmentError."""
|
||||
|
||||
@ -373,7 +373,6 @@ def window_only(current_win_id):
|
||||
@cmdutils.register()
|
||||
def nop():
|
||||
"""Do nothing."""
|
||||
pass
|
||||
|
||||
|
||||
@cmdutils.register()
|
||||
|
||||
@ -42,8 +42,6 @@ class RegistryUnavailableError(Exception):
|
||||
|
||||
"""Exception raised when a certain registry does not exist yet."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class NoWindow(Exception):
|
||||
|
||||
|
||||
@ -285,7 +285,7 @@ class PyQIODevice(io.BufferedIOBase):
|
||||
if not ok:
|
||||
raise QtOSError(self.dev, msg="seek failed!")
|
||||
|
||||
def truncate(self, size=None): # pylint: disable=unused-argument
|
||||
def truncate(self, size=None):
|
||||
raise io.UnsupportedOperation
|
||||
|
||||
@property
|
||||
|
||||
@ -631,7 +631,6 @@ def open_file(filename, cmdline=None):
|
||||
|
||||
def unused(_arg):
|
||||
"""Function which does nothing to avoid pylint complaining."""
|
||||
pass
|
||||
|
||||
|
||||
def expand_windows_drive(path):
|
||||
|
||||
@ -1,10 +1,10 @@
|
||||
# This file is automatically generated by scripts/dev/recompile_requirements.py
|
||||
|
||||
attrs==18.2.0
|
||||
colorama==0.4.0
|
||||
colorama==0.4.1
|
||||
cssutils==1.0.2
|
||||
Jinja2==2.10
|
||||
MarkupSafe==1.1.0
|
||||
Pygments==2.2.0
|
||||
Pygments==2.3.0
|
||||
pyPEG2==2.15.2
|
||||
PyYAML==3.13
|
||||
|
||||
@ -370,7 +370,7 @@ def main():
|
||||
if args.upload is not None:
|
||||
# Fail early when trying to upload without github3 installed
|
||||
# or without API token
|
||||
import github3 # pylint: disable=unused-variable
|
||||
import github3 # pylint: disable=unused-import
|
||||
read_github_token()
|
||||
|
||||
if args.no_asciidoc:
|
||||
|
||||
@ -135,7 +135,7 @@ def _get_command_quickref(cmds):
|
||||
out.append('|Command|Description')
|
||||
for name, cmd in cmds:
|
||||
desc = inspect.getdoc(cmd.handler).splitlines()[0]
|
||||
out.append('|<<{},{}>>|{}'.format(name, name, desc))
|
||||
out.append('|<<{name},{name}>>|{desc}'.format(name=name, desc=desc))
|
||||
out.append('|==============')
|
||||
return '\n'.join(out)
|
||||
|
||||
|
||||
@ -33,8 +33,6 @@ class Error(Exception):
|
||||
|
||||
"""Exception for errors in this module."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def parse():
|
||||
"""Parse command line arguments."""
|
||||
|
||||
@ -34,8 +34,6 @@ class Error(Exception):
|
||||
|
||||
"""Exception raised when linking fails."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def run_py(executable, *code):
|
||||
"""Run the given python code with the given executable."""
|
||||
|
||||
@ -206,7 +206,6 @@ def pytest_configure(config):
|
||||
webengine_env = os.environ.get('QUTE_BDD_WEBENGINE', '')
|
||||
config.webengine = bool(webengine_arg or webengine_env)
|
||||
# Fail early if QtWebEngine is not available
|
||||
# pylint: disable=unused-variable
|
||||
if config.webengine:
|
||||
import PyQt5.QtWebEngineWidgets
|
||||
|
||||
@ -283,7 +282,7 @@ def apply_fake_os(monkeypatch, request):
|
||||
def check_yaml_c_exts():
|
||||
"""Make sure PyYAML C extensions are available on Travis."""
|
||||
if 'TRAVIS' in os.environ:
|
||||
from yaml import CLoader # pylint: disable=unused-variable
|
||||
from yaml import CLoader
|
||||
|
||||
|
||||
@pytest.hookimpl(tryfirst=True, hookwrapper=True)
|
||||
|
||||
@ -842,8 +842,6 @@ class YamlLoader(yaml.SafeLoader):
|
||||
|
||||
"""Custom YAML loader used in compare_session."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
# Translate ... to ellipsis in YAML.
|
||||
YamlLoader.add_constructor('!ellipsis', lambda loader, node: ...)
|
||||
|
||||
@ -39,15 +39,11 @@ class InvalidLine(Exception):
|
||||
|
||||
"""Raised when the process prints a line which is not parsable."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class ProcessExited(Exception):
|
||||
|
||||
"""Raised when the child process did exit."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class WaitForTimeout(Exception):
|
||||
|
||||
@ -271,7 +267,6 @@ class Process(QObject):
|
||||
|
||||
def _after_start(self):
|
||||
"""Do things which should be done immediately after starting."""
|
||||
pass
|
||||
|
||||
def before_test(self):
|
||||
"""Restart process before a test if it exited before."""
|
||||
@ -443,7 +438,6 @@ class Process(QObject):
|
||||
QuteProc._maybe_skip, and call _maybe_skip after every parsed message
|
||||
in wait_for (where it's most likely that new messages arrive).
|
||||
"""
|
||||
pass
|
||||
|
||||
def wait_for(self, timeout=None, *, override_waited_for=False,
|
||||
do_skip=False, divisor=1, after=None, **kwargs):
|
||||
|
||||
@ -82,7 +82,8 @@ class Request(testprocess.Line):
|
||||
for i in range(15):
|
||||
path_to_statuses['/redirect/{}'.format(i)] = [HTTPStatus.FOUND]
|
||||
for suffix in ['', '1', '2', '3', '4', '5', '6']:
|
||||
key = '/basic-auth/user{}/password{}'.format(suffix, suffix)
|
||||
key = ('/basic-auth/user{suffix}/password{suffix}'
|
||||
.format(suffix=suffix))
|
||||
path_to_statuses[key] = [HTTPStatus.UNAUTHORIZED, HTTPStatus.OK]
|
||||
|
||||
default_statuses = [HTTPStatus.OK, HTTPStatus.NOT_MODIFIED]
|
||||
|
||||
@ -306,7 +306,6 @@ class FakeSignal:
|
||||
Currently does nothing, but could be improved to do some sanity
|
||||
checking on the slot.
|
||||
"""
|
||||
pass
|
||||
|
||||
def disconnect(self, slot=None):
|
||||
"""Disconnect the signal from a slot.
|
||||
@ -314,7 +313,6 @@ class FakeSignal:
|
||||
Currently does nothing, but could be improved to do some sanity
|
||||
checking on the slot and see if it actually got connected.
|
||||
"""
|
||||
pass
|
||||
|
||||
def emit(self, *args):
|
||||
"""Emit the signal.
|
||||
@ -322,7 +320,6 @@ class FakeSignal:
|
||||
Currently does nothing, but could be improved to do type checking based
|
||||
on a signature given to __init__.
|
||||
"""
|
||||
pass
|
||||
|
||||
|
||||
@attr.s
|
||||
@ -457,8 +454,6 @@ class BookmarkManagerStub(UrlMarkManagerStub):
|
||||
|
||||
"""Stub for the bookmark-manager object."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class QuickmarkManagerStub(UrlMarkManagerStub):
|
||||
|
||||
|
||||
@ -82,14 +82,14 @@ def test_generate_pdfjs_script_disable_object_url(monkeypatch,
|
||||
if qt == 'new':
|
||||
monkeypatch.setattr(pdfjs.qtutils, 'version_check',
|
||||
lambda version, exact=False, compiled=True:
|
||||
False if version == '5.7.1' else True)
|
||||
version != '5.7.1')
|
||||
elif qt == 'old':
|
||||
monkeypatch.setattr(pdfjs.qtutils, 'version_check',
|
||||
lambda version, exact=False, compiled=True: False)
|
||||
elif qt == '5.7':
|
||||
monkeypatch.setattr(pdfjs.qtutils, 'version_check',
|
||||
lambda version, exact=False, compiled=True:
|
||||
True if version == '5.7.1' else False)
|
||||
version == '5.7.1')
|
||||
else:
|
||||
raise utils.Unreachable
|
||||
|
||||
|
||||
@ -31,7 +31,7 @@ from qutebrowser.browser.webkit import http, rfc6266
|
||||
'attachment; filename="{}"',
|
||||
'inline; {}',
|
||||
'attachment; {}="foo"',
|
||||
'attachment; filename*=iso-8859-1''{}',
|
||||
"attachment; filename*=iso-8859-1''{}",
|
||||
'attachment; filename*={}',
|
||||
])
|
||||
@hypothesis.given(strategies.text(alphabet=[chr(x) for x in range(255)]))
|
||||
|
||||
@ -48,7 +48,6 @@ def _get_cmd(*args, **kwargs):
|
||||
@cmdutils.register(*args, **kwargs)
|
||||
def fun():
|
||||
"""Blah."""
|
||||
pass
|
||||
return cmdutils.cmd_dict['fun']
|
||||
|
||||
|
||||
@ -83,7 +82,6 @@ class TestRegister:
|
||||
@cmdutils.register()
|
||||
def fun():
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
cmd = cmdutils.cmd_dict['fun']
|
||||
assert cmd.handler is fun
|
||||
@ -95,7 +93,6 @@ class TestRegister:
|
||||
@cmdutils.register()
|
||||
def eggs_bacon():
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
assert cmdutils.cmd_dict['eggs-bacon'].name == 'eggs-bacon'
|
||||
assert 'eggs_bacon' not in cmdutils.cmd_dict
|
||||
@ -105,7 +102,6 @@ class TestRegister:
|
||||
@cmdutils.register()
|
||||
def Test(): # noqa: N801,N806 pylint: disable=invalid-name
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
assert cmdutils.cmd_dict['test'].name == 'test'
|
||||
assert 'Test' not in cmdutils.cmd_dict
|
||||
@ -115,7 +111,6 @@ class TestRegister:
|
||||
@cmdutils.register(name='foobar')
|
||||
def fun():
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
assert cmdutils.cmd_dict['foobar'].name == 'foobar'
|
||||
assert 'fun' not in cmdutils.cmd_dict
|
||||
@ -126,20 +121,17 @@ class TestRegister:
|
||||
@cmdutils.register(name='foobar')
|
||||
def fun():
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
with pytest.raises(ValueError):
|
||||
@cmdutils.register(name='foobar')
|
||||
def fun2():
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
def test_instance(self):
|
||||
"""Make sure the instance gets passed to Command."""
|
||||
@cmdutils.register(instance='foobar')
|
||||
def fun(self):
|
||||
"""Blah."""
|
||||
pass
|
||||
assert cmdutils.cmd_dict['fun']._instance == 'foobar'
|
||||
|
||||
def test_star_args(self):
|
||||
@ -147,7 +139,6 @@ class TestRegister:
|
||||
@cmdutils.register()
|
||||
def fun(*args):
|
||||
"""Blah."""
|
||||
pass
|
||||
with pytest.raises(argparser.ArgumentParserError):
|
||||
cmdutils.cmd_dict['fun'].parser.parse_args([])
|
||||
|
||||
@ -195,14 +186,12 @@ class TestRegister:
|
||||
@cmdutils.argument('arg1', flag='b')
|
||||
def fun(arg1=False, arg2=False):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
def test_win_id(self):
|
||||
@cmdutils.register()
|
||||
@cmdutils.argument('win_id', win_id=True)
|
||||
def fun(win_id):
|
||||
"""Blah."""
|
||||
pass
|
||||
assert cmdutils.cmd_dict['fun']._get_call_args(42) == ([42], {})
|
||||
|
||||
def test_count(self):
|
||||
@ -210,7 +199,6 @@ class TestRegister:
|
||||
@cmdutils.argument('count', count=True)
|
||||
def fun(count=0):
|
||||
"""Blah."""
|
||||
pass
|
||||
assert cmdutils.cmd_dict['fun']._get_call_args(42) == ([0], {})
|
||||
|
||||
def test_count_without_default(self):
|
||||
@ -220,7 +208,6 @@ class TestRegister:
|
||||
@cmdutils.argument('count', count=True)
|
||||
def fun(count):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
@pytest.mark.parametrize('hide', [True, False])
|
||||
def test_pos_args(self, hide):
|
||||
@ -228,7 +215,6 @@ class TestRegister:
|
||||
@cmdutils.argument('arg', hide=hide)
|
||||
def fun(arg):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
pos_args = cmdutils.cmd_dict['fun'].pos_args
|
||||
if hide:
|
||||
@ -283,7 +269,6 @@ class TestRegister:
|
||||
@cmdutils.argument('arg', choices=['foo', 'bar'])
|
||||
def fun(arg):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
cmd = cmdutils.cmd_dict['fun']
|
||||
cmd.namespace = cmd.parser.parse_args(['fish'])
|
||||
@ -297,7 +282,6 @@ class TestRegister:
|
||||
@cmdutils.argument('arg', choices=['foo', 'bar'])
|
||||
def fun(*, arg='foo'):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
cmd = cmdutils.cmd_dict['fun']
|
||||
cmd.namespace = cmd.parser.parse_args(['--arg=fish'])
|
||||
@ -312,7 +296,6 @@ class TestRegister:
|
||||
@cmdutils.argument('opt')
|
||||
def fun(foo, bar, opt=False):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
cmd = cmdutils.cmd_dict['fun']
|
||||
assert cmd.get_pos_arg_info(0) == command.ArgInfo(choices=('a', 'b'))
|
||||
@ -324,7 +307,6 @@ class TestRegister:
|
||||
# https://github.com/qutebrowser/qutebrowser/issues/1872
|
||||
def fun(*, target):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
with pytest.raises(TypeError, match="fun: handler has keyword only "
|
||||
"argument 'target' without default!"):
|
||||
@ -334,7 +316,6 @@ class TestRegister:
|
||||
# https://github.com/qutebrowser/qutebrowser/issues/1872
|
||||
def fun(*, target: int):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
with pytest.raises(TypeError, match="fun: handler has keyword only "
|
||||
"argument 'target' without default!"):
|
||||
@ -350,14 +331,12 @@ class TestArgument:
|
||||
@cmdutils.argument('foo')
|
||||
def fun(bar):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
def test_storage(self):
|
||||
@cmdutils.argument('foo', flag='x')
|
||||
@cmdutils.argument('bar', flag='y')
|
||||
def fun(foo, bar):
|
||||
"""Blah."""
|
||||
pass
|
||||
expected = {
|
||||
'foo': command.ArgInfo(flag='x'),
|
||||
'bar': command.ArgInfo(flag='y')
|
||||
@ -372,7 +351,6 @@ class TestArgument:
|
||||
@cmdutils.register()
|
||||
def fun(bar):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
def test_count_and_win_id_same_arg(self):
|
||||
with pytest.raises(TypeError,
|
||||
@ -380,7 +358,6 @@ class TestArgument:
|
||||
@cmdutils.argument('arg', count=True, win_id=True)
|
||||
def fun(arg=0):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
def test_no_docstring(self, caplog):
|
||||
with caplog.at_level(logging.WARNING):
|
||||
@ -388,6 +365,7 @@ class TestArgument:
|
||||
def fun():
|
||||
# no docstring
|
||||
pass
|
||||
|
||||
assert len(caplog.records) == 1
|
||||
assert caplog.messages[0].endswith('test_cmdutils.py has no docstring')
|
||||
|
||||
@ -441,7 +419,6 @@ class TestRun:
|
||||
backend=usertypes.Backend.QtWebEngine)
|
||||
def fun(self):
|
||||
"""Blah."""
|
||||
pass
|
||||
|
||||
monkeypatch.setattr(command.objects, 'backend',
|
||||
usertypes.Backend.QtWebKit)
|
||||
|
||||
@ -105,35 +105,29 @@ def cmdutils_patch(monkeypatch, stubs, miscmodels_patch):
|
||||
@cmdutils.argument('value', completion=miscmodels_patch.value)
|
||||
def set_command(section_=None, option=None, value=None):
|
||||
"""docstring."""
|
||||
pass
|
||||
|
||||
@cmdutils.argument('topic', completion=miscmodels_patch.helptopic)
|
||||
def show_help(tab=False, bg=False, window=False, topic=None):
|
||||
"""docstring."""
|
||||
pass
|
||||
|
||||
@cmdutils.argument('url', completion=miscmodels_patch.url)
|
||||
@cmdutils.argument('count', count=True)
|
||||
def openurl(url=None, related=False, bg=False, tab=False, window=False,
|
||||
count=None):
|
||||
"""docstring."""
|
||||
pass
|
||||
|
||||
@cmdutils.argument('win_id', win_id=True)
|
||||
@cmdutils.argument('command', completion=miscmodels_patch.command)
|
||||
def bind(key, win_id, command=None, *, mode='normal'):
|
||||
"""docstring."""
|
||||
pass
|
||||
|
||||
def tab_give():
|
||||
"""docstring."""
|
||||
pass
|
||||
|
||||
@cmdutils.argument('option', completion=miscmodels_patch.option)
|
||||
@cmdutils.argument('values', completion=miscmodels_patch.value)
|
||||
def config_cycle(option, *values):
|
||||
"""For testing varargs."""
|
||||
pass
|
||||
|
||||
cmd_utils = stubs.FakeCmdUtils({
|
||||
'set': command.Command(name='set', handler=set_command),
|
||||
|
||||
@ -73,7 +73,7 @@ def test_patched_errwindow(capfd, mocker, monkeypatch):
|
||||
monkeypatch.setattr(checkpyver.sys, 'exit', lambda status: None)
|
||||
|
||||
try:
|
||||
import tkinter # pylint: disable=unused-variable
|
||||
import tkinter # pylint: disable=unused-import
|
||||
except ImportError:
|
||||
tk_mock = mocker.patch('qutebrowser.misc.checkpyver.Tk',
|
||||
spec=['withdraw'], new_callable=mocker.Mock)
|
||||
|
||||
@ -336,8 +336,6 @@ class SavefileTestException(Exception):
|
||||
|
||||
"""Exception raised in TestSavefileOpen for testing."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
@pytest.mark.usefixtures('qapp')
|
||||
class TestSavefileOpen:
|
||||
@ -541,7 +539,6 @@ if test_file is not None:
|
||||
|
||||
def testReadinto_text(self):
|
||||
"""Skip this test as BufferedIOBase seems to fail it."""
|
||||
pass
|
||||
|
||||
class PyOtherFileTests(PyIODeviceTestMixin, test_file.OtherFileTests,
|
||||
unittest.TestCase):
|
||||
@ -550,11 +547,9 @@ if test_file is not None:
|
||||
|
||||
def testSetBufferSize(self):
|
||||
"""Skip this test as setting buffer size is unsupported."""
|
||||
pass
|
||||
|
||||
def testTruncateOnWindows(self):
|
||||
"""Skip this test truncating is unsupported."""
|
||||
pass
|
||||
|
||||
|
||||
class FailingQIODevice(QIODevice):
|
||||
|
||||
@ -409,8 +409,6 @@ class GotException(Exception):
|
||||
|
||||
"""Exception used for TestDisabledExcepthook."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def excepthook(_exc, _val, _tb):
|
||||
pass
|
||||
@ -465,9 +463,7 @@ class TestPreventExceptions:
|
||||
def test_raising(self, caplog):
|
||||
"""Test with a raising function."""
|
||||
with caplog.at_level(logging.ERROR, 'misc'):
|
||||
# pylint: disable=assignment-from-no-return
|
||||
ret = self.func_raising()
|
||||
# pylint: enable=assignment-from-no-return
|
||||
assert ret == 42
|
||||
expected = 'Error in test_utils.TestPreventExceptions.func_raising'
|
||||
assert caplog.messages == [expected]
|
||||
@ -490,9 +486,7 @@ class TestPreventExceptions:
|
||||
def test_predicate_true(self, caplog):
|
||||
"""Test with a True predicate."""
|
||||
with caplog.at_level(logging.ERROR, 'misc'):
|
||||
# pylint: disable=assignment-from-no-return
|
||||
ret = self.func_predicate_true()
|
||||
# enable: disable=assignment-from-no-return
|
||||
assert ret == 42
|
||||
assert len(caplog.records) == 1
|
||||
|
||||
@ -512,8 +506,6 @@ class Obj:
|
||||
|
||||
"""Test object for test_get_repr()."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
@pytest.mark.parametrize('constructor, attrs, expected', [
|
||||
(False, {}, '<test_utils.Obj>'),
|
||||
@ -534,12 +526,10 @@ class QualnameObj():
|
||||
|
||||
def func(self):
|
||||
"""Test method for test_qualname."""
|
||||
pass
|
||||
|
||||
|
||||
def qualname_func(_blah):
|
||||
"""Test function for test_qualname."""
|
||||
pass
|
||||
|
||||
|
||||
QUALNAME_OBJ = QualnameObj()
|
||||
@ -578,8 +568,6 @@ class TestIsEnum:
|
||||
|
||||
"""Test class for is_enum."""
|
||||
|
||||
pass
|
||||
|
||||
assert not utils.is_enum(Test)
|
||||
|
||||
def test_object(self):
|
||||
@ -597,7 +585,6 @@ class TestRaises:
|
||||
|
||||
def do_nothing(self):
|
||||
"""Helper function which does nothing."""
|
||||
pass
|
||||
|
||||
@pytest.mark.parametrize('exception, value, expected', [
|
||||
(ValueError, 'a', True),
|
||||
|
||||
@ -29,8 +29,6 @@ class Parent(QObject):
|
||||
|
||||
"""Class for test_parent()."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
def test_parent():
|
||||
"""Make sure the parent is set correctly."""
|
||||
|
||||
Loading…
Reference in New Issue
Block a user