Issue 33917: Fix and document idlelib/idle_test/template.py (original) (raw)

Created on 2018-06-20 16:54 by doko, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 7830 merged terry.reedy,2018-06-20 20:40
PR 7831 merged miss-islington,2018-06-20 21:09
PR 7832 merged miss-islington,2018-06-20 21:10
Messages (7)
msg320099 - (view) Author: Matthias Klose (doko) * (Python committer) Date: 2018-06-20 16:54
Seen with the 3.6 branch and the 3.7 branch at least. This file is installed by default, so should be valid syntax? If it's supposed to be that way, maybe call the template like template.py.in? Unpacking libpython3.6-testsuite (3.6.6~rc1-3) ... Setting up libpython3.6-testsuite (3.6.6~rc1-3) ... File "/usr/lib/python3.6/idlelib/idle_test/template.py", line 3 from idlelib import ^ SyntaxError: invalid syntax
msg320103 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2018-06-20 18:44
I wrote it intentionally the way it is, for #33855, as it made it maximally easily to use to complete the test suite. Since it caused a problem, I added a valid dummy name to be replaced, and fixed the other syntax error. The file now compiles, runs, and tests OK. New IDLE modules needing a new test file should be rare. I will submit the PR after I edit the readme to document the usage of the template.
msg320112 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2018-06-20 20:57
Tal and Cheryl: I revised the template for IDLE tests, pulled it out of README.txt into template.py, and explained it in the readme. It has new code for cleaning up after instantiating EditorWindow in tests. Please follow for any new test file, including test_tooltip and test_squeezer. (They should both be pretty close already.)
msg320114 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2018-06-20 21:08
New changeset 87a927325e3856621790a39d8718ff24a19510aa by Terry Jan Reedy in branch 'master': bpo-33917: Fix and document idlelib/idle_test/template.py (GH-7830) https://github.com/python/cpython/commit/87a927325e3856621790a39d8718ff24a19510aa
msg320117 - (view) Author: miss-islington (miss-islington) Date: 2018-06-20 21:27
New changeset a717c5646b36a42e4eb0ec4ae2a22520ff47d346 by Miss Islington (bot) in branch '3.7': bpo-33917: Fix and document idlelib/idle_test/template.py (GH-7830) https://github.com/python/cpython/commit/a717c5646b36a42e4eb0ec4ae2a22520ff47d346
msg320119 - (view) Author: miss-islington (miss-islington) Date: 2018-06-20 21:33
New changeset 7c853d0620e1ea838f46642dbaa8ef142d80aa01 by Miss Islington (bot) in branch '3.6': bpo-33917: Fix and document idlelib/idle_test/template.py (GH-7830) https://github.com/python/cpython/commit/7c853d0620e1ea838f46642dbaa8ef142d80aa01
msg320527 - (view) Author: Ned Deily (ned.deily) * (Python committer) Date: 2018-06-27 00:12
FTR, the problem change was merged/backported after 3.7.0rc1 and 3.6.6rc1 so doko must have been working from a development branch and not a release tag. The original and its fix will first be officially released in 3.7.1 and 3.6.7 so there's really no need for a correction NEWS entry; it doesn't hurt either.
History
Date User Action Args
2022-04-11 14:59:02 admin set github: 78098
2018-06-27 00:12:17 ned.deily set priority: release blocker -> messages: +
2018-06-20 21:59:44 terry.reedy set keywords: - patchstatus: open -> closedresolution: fixedstage: patch review -> resolved
2018-06-20 21:33:21 miss-islington set messages: +
2018-06-20 21:27:46 miss-islington set nosy: + miss-islingtonmessages: +
2018-06-20 21:10:44 miss-islington set pull_requests: + <pull%5Frequest7441>
2018-06-20 21:09:45 miss-islington set keywords: + patchstage: commit review -> patch reviewpull_requests: + <pull%5Frequest7440>
2018-06-20 21:08:34 terry.reedy set messages: +
2018-06-20 20:57:53 terry.reedy set stage: needs patch -> commit review
2018-06-20 20:57:40 terry.reedy set nosy: + taleinat, cheryl.sabellamessages: + keywords: - patchstage: patch review -> needs patch
2018-06-20 20:40:06 terry.reedy set keywords: + patchstage: needs patch -> patch reviewpull_requests: + <pull%5Frequest7439>
2018-06-20 18:45:02 terry.reedy set nosy: + ned.deilyversions: + Python 3.6, Python 3.7, Python 3.8
2018-06-20 18:44:47 terry.reedy set type: behaviortitle: idlelib/idle_test/template.py has invalid syntax -> Fix and document idlelib/idle_test/template.pymessages: + stage: needs patch
2018-06-20 18:41:47 terry.reedy link issue33855 dependencies
2018-06-20 16:54:43 doko create