Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add tests for Tix #65536

Closed
zware opened this issue Apr 23, 2014 · 5 comments
Closed

Add tests for Tix #65536

zware opened this issue Apr 23, 2014 · 5 comments
Labels
easy tests Tests in the Lib/test dir topic-tkinter type-feature A feature request or enhancement

Comments

@zware
Copy link
Member

zware commented Apr 23, 2014

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:

assignee = None
closed_at = <Date 2016-09-15.03:37:42.130>
created_at = <Date 2014-04-23.20:46:48.419>
labels = ['easy', 'tests', 'type-feature', 'expert-tkinter']
title = 'Add tests for Tix'
updated_at = <Date 2016-09-15.03:37:42.128>
user = 'https://github.com/zware'

bugs.python.org fields:

activity = <Date 2016-09-15.03:37:42.128>
actor = 'zach.ware'
assignee = 'none'
closed = True
closed_date = <Date 2016-09-15.03:37:42.130>
closer = 'zach.ware'
components = ['Tests', 'Tkinter']
creation = <Date 2014-04-23.20:46:48.419>
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
Copy link
Member Author

zware commented Apr 23, 2014

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.

@zware zware added tests Tests in the Lib/test dir topic-tkinter easy type-feature A feature request or enhancement labels Apr 23, 2014
@terryjreedy
Copy link
Member

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

@python-dev
Copy link
Mannequin

python-dev mannequin commented May 18, 2015

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
Copy link
Member Author

zware commented May 18, 2015

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
Copy link
Member Author

zware commented Sep 15, 2016

Tix is now deprecated, extra tests are unnecessary.

@zware zware closed this as completed Sep 15, 2016
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easy tests Tests in the Lib/test dir topic-tkinter type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zware @terryjreedy