From 33915b65cf1e2b9bd734e75b1ac8d2add2e61296 Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Sun, 2 Aug 2015 17:19:20 +0200 Subject: [PATCH] Test getting path without Content-Disposition. --- tests/browser/http/test_http.py | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) diff --git a/tests/browser/http/test_http.py b/tests/browser/http/test_http.py index 38ef1b42b..a6306233a 100644 --- a/tests/browser/http/test_http.py +++ b/tests/browser/http/test_http.py @@ -23,9 +23,36 @@ Note that tests for parse_content_disposition are in their own test_content_disposition.py file. """ +import pytest +from PyQt5.QtCore import QUrl + from qutebrowser.browser import http + +class TestNoContentDisposition: + + """Test parse_content_disposition with no Content-Disposition header.""" + + @pytest.mark.parametrize('url', ['http://example.com/path', + 'http://example.com/foo/path']) + def test_url(self, stubs, url): + """Test with a filename in the URL.""" + reply = stubs.FakeNetworkReply(url=QUrl(url)) + inline, filename = http.parse_content_disposition(reply) + assert inline + assert filename == 'path' + + @pytest.mark.parametrize('url', ['http://example.com', + 'http://example.com/']) + def test_none(self, stubs, url): + """Test with no filename at all.""" + reply = stubs.FakeNetworkReply(url=QUrl(url)) + inline, filename = http.parse_content_disposition(reply) + assert inline + assert filename == 'qutebrowser-download' + + class TestParseContentType: """Test for parse_content_type."""