Movatterモバイル変換


[0]ホーム

URL:


homepage

Issue21939

This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title:IDLE - Test Percolator
Type:enhancementStage:resolved
Components:IDLEVersions:Python 3.6, Python 3.5
process
Status:closedResolution:fixed
Dependencies:Superseder:
Assigned To: terry.reedyNosy List: Saimadhav.Heblikar, jesstess, python-dev, taleinat, terry.reedy
Priority:normalKeywords:patch

Created on2014-07-08 14:19 bySaimadhav.Heblikar, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.

Files
File nameUploadedDescriptionEdit
test_percolator-34.diffSaimadhav.Heblikar,2014-07-08 14:19review
test_percolator-34-v2.diffSaimadhav.Heblikar,2014-07-17 07:00review
Messages (4)
msg222561 -(view)Author: Saimadhav Heblikar (Saimadhav.Heblikar)*Date: 2014-07-08 14:19
Attached is a unittest for idlelib.Percolator.2.7 version will be added once this is OK.
msg222730 -(view)Author: Tal Einat (taleinat)*(Python committer)Date: 2014-07-11 09:08
I've reviewed the patch and made my remarks in the review tool.These tests don't test the central functionality of Percolator nearly enough. We should test, at least:1) That the text actually went through the filter (and not directly to the Textwidget).2) That if a filter modifies the text, the modified text appears in the Textwidget.3) That a filter can stop the event from continuing through the followingfilters.4) That having more than one filter modify the arguments works.5) That not having any filters works.6) That this doesn't only work on a Text widget's "insert" and "delete" events.There are probably more things we should test. Take a look at what this is usedfor in practice in the code for ideas.
msg265800 -(view)Author: Roundup Robot (python-dev)(Python triager)Date: 2016-05-18 02:48
New changeseteebccf05903e by Terry Jan Reedy in branch '3.5':Issue#21939: Test IDLE percolator.  Original patch by Saimadhav Heblikar.https://hg.python.org/cpython/rev/eebccf05903e
msg265801 -(view)Author: Terry J. Reedy (terry.reedy)*(Python committer)Date: 2016-05-18 02:54
Tal, thanks for the suggestions.  Saimadhav added tests for at least some of them.  Part of the difficulty in adding IDLE tests is that it is too easy to write tests to the code instead of the use of the code.  This is especially true when side-effects (changing a widget) are the main intended effect.  I plan to work more on the latter.
History
DateUserActionArgs
2022-04-11 14:58:05adminsetgithub: 66138
2016-05-18 02:54:50terry.reedysetstatus: open -> closed

assignee:terry.reedy
versions: + Python 3.6, - Python 2.7, Python 3.4
messages: +msg265801
type: enhancement
resolution: fixed
stage: resolved
2016-05-18 02:48:54python-devsetnosy: +python-dev
messages: +msg265800
2014-07-17 07:00:27Saimadhav.Heblikarsetfiles: +test_percolator-34-v2.diff
2014-07-11 09:08:23taleinatsetmessages: +msg222730
2014-07-08 14:19:46Saimadhav.Heblikarcreate
Supported byThe Python Software Foundation,
Powered byRoundup
Copyright © 1990-2022,Python Software Foundation
Legal Statements

[8]ページ先頭

©2009-2026 Movatter.jp