🌐 AI搜索 & 代理 主页
Skip to content

gh-104265 Disallow instantiation of _csv.Reader and _csv.Writer #104266

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
erlend-aasland merged 13 commits into from
May 7, 2023
Merged

gh-104265 Disallow instantiation of _csv.Reader and _csv.Writer #104266

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ae69f9b
Disallow instantiation and subtyping of `_csv.Reader` and `_csv.Write…
chgnrdv May 7, 2023
7d1fcd8
added test
chgnrdv May 7, 2023
aa34fae
changed tests to use check_disallow_instantiation, separated tests
chgnrdv May 7, 2023
6c45ff8
reverted Py_TPFLAGS_BASETYPE flag deletion, removed corresponding tests
chgnrdv May 7, 2023
7d9ec2a
📜🤖 Added by blurb_it.
blurb-it[bot] May 7, 2023
19bddd0
Update 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
b3f4a65
Update 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
a022e91
Update 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
52aacfd
📜🤖 Added by blurb_it.
blurb-it[bot] May 7, 2023
d31d214
Delete 2023-05-07-16-13-07.gh-issue-104265.fU60bZ.rst
chgnrdv May 7, 2023
2804744
Update Misc/NEWS.d/next/Library/2023-05-07-19-56-45.gh-issue-104265.f…
erlend-aasland May 7, 2023
8255467
applied suggestion by erlend-aasland
chgnrdv May 7, 2023
73609a8
Merge branch '_csv-make-Reader-Writer-types-non-instantiable' of gith…
chgnrdv May 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
changed tests to use check_disallow_instantiation, separated tests
  • Loading branch information
@chgnrdv
chgnrdv committed May 7, 2023
commit aa34fae64b55982bb0abb8a306f4075fe17292c1
16 changes: 10 additions & 6 deletions Lib/test/test_csv.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
import gc
import pickle
from test import support
from test.support import warnings_helper
from test.support import warnings_helper, check_disallow_instantiation
from itertools import permutations
from textwrap import dedent
from collections import OrderedDict
Expand Down Expand Up @@ -1431,13 +1431,17 @@ def test_subclassable(self):
# issue 44089
class Foo(csv.Error): ...

def test_issue104265(self):
with self.assertRaisesRegex(TypeError, "cannot create '_csv.reader' instances"):
_csv.Reader()
with self.assertRaisesRegex(TypeError, "cannot create '_csv.writer' instances"):
_csv.Writer()
def test_reader_disallow_instantiation(self):
check_disallow_instantiation(self, _csv.Reader)

def test_writer_disallow_instantiation(self):
check_disallow_instantiation(self, _csv.Writer)

def test_reader_not_basetype(self):
with self.assertRaisesRegex(TypeError, "type '_csv.reader' is not an acceptable base type"):
class Foo(_csv.Reader): pass

def test_writer_not_basetype(self):
with self.assertRaisesRegex(TypeError, "type '_csv.writer' is not an acceptable base type"):
class Foo(_csv.Writer): pass

Expand Down