Repository navigation
Commit 8486016
The SimpleDialog keyboard tests generate key events after focus_force(),
which on Windows are dropped until the toplevel is mapped, so they could
fail intermittently (seen on the Windows10 buildbot as
test_return_no_default). Wait until the window is mapped in these tests,
as GH-152599 did for the other keyboard tests.
(cherry picked from commit 7d619c2)
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 9daaae8 commit 8486016
1 file changed
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -49,6 +49,7 @@ def test_default_button(self): | |||
| 49 | 49 | def test_return_activates_default(self): | |
| 50 | 50 | # <Return> invokes the default button. | |
| 51 | 51 | d = self.create() # default 0 | |
| 52 | + self.require_mapped(d.root) | ||
| 52 | 53 | d.root.focus_force() | |
| 53 | 54 | d.root.update() | |
| 54 | 55 | d.root.event_generate('<Return>') | |
@@ -59,6 +60,7 @@ def test_return_no_default(self): | |||
| 59 | 60 | # With no default button, <Return> rings the bell and leaves the dialog | |
| 60 | 61 | # open instead of activating a button. | |
| 61 | 62 | d = self.create(default=None) | |
| 63 | + self.require_mapped(d.root) | ||
| 62 | 64 | d.root.focus_force() | |
| 63 | 65 | d.root.update() | |
| 64 | 66 | bells = [] | |
0 commit comments