Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
24 changes: 10 additions & 14 deletions python2/test_typing.py
Original file line number Diff line number Diff line change
Expand Up @@ -1315,13 +1315,10 @@ class MyDict(typing.Dict[str, int]):
self.assertIsSubclass(MyDict, dict)
self.assertNotIsSubclass(dict, MyDict)

def test_no_defaultdict_instantiation(self):
with self.assertRaises(TypeError):
typing.DefaultDict()
with self.assertRaises(TypeError):
typing.DefaultDict[KT, VT]()
with self.assertRaises(TypeError):
typing.DefaultDict[str, int]()
def test_defaultdict_instantiation(self):
self.assertIs(type(typing.DefaultDict()), collections.defaultdict)
self.assertIs(type(typing.DefaultDict[KT, VT]()), collections.defaultdict)
self.assertIs(type(typing.DefaultDict[str, int]()), collections.defaultdict)

def test_defaultdict_subclass(self):

Expand All @@ -1334,13 +1331,12 @@ class MyDefDict(typing.DefaultDict[str, int]):
self.assertIsSubclass(MyDefDict, collections.defaultdict)
self.assertNotIsSubclass(collections.defaultdict, MyDefDict)

def test_no_deque_instantiation(self):
with self.assertRaises(TypeError):
typing.Deque()
with self.assertRaises(TypeError):
typing.Deque[T]()
with self.assertRaises(TypeError):
typing.Deque[int]()
def test_deque_instantiation(self):
self.assertIs(type(typing.Deque()), collections.deque)
self.assertIs(type(typing.Deque[T]()), collections.deque)
self.assertIs(type(typing.Deque[int]()), collections.deque)
class D(typing.Deque[T]): pass
self.assertIs(type(D[int]()), D)

def test_no_set_instantiation(self):
with self.assertRaises(TypeError):
Expand Down
6 changes: 2 additions & 4 deletions python2/typing.py
Original file line number Diff line number Diff line change
Expand Up @@ -1714,8 +1714,7 @@ class Deque(collections.deque, MutableSequence[T]):

def __new__(cls, *args, **kwds):
if _geqv(cls, Deque):
raise TypeError("Type Deque cannot be instantiated; "
"use deque() instead")
return collections.deque(*args, **kwds)
return _generic_new(collections.deque, cls, *args, **kwds)


Expand Down Expand Up @@ -1780,8 +1779,7 @@ class DefaultDict(collections.defaultdict, MutableMapping[KT, VT]):

def __new__(cls, *args, **kwds):
if _geqv(cls, DefaultDict):
raise TypeError("Type DefaultDict cannot be instantiated; "
"use collections.defaultdict() instead")
return collections.defaultdict(*args, **kwds)
return _generic_new(collections.defaultdict, cls, *args, **kwds)


Expand Down
24 changes: 10 additions & 14 deletions src/test_typing.py
Original file line number Diff line number Diff line change
Expand Up @@ -1713,13 +1713,10 @@ class MyDict(typing.Dict[str, int]):
self.assertIsSubclass(MyDict, dict)
self.assertNotIsSubclass(dict, MyDict)

def test_no_defaultdict_instantiation(self):
with self.assertRaises(TypeError):
typing.DefaultDict()
with self.assertRaises(TypeError):
typing.DefaultDict[KT, VT]()
with self.assertRaises(TypeError):
typing.DefaultDict[str, int]()
def test_defaultdict_instantiation(self):
self.assertIs(type(typing.DefaultDict()), collections.defaultdict)
self.assertIs(type(typing.DefaultDict[KT, VT]()), collections.defaultdict)
self.assertIs(type(typing.DefaultDict[str, int]()), collections.defaultdict)

def test_defaultdict_subclass(self):

Expand All @@ -1732,13 +1729,12 @@ class MyDefDict(typing.DefaultDict[str, int]):
self.assertIsSubclass(MyDefDict, collections.defaultdict)
self.assertNotIsSubclass(collections.defaultdict, MyDefDict)

def test_no_deque_instantiation(self):
with self.assertRaises(TypeError):
typing.Deque()
with self.assertRaises(TypeError):
typing.Deque[T]()
with self.assertRaises(TypeError):
typing.Deque[int]()
def test_deque_instantiation(self):
self.assertIs(type(typing.Deque()), collections.deque)
self.assertIs(type(typing.Deque[T]()), collections.deque)
self.assertIs(type(typing.Deque[int]()), collections.deque)
class D(typing.Deque[T]): ...
self.assertIs(type(D[int]()), D)

def test_no_set_instantiation(self):
with self.assertRaises(TypeError):
Expand Down
6 changes: 2 additions & 4 deletions src/typing.py
Original file line number Diff line number Diff line change
Expand Up @@ -1825,8 +1825,7 @@ class Deque(collections.deque, MutableSequence[T], extra=collections.deque):

def __new__(cls, *args, **kwds):
if _geqv(cls, Deque):
raise TypeError("Type Deque cannot be instantiated; "
"use deque() instead")
return collections.deque(*args, **kwds)
return _generic_new(collections.deque, cls, *args, **kwds)


Expand Down Expand Up @@ -1895,8 +1894,7 @@ class DefaultDict(collections.defaultdict, MutableMapping[KT, VT],

def __new__(cls, *args, **kwds):
if _geqv(cls, DefaultDict):
raise TypeError("Type DefaultDict cannot be instantiated; "
"use collections.defaultdict() instead")
return collections.defaultdict(*args, **kwds)
return _generic_new(collections.defaultdict, cls, *args, **kwds)


Expand Down