diff options
| author | Akuli <akuviljanen17@gmail.com> | 2020-07-27 01:48:17 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-07-27 01:48:17 (GMT) |
| commit | f1d40f941a6483b1d4ea10f1051ace7b426fb8e7 (patch) | |
| tree | 6994171a9e8539ec7478bb51689683505b97fa1e /Lib/tkinter | |
| parent | 52bf4706a3e35dd001f12e102c4082c4987daad5 (diff) | |
| download | cpython-f1d40f941a6483b1d4ea10f1051ace7b426fb8e7.zip cpython-f1d40f941a6483b1d4ea10f1051ace7b426fb8e7.tar.gz cpython-f1d40f941a6483b1d4ea10f1051ace7b426fb8e7.tar.bz2 | |
bpo-41384: Raise TclError in tkinter.OptionMenu (GH-21601)
... when an unknown option is passed. TypeError was being raised because a 2to3 fix was missing.
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
Diffstat (limited to 'Lib/tkinter')
| -rw-r--r-- | Lib/tkinter/__init__.py | 2 | ||||
| -rw-r--r-- | Lib/tkinter/test/test_tkinter/test_widgets.py | 4 |
2 files changed, 5 insertions, 1 deletions
diff --git a/Lib/tkinter/__init__.py b/Lib/tkinter/__init__.py index a3378d0..1067ab6 100644 --- a/Lib/tkinter/__init__.py +++ b/Lib/tkinter/__init__.py @@ -3965,7 +3965,7 @@ class OptionMenu(Menubutton): if 'command' in kwargs: del kwargs['command'] if kwargs: - raise TclError('unknown option -'+kwargs.keys()[0]) + raise TclError('unknown option -'+next(iter(kwargs))) menu.add_command(label=value, command=_setit(variable, value, callback)) for v in values: diff --git a/Lib/tkinter/test/test_tkinter/test_widgets.py b/Lib/tkinter/test/test_tkinter/test_widgets.py index 16e9d93..721e813 100644 --- a/Lib/tkinter/test/test_tkinter/test_widgets.py +++ b/Lib/tkinter/test/test_tkinter/test_widgets.py @@ -307,6 +307,10 @@ class OptionMenuTest(MenubuttonTest, unittest.TestCase): def create(self, default='b', values=('a', 'b', 'c'), **kwargs): return tkinter.OptionMenu(self.root, None, default, *values, **kwargs) + def test_bad_kwarg(self): + with self.assertRaisesRegex(TclError, r"^unknown option -image$"): + tkinter.OptionMenu(self.root, None, 'b', image='') + @add_standard_options(IntegerSizeTests, StandardOptionsTests) class EntryTest(AbstractWidgetTest, unittest.TestCase): |