New submission from Terry J. Reedy <tjre...@udel.edu>:

PR 10454 for #35196 added, among other things, more tests to test_squeezer.py.  
SqueezerTest.test_reload initially worked on Mac and personal Windows machines. 
It failed on Cheryl Sabella's personal Ubuntu machine because doubling the 
nominal font size did not necessarily exactly double the reported pixel size of 
'0'.  This was easily fixed by testing only that the size increased.

self.assertGreater(squeezer.zero_char_width, orig_zero_char_width)

It failed on CI Linux and Windows machines because the pixel size did not 
increase at all. This was fixed for the CI machines by directly assigning a new 
font tuple to text['font'] instead of involving the idleConf machinery.  
However, after merging, it failed with the same error that previously occurred 
on the CI machines.

AssertionError: 6 not greater than 6.

The initial fix will be to disable the assertion.

----------
assignee: terry.reedy
components: IDLE
messages: 333558
nosy: taleinat, terry.reedy
priority: normal
severity: normal
stage: needs patch
status: open
title: IDLE: Fix squeezer test_reload.
type: behavior
versions: Python 3.7, Python 3.8

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue35730>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to