diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2017-03-05 22:34:54 -0500 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2017-03-05 22:34:54 -0500 |
commit | ced3567f9eafec06cfa61583cac19161c0a330de (patch) | |
tree | a245a82ff1bc841b5151736add82db529e38cb30 /tests/modules/process_test/try_execfile.py | |
parent | 05d295b8ec8628c252ed591f7f4cc6c1cf7f2ca1 (diff) | |
download | python-coveragepy-git-ced3567f9eafec06cfa61583cac19161c0a330de.tar.gz |
Make the list of builtins more compact, since they are never the problem in these tests
Diffstat (limited to 'tests/modules/process_test/try_execfile.py')
-rw-r--r-- | tests/modules/process_test/try_execfile.py | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/tests/modules/process_test/try_execfile.py b/tests/modules/process_test/try_execfile.py index d03a8996..ec7dcbe5 100644 --- a/tests/modules/process_test/try_execfile.py +++ b/tests/modules/process_test/try_execfile.py @@ -20,7 +20,10 @@ differences and get a clean diff. """ -import json, os, sys +import itertools +import json +import os +import sys # sys.path varies by execution environments. Coverage.py uses setuptools to # make console scripts, which means pkg_resources is imported. pkg_resources @@ -65,12 +68,20 @@ FN_VAL = my_function("fooey") loader = globals().get('__loader__') fullname = getattr(loader, 'fullname', None) or getattr(loader, 'name', None) +# A more compact grouped-by-first-letter list of builtins. +def word_group(w): + """Clump AB, CD, EF, etc.""" + return chr((ord(w[0]) + 1) & 0xFE) + +builtin_dir = [" ".join(s) for _, s in itertools.groupby(dir(__builtins__), key=word_group)] + globals_to_check = { + 'os.getcwd': os.getcwd(), '__name__': __name__, '__file__': __file__, '__doc__': __doc__, '__builtins__.has_open': hasattr(__builtins__, 'open'), - '__builtins__.dir': dir(__builtins__), + '__builtins__.dir': builtin_dir, '__loader__ exists': loader is not None, '__loader__.fullname': fullname, '__package__': __package__, |