search

python / cpython

The Python programming language
https://www.python.org
Other
63.17k stars 30.25k forks source link

Add tests for Tix #65536

Closed zware closed 8 years ago

zware commented 10 years ago
BPO 21337
Nosy @terryjreedy, @zware, @serhiy-storchaka

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: ```python assignee = None closed_at = created_at = labels = ['easy', 'tests', 'type-feature', 'expert-tkinter'] title = 'Add tests for Tix' updated_at = user = 'https://github.com/zware' ``` bugs.python.org fields: ```python activity = actor = 'zach.ware' assignee = 'none' closed = True closed_date = closer = 'zach.ware' components = ['Tests', 'Tkinter'] creation = creator = 'zach.ware' dependencies = [] files = [] hgrepos = [] issue_num = 21337 keywords = ['easy'] message_count = 5.0 messages = ['217089', '233367', '243448', '243449', '276511'] nosy_count = 4.0 nosy_names = ['terry.reedy', 'python-dev', 'zach.ware', 'serhiy.storchaka'] pr_nums = [] priority = 'low' resolution = 'out of date' stage = 'resolved' status = 'closed' superseder = None type = 'enhancement' url = 'https://bugs.python.org/issue21337' versions = ['Python 3.5'] ```

zware commented 10 years ago

We should have some tests for Tix, which currently has none. The Windows buildbots will be able to run the tests, but Tix is not guaranteed to be available elsewhere.

terryjreedy commented 9 years ago

A minimal test would be that the one in the doc.

from tkinter import tix
root = tix.Tk()
root.tk.eval('package require Tix')

This passes on my 3.4.2 win7. I believe the first line should work on any system with _tkinter, whereas https://stackoverflow.com/questions/27751923/tix-widgets-installation-issue

reports failure of the second line on his Mac with ... self.tk.eval('package require Tix') _tkinter.TclError: can't find package Tix

1762cc99-3127-4a62-9baf-30c3d0f51ef7 commented 9 years ago

New changeset eeea91d0506b by Zachary Ware in branch 'default': Issue bpo-21337: Add bare-bones Tix test https://hg.python.org/cpython/rev/eeea91d0506b

zware commented 9 years ago

As a prerequisite for bpo-20035, I've added the simplest test possible that just makes sure Tix is available on Windows. I only added it on default as I don't have the facilities to test it properly on 2.7 and 3.4 on Windows (which is the only platform actually testing anything so far), but it should be simple enough to copy it over if anybody wants to or ever writes some real tests for Tix.

Leaving this issue open as some real tests would still be nice.

zware commented 8 years ago

Tix is now deprecated, extra tests are unnecessary.