rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

rewrite tests

Browse Source
This commit is contained in:
gammelon 2018-03-09 15:52:03 +01:00
parent a730290d40
commit 7e3c966afe
2 changed files with 29 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
qutebrowser/utils/urlutils.py
View File

@ -103,8 +103,7 @@ def _get_search_url(txt):
template = config.val.url.searchengines[engine]
url = qurl_from_user_input(template.format(urllib.parse.quote(term)))
if config.val.url.open_base_url and \
term in config.val.url.searchengines.keys():
if config.val.url.open_base_url and term in config.val.url.searchengines:
url = qurl_from_user_input(config.val.url.searchengines[term])
url.setPath(None)
url.setFragment(None)

49
tests/unit/utils/test_urlutils.py
View File

@ -277,25 +277,17 @@ class TestFuzzyUrl:
def test_special_urls(url, special):
assert urlutils.is_special_url(QUrl(url)) == special
@pytest.mark.parametrize('url, host, query, open_base_url', [
('testfoo', 'www.example.com', 'q=testfoo', False),
('test testfoo', 'www.qutebrowser.org', 'q=testfoo', False),
('test testfoo bar foo', 'www.qutebrowser.org', 'q=testfoo bar foo', False),
('test testfoo ', 'www.qutebrowser.org', 'q=testfoo', False),
('!python testfoo', 'www.example.com', 'q=%21python testfoo', False),
('blub testfoo', 'www.example.com', 'q=blub testfoo', False),
('stripped ', 'www.example.com', 'q=stripped', False),
('test-with-dash testfoo', 'www.example.org', 'q=testfoo', False),
('testfoo', 'www.example.com', 'q=testfoo', True),
('test testfoo', 'www.qutebrowser.org', 'q=testfoo', True),
('test testfoo bar foo', 'www.qutebrowser.org', 'q=testfoo bar foo', True),
('test testfoo ', 'www.qutebrowser.org', 'q=testfoo', True),
('!python testfoo', 'www.example.com', 'q=%21python testfoo', True),
('blub testfoo', 'www.example.com', 'q=blub testfoo', True),
('stripped ', 'www.example.com', 'q=stripped', True),
('test-with-dash testfoo', 'www.example.org', 'q=testfoo', True),
('test', 'www.qutebrowser.org', '', True),
('test-with-dash', 'www.example.org', '', True),
@pytest.mark.parametrize('open_base_url', [True, False])
@pytest.mark.parametrize('url, host, query', [
('testfoo', 'www.example.com', 'q=testfoo'),
('test testfoo', 'www.qutebrowser.org', 'q=testfoo'),
('test testfoo bar foo', 'www.qutebrowser.org', 'q=testfoo bar foo'),
('test testfoo ', 'www.qutebrowser.org', 'q=testfoo'),
('!python testfoo', 'www.example.com', 'q=%21python testfoo'),
('blub testfoo', 'www.example.com', 'q=blub testfoo'),
('stripped ', 'www.example.com', 'q=stripped'),
('test-with-dash testfoo', 'www.example.org', 'q=testfoo'),
])
def test_get_search_url(config_stub, url, host, query, open_base_url):
"""Test _get_search_url().
@ -308,13 +300,28 @@ def test_get_search_url(config_stub, url, host, query, open_base_url):
config_stub.val.url.open_base_url = open_base_url
url = urlutils._get_search_url(url)
if open_base_url and query == '':
assert url.path() == ''
assert url.fragment() == ''
assert not url.path()
assert not url.fragment()
assert url.host() == host
assert url.query() == query
@pytest.mark.parametrize('url, host, query', [
('test', 'www.qutebrowser.org', ''),
('test-with-dash', 'www.example.org', ''),
])
def test_get_search_url_open_base_url(config_stub, url, host, query):
"""Test _get_search_url() with url.open_base_url_enabled.
Args:
url: The "URL" to enter.
host: The expected search machine host.
query: The expected search query.
"""
test_get_search_url(config_stub, url, host, query, True)
@pytest.mark.parametrize('url', ['\n', ' ', '\n '])
def test_get_search_url_invalid(url):
with pytest.raises(ValueError):