diff options
| author | Florian Bruhin <me@the-compiler.org> | 2021-01-17 12:16:55 +0100 |
|---|---|---|
| committer | Florian Bruhin <me@the-compiler.org> | 2021-01-17 12:24:00 +0100 |
| commit | 3255969874c36b334a51ff117e77bd75ce2ade6a (patch) | |
| tree | e298eff17e680ef19e8f5b4111478e88a1473db1 | |
| parent | bdbf26eddf92be3aea264eb69dfcc13575ea1848 (diff) | |
| download | qutebrowser-3255969874c36b334a51ff117e77bd75ce2ade6a.tar.gz qutebrowser-3255969874c36b334a51ff117e77bd75ce2ade6a.zip | |
rfc6266: Adjust two-space filename check
| -rw-r--r-- | tests/unit/browser/webkit/http/test_content_disposition.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/tests/unit/browser/webkit/http/test_content_disposition.py b/tests/unit/browser/webkit/http/test_content_disposition.py index 6431465d7..efd11cfba 100644 --- a/tests/unit/browser/webkit/http/test_content_disposition.py +++ b/tests/unit/browser/webkit/http/test_content_disposition.py @@ -81,9 +81,16 @@ def test_inline_caps(header_checker): def test_att_double_space(header_checker): - """'attachment' with double space in the filename.""" - header_checker.check_filename('attachment; filename="foo bar.html"', - 'foo bar.html') + """'attachment' with double space in the filename. + + It's unclear how this should be handled exactly. The original test expected 'foo + bar.html' as filename, but there doesn't seem to be a place in the RFC actually + specifying that. The test was added in 8ce779261f08b20fd3f6e889204eb88e6a6800c1 but + without further explanations why. Thus, it's now updated to expect the original + filename, with two spaces. + """ + header_checker.check_filename( + 'attachment; filename="foo bar.html"', 'foo bar.html') def test_iso2231_langtag(header_checker): |