diff options
author | George Claghorn <george@basecamp.com> | 2017-08-14 16:04:11 -0400 |
---|---|---|
committer | George Claghorn <george@basecamp.com> | 2017-08-14 16:04:11 -0400 |
commit | 3cfadd5fa2306ec1f382343220805546e1444cd8 (patch) | |
tree | 8006b482e9d975ec28eb84795d83a6115763ad0a /activestorage/test/filename_test.rb | |
parent | 09dfabccd1015d89c5503e6c556b87f6a0db5a5a (diff) | |
download | rails-3cfadd5fa2306ec1f382343220805546e1444cd8.tar.gz rails-3cfadd5fa2306ec1f382343220805546e1444cd8.tar.bz2 rails-3cfadd5fa2306ec1f382343220805546e1444cd8.zip |
Move test to its proper place
ActiveStorage::Filename is a model, so its test case belongs in test/models.
Diffstat (limited to 'activestorage/test/filename_test.rb')
-rw-r--r-- | activestorage/test/filename_test.rb | 38 |
1 files changed, 0 insertions, 38 deletions
diff --git a/activestorage/test/filename_test.rb b/activestorage/test/filename_test.rb deleted file mode 100644 index f1e4a467ba..0000000000 --- a/activestorage/test/filename_test.rb +++ /dev/null @@ -1,38 +0,0 @@ -# frozen_string_literal: true - -require "test_helper" - -class ActiveStorage::FilenameTest < ActiveSupport::TestCase - test "sanitize" do - "%$|:;/\t\r\n\\".each_char do |character| - filename = ActiveStorage::Filename.new("foo#{character}bar.pdf") - assert_equal "foo-bar.pdf", filename.sanitized - assert_equal "foo-bar.pdf", filename.to_s - end - end - - test "sanitize transcodes to valid UTF-8" do - { "\xF6".dup.force_encoding(Encoding::ISO8859_1) => "ö", - "\xC3".dup.force_encoding(Encoding::ISO8859_1) => "Ã", - "\xAD" => "�", - "\xCF" => "�", - "\x00" => "", - }.each do |actual, expected| - assert_equal expected, ActiveStorage::Filename.new(actual).sanitized - end - end - - test "strips RTL override chars used to spoof unsafe executables as docs" do - # Would be displayed in Windows as "evilexe.pdf" due to the right-to-left - # (RTL) override char! - assert_equal "evil-fdp.exe", ActiveStorage::Filename.new("evil\u{202E}fdp.exe").sanitized - end - - test "compare case-insensitively" do - assert_operator ActiveStorage::Filename.new("foobar.pdf"), :==, ActiveStorage::Filename.new("FooBar.PDF") - end - - test "compare sanitized" do - assert_operator ActiveStorage::Filename.new("foo-bar.pdf"), :==, ActiveStorage::Filename.new("foo\tbar.pdf") - end -end |