mirror of
				https://github.com/yt-dlp/yt-dlp.git
				synced 2025-10-31 14:45:14 +00:00 
			
		
		
		
	[cleanup] Mark some compat variables for removal (#2173)
Authored by fstirlitz, pukkandan
This commit is contained in:
		| @@ -13,14 +13,10 @@ sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) | ||||
| from yt_dlp.compat import ( | ||||
|     compat_getenv, | ||||
|     compat_setenv, | ||||
|     compat_etree_Element, | ||||
|     compat_etree_fromstring, | ||||
|     compat_expanduser, | ||||
|     compat_shlex_split, | ||||
|     compat_str, | ||||
|     compat_struct_unpack, | ||||
|     compat_urllib_parse_quote, | ||||
|     compat_urllib_parse_quote_plus, | ||||
|     compat_urllib_parse_unquote, | ||||
|     compat_urllib_parse_unquote_plus, | ||||
|     compat_urllib_parse_urlencode, | ||||
| @@ -55,27 +51,6 @@ class TestCompat(unittest.TestCase): | ||||
|             dir(yt_dlp.compat))) - set(['unicode_literals']) | ||||
|         self.assertEqual(all_names, sorted(present_names)) | ||||
| 
 | ||||
|     def test_compat_urllib_parse_quote(self): | ||||
|         self.assertEqual(compat_urllib_parse_quote('abc def'), 'abc%20def') | ||||
|         self.assertEqual(compat_urllib_parse_quote('/user/abc+def'), '/user/abc%2Bdef') | ||||
|         self.assertEqual(compat_urllib_parse_quote('/user/abc+def', safe='+'), '%2Fuser%2Fabc+def') | ||||
|         self.assertEqual(compat_urllib_parse_quote(''), '') | ||||
|         self.assertEqual(compat_urllib_parse_quote('%'), '%25') | ||||
|         self.assertEqual(compat_urllib_parse_quote('%', safe='%'), '%') | ||||
|         self.assertEqual(compat_urllib_parse_quote('津波'), '%E6%B4%A5%E6%B3%A2') | ||||
|         self.assertEqual( | ||||
|             compat_urllib_parse_quote('''<meta property="og:description" content="▁▂▃▄%▅▆▇█" /> | ||||
| %<a href="https://ar.wikipedia.org/wiki/تسونامي">%a''', safe='<>=":%/ \r\n'), | ||||
|             '''<meta property="og:description" content="%E2%96%81%E2%96%82%E2%96%83%E2%96%84%%E2%96%85%E2%96%86%E2%96%87%E2%96%88" /> | ||||
| %<a href="https://ar.wikipedia.org/wiki/%D8%AA%D8%B3%D9%88%D9%86%D8%A7%D9%85%D9%8A">%a''') | ||||
|         self.assertEqual( | ||||
|             compat_urllib_parse_quote('''(^◣_◢^)っ︻デ═一    ⇀    ⇀    ⇀    ⇀    ⇀    ↶%I%Break%25Things%''', safe='% '), | ||||
|             '''%28%5E%E2%97%A3_%E2%97%A2%5E%29%E3%81%A3%EF%B8%BB%E3%83%87%E2%95%90%E4%B8%80    %E2%87%80    %E2%87%80    %E2%87%80    %E2%87%80    %E2%87%80    %E2%86%B6%I%Break%25Things%''') | ||||
| 
 | ||||
|     def test_compat_urllib_parse_quote_plus(self): | ||||
|         self.assertEqual(compat_urllib_parse_quote_plus('abc def'), 'abc+def') | ||||
|         self.assertEqual(compat_urllib_parse_quote_plus('/abc def'), '%2Fabc+def') | ||||
| 
 | ||||
|     def test_compat_urllib_parse_unquote(self): | ||||
|         self.assertEqual(compat_urllib_parse_unquote('abc%20def'), 'abc def') | ||||
|         self.assertEqual(compat_urllib_parse_unquote('%7e/abc+def'), '~/abc+def') | ||||
| @@ -109,17 +84,6 @@ class TestCompat(unittest.TestCase): | ||||
|         self.assertEqual(compat_urllib_parse_urlencode([(b'abc', 'def')]), 'abc=def') | ||||
|         self.assertEqual(compat_urllib_parse_urlencode([(b'abc', b'def')]), 'abc=def') | ||||
| 
 | ||||
|     def test_compat_shlex_split(self): | ||||
|         self.assertEqual(compat_shlex_split('-option "one two"'), ['-option', 'one two']) | ||||
|         self.assertEqual(compat_shlex_split('-option "one\ntwo" \n -flag'), ['-option', 'one\ntwo', '-flag']) | ||||
|         self.assertEqual(compat_shlex_split('-val 中文'), ['-val', '中文']) | ||||
| 
 | ||||
|     def test_compat_etree_Element(self): | ||||
|         try: | ||||
|             compat_etree_Element.items | ||||
|         except AttributeError: | ||||
|             self.fail('compat_etree_Element is not a type') | ||||
| 
 | ||||
|     def test_compat_etree_fromstring(self): | ||||
|         xml = ''' | ||||
|             <root foo="bar" spam="中文"> | ||||
|   | ||||
| @@ -11,11 +11,12 @@ sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) | ||||
| import io | ||||
| import re | ||||
| import string | ||||
| import urllib.request | ||||
| 
 | ||||
| from test.helper import FakeYDL, is_download_test | ||||
| from yt_dlp.extractor import YoutubeIE | ||||
| from yt_dlp.jsinterp import JSInterpreter | ||||
| from yt_dlp.compat import compat_str, compat_urlretrieve | ||||
| from yt_dlp.compat import compat_str | ||||
| 
 | ||||
| _SIG_TESTS = [ | ||||
|     ( | ||||
| @@ -147,7 +148,7 @@ def t_factory(name, sig_func, url_pattern): | ||||
|             fn = os.path.join(self.TESTDATA_DIR, basename) | ||||
| 
 | ||||
|             if not os.path.exists(fn): | ||||
|                 compat_urlretrieve(url, fn) | ||||
|                 urllib.request.urlretrieve(url, fn) | ||||
|             with io.open(fn, encoding='utf-8') as testf: | ||||
|                 jscode = testf.read() | ||||
|             self.assertEqual(sig_func(jscode, sig_input), expected_sig) | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 pukkandan
					pukkandan