Skip to content

Idle: update htest.py and htests #66819

Closed

Description

@terryjreedy
BPO 22629
Nosy @terryjreedy
Files
  • @htest-34.diff
  • @htest-34-2.diff
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/terryjreedy'
    closed_at = None
    created_at = <Date 2014-10-14.03:33:50.580>
    labels = ['expert-IDLE', 'type-bug', '3.8']
    title = 'Idle: update htest.py and htests'
    updated_at = <Date 2019-02-21.00:50:19.172>
    user = 'https://github.com/terryjreedy'

    bugs.python.org fields:

    activity = <Date 2019-02-21.00:50:19.172>
    actor = 'terry.reedy'
    assignee = 'terry.reedy'
    closed = False
    closed_date = None
    closer = None
    components = ['IDLE']
    creation = <Date 2014-10-14.03:33:50.580>
    creator = 'terry.reedy'
    dependencies = []
    files = ['36906', '36954']
    hgrepos = []
    issue_num = 22629
    keywords = ['patch']
    message_count = 5.0
    messages = ['229278', '229565', '229566', '229567', '336168']
    nosy_count = 3.0
    nosy_names = ['terry.reedy', 'python-dev', 'Saimadhav.Heblikar']
    pr_nums = []
    priority = 'normal'
    resolution = None
    stage = 'needs patch'
    status = 'open'
    superseder = None
    type = 'behavior'
    url = 'https://bugs.python.org/issue22629'
    versions = ['Python 3.8']

    Linked PRs

    Activity

    1. terryjreedy commented on Oct 14, 2014

      @terryjreedy
      MemberAuthor

      After using idle-test/htest.py and individual htests when editing files, a few updates are needed.

      1. Improve the docstring for htest.py; add info about "# htest #".

      2. Use Toplevel(parent) instead of Tk() for test-specific childs windows of the main, master window. Currently, for some tests, such as CallTipWindow, the focus does not properly shift to the test window. Closing the master window first leaves the test window open. Closing the test window second then causes several tcl errors (printed to console if available), a Windows message, and an Idle freeze of 5 seconds or more before Idle works again. The change in this initial patch fixes the focus, non-close, error, and hang problems. One click and the test window is ready to go; one click on the master and both go away.

      3. Coding style issues -- I prefer to use winfo_rootx/y, as done elsewhere in Idle, to using re (close call). For CallTipWindow, the inner class is not needed now, if it ever was.

    2. terryjreedy commented on Oct 17, 2014

      @terryjreedy
      MemberAuthor

      I reviewed and edited htest.py docstring and specs up to grep. Unfortunately, things were not quite alphabetical. Reviewed and edited corresponding idlelib module callables up to first line of grep._grep_dialog. I am pushing this as a first chunk.

      Covered _calltip_window, ClassBrowser, _color_delegator, ConfigDialog, _dyn_option_menu, _editor_window, _grep_dialog. Checked x(parent), # htest #, (htest in widget), TopLevel(parent), winfo_x, bindings: see _calltip_window.

    3. changed the title [-]Idle: update htest.py and hests[/-] [+]Idle: update htest.py and htests[/+] on Oct 17, 2014
    4. self-assigned this
      on Oct 17, 2014
    5. python-dev commented on Oct 17, 2014

      python-devmannequin
      Mannequin

      New changeset f5be7fc270d1 by Terry Jan Reedy in branch '2.7':
      Issue bpo-22629: Revise idle_test.htest, mostly docstring. Start revision of
      https://hg.python.org/cpython/rev/f5be7fc270d1

      New changeset e73f1d813f1f by Terry Jan Reedy in branch '3.4':
      Issue bpo-22629: Revise idle_test.htest, mostly docstring. Start revision of
      https://hg.python.org/cpython/rev/e73f1d813f1f

      New changeset d8ac11794f26 by Terry Jan Reedy in branch 'default':
      Merge with 3.4: Issue bpo-22629
      https://hg.python.org/cpython/rev/d8ac11794f26

    6. terryjreedy commented on Feb 21, 2019

      @terryjreedy
      MemberAuthor

      As far as I can tell, the entire diff in @htest-34-2.diff was pushed. My comment about 'up to first line of grep' was correct alphabetically but confusing because files with a '# htest #' section were not completely alphabetical in the file. I need to review current code in light of this issue and a couple of others.

    7. transferred this issue fromon Apr 10, 2022
    8. terryjreedy commented on Dec 1, 2023

      @terryjreedy
      MemberAuthor

      First patch: Review docstring again. Alphabetize specs.
      Later: Review spec and callable grep onwards.
      Any closing issues not fixed in edit consider in #112569.

    9. added 2 commits that reference this issue on Dec 1, 2023
    10. 17 remaining items

    11. added a commit that references this issue on Dec 4, 2023
    12. added 2 commits that reference this issue on Dec 4, 2023
    13. added 2 commits that reference this issue on Dec 4, 2023
    14. added 4 commits that reference this issue on Feb 11, 2024
    15. added 4 commits that reference this issue on Sep 2, 2024
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Metadata

    Metadata

    Assignees

    Labels

    3.8 (EOL)end of lifetestsTests in the Lib/test dirtopic-IDLEtype-bugAn unexpected behavior, bug, or error

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions