Terry J. Reedy added the comment:

Code also suggests that this function should be elsewhere, such as in 
iobinding. This should be in the same place as regular open, wherever that is, 
since it is an alternative open.

Document should be upgraded a bit: "Load module opens Python-coded modules on 
sys.path in an editor window. The default name is the currently selected text, 
if any."

Tests should be added with this issue. This is the smallest dialog, and the 
structure of the revised code (#18055) gives a skeleton for a testcase: 
test_python_module (window open), test_builtin_module (Exception), 
test_bad_path (different Exception), and maybe even test_bad loader. We should 
look at tkinter tests for simulating name entry and click.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue18064>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to