Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit bbaeac7

Browse files
committed
Move override_builtins test to regression testing framework. Fix a new bug this regression test found by overriding "bytes" on Python >= 2.6.
1 parent bebb17f commit bbaeac7

3 files changed

Lines changed: 25 additions & 17 deletions

File tree

lib/matplotlib/pylab.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -292,3 +292,6 @@ def save(*args, **kwargs):
292292

293293
# don't let numpy's datetime hide stdlib
294294
import datetime
295+
296+
if sys.version_info > (2, 6, 0):
297+
bytes = __builtins__['bytes']

lib/matplotlib/tests/test_basic.py

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,31 @@
11
from nose.tools import assert_equal
22
from matplotlib.testing.decorators import knownfailureif
3+
import sys
34

45
def test_simple():
56
assert_equal(1+1,2)
67

78
@knownfailureif(True)
89
def test_simple_knownfail():
910
assert_equal(1+1,3)
11+
12+
from pylab import *
13+
def test_override_builtins():
14+
ok_to_override = set([
15+
'__name__',
16+
'__doc__',
17+
'__package__',
18+
'any',
19+
'all',
20+
'sum'
21+
])
22+
23+
overridden = False
24+
for key in globals().keys():
25+
if key in dir(sys.modules["__builtin__"]):
26+
if (globals()[key] != getattr(sys.modules["__builtin__"], key) and
27+
key not in ok_to_override):
28+
print "'%s' was overridden in globals()." % key
29+
overridden = True
30+
31+
assert not overridden

unit/override_builtins.py

Lines changed: 0 additions & 17 deletions
This file was deleted.

0 commit comments

Comments
 (0)