exc not defined in this function, raise the correct error... · gstack13/python-future@da5e6d2 · GitHub

gstack13 / python-future Public

forked from PythonCharmers/python-future

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit da5e6d2

Browse filesBrowse the repository at this point in the historyBrowse files

jmgurney

committed

exc not defined in this function, raise the correct error...

1 parent ecf1121 commit da5e6d2
Copy full SHA for da5e6d2

2 files changed

+7-1Lines changed: 7 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• src/future/utils
  • surrogateescape.py
• tests/test_future
  • test_surrogateescape.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎src/future/utils/surrogateescape.py‎

Copy file name to clipboardExpand all lines: src/future/utils/surrogateescape.py

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -83,7 +83,7 @@ def replace_surrogate_encode(mystring):
83	83	# The following magic comes from Py3.3's Python/codecs.c file:
84	84	if not 0xD800 <= code <= 0xDCFF:
85	85	# Not a surrogate. Fail with the original exception.
86		- raise exc
	86	+ raise NotASurrogateError
87	87	# mybytes = [0xe0 | (code >> 12),
88	88	# 0x80 | ((code >> 6) & 0x3f),
89	89	# 0x80 | (code & 0x3f)]

Collapse file

‎tests/test_future/test_surrogateescape.py‎

Copy file name to clipboardExpand all lines: tests/test_future/test_surrogateescape.py

+6Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -33,6 +33,12 @@ def test_encode_ascii_surrogateescape(self):
33	33	b = payload.encode('ascii', 'surrogateescape')
34	34	self.assertEqual(b, b'cMO2c3RhbA\xc3\xa1=\n')
35	35
	36	+ def test_encode_ascii_unicode(self):
	37	+ """
	38	+ Verify that exceptions are raised properly.
	39	+ """
	40	+ self.assertRaises(UnicodeEncodeError, u'\N{SNOWMAN}'.encode, 'US-ASCII', 'surrogateescape')
	41	+
36	42	@expectedFailurePY2
37	43	def test_encode_ascii_surrogateescape_non_newstr(self):
38	44	"""

0 commit comments

Comments

0 (0)