diff options
author | Douwe Maan <douwe@gitlab.com> | 2018-01-19 11:44:55 +0300 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2018-01-19 11:44:55 +0300 |
commit | 812413863bec3b1f5f3fcf8a8b71906fe5ca2949 (patch) | |
tree | 3fc7001847e8a02a1294010e65f03bc2a4be3636 /spec | |
parent | a1c0964d7170a98214c37e1a45ce8223c83a33f5 (diff) | |
parent | 3b0235318ef1fd39311910da7eb1893a4d3ae312 (diff) |
Merge branch '42159-utf8-uploads' into 'master'
Correctly escape UTF-8 path elements for uploads
Closes #42159
See merge request gitlab-org/gitlab-ce!16560
Diffstat (limited to 'spec')
-rw-r--r-- | spec/lib/banzai/filter/relative_link_filter_spec.rb | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/spec/lib/banzai/filter/relative_link_filter_spec.rb b/spec/lib/banzai/filter/relative_link_filter_spec.rb index 7e17457ce70..3ca4652f7cc 100644 --- a/spec/lib/banzai/filter/relative_link_filter_spec.rb +++ b/spec/lib/banzai/filter/relative_link_filter_spec.rb @@ -278,18 +278,19 @@ describe Banzai::Filter::RelativeLinkFilter do expect(doc.at_css('a')['href']).to eq 'http://example.com' end - it 'supports Unicode filenames' do + it 'supports unescaped Unicode filenames' do path = '/uploads/한글.png' - escaped = Addressable::URI.escape(path) + doc = filter(link(path)) - # Stub these methods so the file doesn't actually need to be in the repo - allow_any_instance_of(described_class) - .to receive(:file_exists?).and_return(true) - allow_any_instance_of(described_class) - .to receive(:image?).with(path).and_return(true) + expect(doc.at_css('a')['href']).to eq("/#{project.full_path}/uploads/%ED%95%9C%EA%B8%80.png") + end + it 'supports escaped Unicode filenames' do + path = '/uploads/한글.png' + escaped = Addressable::URI.escape(path) doc = filter(image(escaped)) - expect(doc.at_css('img')['src']).to match "/#{project.full_path}/uploads/%ED%95%9C%EA%B8%80.png" + + expect(doc.at_css('img')['src']).to eq("/#{project.full_path}/uploads/%ED%95%9C%EA%B8%80.png") end end |