Issue 33963: IDLE macosx: add tests. (original) (raw)

Created on 2018-06-26 03:50 by terry.reedy, last changed 2022-04-11 14:59 by admin.

Messages (4)
msg320468 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2018-06-26 03:50
For overrideRootMenu, move nested functions to module level. Try to move 3 imports to module level. Test startup on mac. Mock 'window' with mock functions.
msg320469 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2018-06-26 04:01
Ned, I would like to rename at least some of the functions in idlelib.macosx. For instance, function overrideRootMenu to override_root_menu. Would this affect any of the installation adjustments? IE, does the installer import the module for whatever reason? Also, do the windowing types on currently supported machines still include all of Xquartz, Cocoa, Carbon (both Aqua), and other? Is the windowing type determined by the hardware, or by the macOS version?
msg320471 - (view) Author: Ned Deily (ned.deily) * (Python committer) Date: 2018-06-26 05:39
> Would this affect any of the installation adjustments? I don't think so but there's one way to be sure :) If you merge your changes to master, we can doublecheck before backporting them. > Is the windowing type determined by the hardware, or by the macOS version? The windowing type is determined by the variant of Tk that tkinter is dynamically linking with. While Cocoa Tk is now the most common, it is still possible to run into the other variants; for example, MacPorts supports building an X11 (Xquartz) variant of Tk 8.6 along with a Cocoa one. If support for the various window types isn't causing a maintenance burden, it would be nice to retain them.
msg320671 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2018-06-28 15:43
Mark Roseman emailed me that moving the 3 imports within overrideRootMenu to the top of the module does not appear to cause a problem.
History
Date User Action Args
2022-04-11 14:59:02 admin set github: 78144
2018-12-12 02:34:48 terry.reedy set versions: - Python 3.6
2018-06-28 22🔞47 markroseman set nosy: + markrosemancomponents: + IDLEassignee: terry.reedy
2018-06-28 15:43:51 terry.reedy set messages: +
2018-06-26 05:39:07 ned.deily set messages: +
2018-06-26 04:02:19 terry.reedy link issue33964 dependencies
2018-06-26 04:01:46 terry.reedy set nosy: + ned.deilymessages: +
2018-06-26 03:50:48 terry.reedy set title: IDLE macosc: add tests. -> IDLE macosx: add tests.
2018-06-26 03:50:35 terry.reedy create