[Python-checkins] cpython (merge 3.3 -> default): Issue #6160: The bbox() method of tkinter.Spinbox now returns a tuple of

[serhiy.storchaka]

http://hg.python.org/cpython/rev/75d8b9136fa6
changeset:   86874:75d8b9136fa6
parent:      86871:6e592d972b86
parent:      86873:5bdbf2258563
user:        Serhiy Storchaka <storchaka at gmail.com>
date:        Sun Nov 03 14:15:00 2013 +0200
summary:
  Issue #6160: The bbox() method of tkinter.Spinbox now returns a tuple of
integers instead of a string.  Based on patch by Guilherme Polo.

files:
  Lib/tkinter/__init__.py                       |   2 +-
  Lib/tkinter/test/test_tkinter/test_widgets.py |  12 ++++++++++
  Misc/NEWS                                     |   3 ++
  3 files changed, 16 insertions(+), 1 deletions(-)


diff --git a/Lib/tkinter/__init__.py b/Lib/tkinter/__init__.py
--- a/Lib/tkinter/__init__.py
+++ b/Lib/tkinter/__init__.py
@@ -3472,7 +3472,7 @@
         bounding box may refer to a region outside the
         visible area of the window.
         """
-        return self.tk.call(self._w, 'bbox', index)
+        return self._getints(self.tk.call(self._w, 'bbox', index)) or None
 
     def delete(self, first, last=None):
         """Delete one or more elements of the spinbox.
diff --git a/Lib/tkinter/test/test_tkinter/test_widgets.py b/Lib/tkinter/test/test_tkinter/test_widgets.py
--- a/Lib/tkinter/test/test_tkinter/test_widgets.py
+++ b/Lib/tkinter/test/test_tkinter/test_widgets.py
@@ -457,6 +457,18 @@
         widget = self.create()
         self.checkBooleanParam(widget, 'wrap')
 
+    def test_bbox(self):
+        widget = self.create()
+        bbox = widget.bbox(0)
+        self.assertEqual(len(bbox), 4)
+        for item in bbox:
+            self.assertIsInstance(item, int)
+
+        self.assertRaises(tkinter.TclError, widget.bbox, 'noindex')
+        self.assertRaises(tkinter.TclError, widget.bbox, None)
+        self.assertRaises(TypeError, widget.bbox)
+        self.assertRaises(TypeError, widget.bbox, 0, 1)
+
 
 @add_standard_options(StandardOptionsTests)
 class TextTest(AbstractWidgetTest, unittest.TestCase):
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -31,6 +31,9 @@
 Library
 -------
 
+- Issue #6160: The bbox() method of tkinter.Spinbox now returns a tuple of
+  integers instead of a string.  Based on patch by Guilherme Polo.
+
 - Issue #19403: contextlib.redirect_stdout is now reentrant
 
 - Issue #19286: Directories in ``package_data`` are no longer added to

-- 
Repository URL: http://hg.python.org/cpython