Skip to content

Commit 59a7b94

Browse files
[3.12] gh-70835: Clarify error message for CSV file opened with wrong newline (GH-113786) (GH-113905)
Based on patch by SilentGhost. (cherry picked from commit 568d220) Co-authored-by: Serhiy Storchaka <[email protected]>
1 parent 84677ff commit 59a7b94

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

Lib/test/test_csv.py

+12-7
Original file line numberDiff line numberDiff line change
@@ -309,13 +309,18 @@ def test_read_oddinputs(self):
309309
[b'abc'], None)
310310

311311
def test_read_eol(self):
312-
self._read_test(['a,b'], [['a','b']])
313-
self._read_test(['a,b\n'], [['a','b']])
314-
self._read_test(['a,b\r\n'], [['a','b']])
315-
self._read_test(['a,b\r'], [['a','b']])
316-
self.assertRaises(csv.Error, self._read_test, ['a,b\rc,d'], [])
317-
self.assertRaises(csv.Error, self._read_test, ['a,b\nc,d'], [])
318-
self.assertRaises(csv.Error, self._read_test, ['a,b\r\nc,d'], [])
312+
self._read_test(['a,b', 'c,d'], [['a','b'], ['c','d']])
313+
self._read_test(['a,b\n', 'c,d\n'], [['a','b'], ['c','d']])
314+
self._read_test(['a,b\r\n', 'c,d\r\n'], [['a','b'], ['c','d']])
315+
self._read_test(['a,b\r', 'c,d\r'], [['a','b'], ['c','d']])
316+
317+
errmsg = "with newline=''"
318+
with self.assertRaisesRegex(csv.Error, errmsg):
319+
next(csv.reader(['a,b\rc,d']))
320+
with self.assertRaisesRegex(csv.Error, errmsg):
321+
next(csv.reader(['a,b\nc,d']))
322+
with self.assertRaisesRegex(csv.Error, errmsg):
323+
next(csv.reader(['a,b\r\nc,d']))
319324

320325
def test_read_eof(self):
321326
self._read_test(['a,"'], [['a', '']])

Modules/_csv.c

+2-1
Original file line numberDiff line numberDiff line change
@@ -841,7 +841,8 @@ parse_process_char(ReaderObj *self, _csvstate *module_state, Py_UCS4 c)
841841
self->state = START_RECORD;
842842
else {
843843
PyErr_Format(module_state->error_obj,
844-
"new-line character seen in unquoted field - do you need to open the file in universal-newline mode?");
844+
"new-line character seen in unquoted field - "
845+
"do you need to open the file with newline=''?");
845846
return -1;
846847
}
847848
break;

0 commit comments

Comments
 (0)