From 3255969874c36b334a51ff117e77bd75ce2ade6a Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Sun, 17 Jan 2021 12:16:55 +0100 Subject: rfc6266: Adjust two-space filename check --- tests/unit/browser/webkit/http/test_content_disposition.py | 13 ++++++++++--- 1 file 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): -- cgit v1.2.3-54-g00ecf