Remove exclamation mark in --restrict-filenames mode

[youtube-dl] / test / test_utils.py

diff --git a/test/test_utils.py b/test/test_utils.py
index 4208ee653243562b53fc011329907d471e76b712..a3a23fbb4a5a71da86488c951b151c5c42e17456 100644 (file)
--- a/test/test_utils.py
+++ b/test/test_utils.py
@@ -53,14 +53,21 @@ class TestUtil(unittest.TestCase):
                self.assertEqual(u'yes_no', sanitize_filename(u'yes? no', restricted=True))
                self.assertEqual(u'this_-_that', sanitize_filename(u'this: that', restricted=True))
 
-               self.assertEqual(sanitize_filename(u'aäb', restricted=True), u'a_b')
+               self.assertEqual(sanitize_filename(u'aäb中国的c', restricted=True), u'a_b_c')
                self.assertTrue(sanitize_filename(u'ö', restricted=True) != u'') # No empty filename
 
-               forbidden = u'"\0\\/&: \'\t\n'
+               forbidden = u'"\0\\/&!: \'\t\n'
                for fc in forbidden:
                        for fbc in forbidden:
                                self.assertTrue(fbc not in sanitize_filename(fc, restricted=True))
 
+               # Handle a common case more neatly
+               self.assertEqual(sanitize_filename(u'大声带 - Song', restricted=True), u'Song')
+               self.assertEqual(sanitize_filename(u'总统: Speech', restricted=True), u'Speech')
+               # .. but make sure the file name is never empty
+               self.assertTrue(sanitize_filename(u'-', restricted=True) != u'')
+               self.assertTrue(sanitize_filename(u':', restricted=True) != u'')
+
        def test_ordered_set(self):
                self.assertEqual(orderedSet([1,1,2,3,4,4,5,6,7,3,5]), [1,2,3,4,5,6,7])
                self.assertEqual(orderedSet([]), [])