Skip to content

Commit ddf4f55

Browse files
committed
Enable tests that now pass thanks to new scope scheme.
1 parent 6ad528c commit ddf4f55

File tree

1 file changed

+13
-18
lines changed

1 file changed

+13
-18
lines changed

tests/snippets/test_exec.py

Lines changed: 13 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -13,26 +13,21 @@
1313

1414
exec("assert max(1, 5, square(5)) == 25", None)
1515

16-
#
17-
# These doesn't work yet:
18-
#
1916
# Local environment shouldn't replace global environment:
20-
#
21-
# exec("assert max(1, 5, square(5)) == 25", None, {})
22-
#
17+
exec("assert max(1, 5, square(5)) == 25", None, {})
18+
2319
# Closures aren't available if local scope is replaced:
24-
#
25-
# def g():
26-
# seven = "seven"
27-
# def f():
28-
# try:
29-
# exec("seven", None, {})
30-
# except NameError:
31-
# pass
32-
# else:
33-
# raise NameError("seven shouldn't be in scope")
34-
# f()
35-
# g()
20+
def g():
21+
seven = "seven"
22+
def f():
23+
try:
24+
exec("seven", None, {})
25+
except NameError:
26+
pass
27+
else:
28+
raise NameError("seven shouldn't be in scope")
29+
f()
30+
g()
3631

3732
try:
3833
exec("", 1)

0 commit comments

Comments
 (0)