This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author terry.reedy
Recipients louielu, terry.reedy, vstinner
Date 2019-03-20.19:35:54
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1553110554.96.0.612943788202.issue30869@roundup.psfhosted.org>
In-reply-to
Content
I closed PR 2614 in response to Victor's comments, in particular his preference that tests not even read .idlerc.  Since we do not want tests to depend on the contents, there is no need to even read the contents.  (And the exception of testing write-read round-tripping requires a fresh temporary directory.)

I propose that when testing, config.py should create idleConf.usercfg empty, without reference to .idlerc.  The result would be the same as now after usercfg is replaced with testcfg, as in test_configdialog, without the need to revert the replacement.

test_idle has the equivalent of
  import idlelib
  idlelib.testing = True
The same should be added to any test module that sets options.  Then idlelib.testing can be used to change config.py behavior.

Victor: what changes to a module trigger the 'environment changed' warning/error?  The above unreverted change to idlelib.__init__ does not.  The unreverted change to idleCong in test_configdialog did, until fixed.
History
Date User Action Args
2019-03-20 19:35:55terry.reedysetrecipients: + terry.reedy, vstinner, louielu
2019-03-20 19:35:54terry.reedysetmessageid: <1553110554.96.0.612943788202.issue30869@roundup.psfhosted.org>
2019-03-20 19:35:54terry.reedylinkissue30869 messages
2019-03-20 19:35:54terry.reedycreate