changeset:   102122:45d4cea97b04
branch:      2.7
user:        Benjamin Peterson <benjamin@python.org>
date:        Tue Jun 21 23:12:54 2016 -0700
files:       Lib/idlelib/aboutDialog.py Lib/idlelib/idle_test/test_helpabout.py Lib/idlelib/textView.py Misc/NEWS
description:
fix idle about dialog #27365


diff -r ad037bba60bb -r 45d4cea97b04 Lib/idlelib/aboutDialog.py
--- a/Lib/idlelib/aboutDialog.py	Wed Jun 15 17:12:38 2016 -0500
+++ b/Lib/idlelib/aboutDialog.py	Tue Jun 21 23:12:54 2016 -0700
@@ -129,7 +129,7 @@
         self.display_file_text('About - Readme', 'README.txt', 'ascii')
 
     def ShowIDLENEWS(self):
-        self.display_file_text('About - NEWS', 'NEWS.txt', 'ascii')
+        self.display_file_text('About - NEWS', 'NEWS.txt', 'utf-8')
 
     def display_printer_text(self, title, printer):
         printer._Printer__setup()
@@ -144,5 +144,7 @@
         self.destroy()
 
 if __name__ == '__main__':
+    import unittest
+    unittest.main('idlelib.idle_test.test_helpabout', verbosity=2, exit=False)
     from idlelib.idle_test.htest import run
     run(AboutDialog)
diff -r ad037bba60bb -r 45d4cea97b04 Lib/idlelib/idle_test/test_helpabout.py
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/Lib/idlelib/idle_test/test_helpabout.py	Tue Jun 21 23:12:54 2016 -0700
@@ -0,0 +1,52 @@
+'''Test idlelib.help_about.
+
+Coverage:
+'''
+from idlelib import aboutDialog as help_about
+from idlelib import textView as textview
+from idlelib.idle_test.mock_idle import Func
+from idlelib.idle_test.mock_tk import Mbox
+import unittest
+
+About = help_about.AboutDialog
+class Dummy_about_dialog():
+    # Dummy class for testing file display functions.
+    idle_credits = About.ShowIDLECredits.im_func
+    idle_readme = About.ShowIDLEAbout.im_func
+    idle_news = About.ShowIDLENEWS.im_func
+    # Called by the above
+    display_file_text = About.display_file_text.im_func
+
+
+class DisplayFileTest(unittest.TestCase):
+    "Test that .txt files are found and properly decoded."
+    dialog = Dummy_about_dialog()
+
+    @classmethod
+    def setUpClass(cls):
+        cls.orig_mbox = textview.tkMessageBox
+        cls.orig_view = textview.view_text
+        cls.mbox = Mbox()
+        cls.view = Func()
+        textview.tkMessageBox = cls.mbox
+        textview.view_text = cls.view
+        cls.About = Dummy_about_dialog()
+
+    @classmethod
+    def tearDownClass(cls):
+        textview.tkMessageBox = cls.orig_mbox
+        textview.view_text = cls.orig_view.im_func
+
+    def test_file_isplay(self):
+        for handler in (self.dialog.idle_credits,
+                        self.dialog.idle_readme,
+                        self.dialog.idle_news):
+            self.mbox.showerror.message = ''
+            self.view.called = False
+            handler()
+            self.assertEqual(self.mbox.showerror.message, '')
+            self.assertEqual(self.view.called, True)
+
+
+if __name__ == '__main__':
+    unittest.main(verbosity=2)
diff -r ad037bba60bb -r 45d4cea97b04 Lib/idlelib/textView.py
--- a/Lib/idlelib/textView.py	Wed Jun 15 17:12:38 2016 -0500
+++ b/Lib/idlelib/textView.py	Tue Jun 21 23:12:54 2016 -0700
@@ -79,6 +79,10 @@
         tkMessageBox.showerror(title='File Load Error',
                                message='Unable to load file %r .' % filename,
                                parent=parent)
+    except UnicodeDecodeError as err:
+        showerror(title='Unicode Decode Error',
+                  message=str(err),
+                  parent=parent)
     else:
         return view_text(parent, title, textFile.read(), modal)
 
diff -r ad037bba60bb -r 45d4cea97b04 Misc/NEWS
--- a/Misc/NEWS	Wed Jun 15 17:12:38 2016 -0500
+++ b/Misc/NEWS	Tue Jun 21 23:12:54 2016 -0700
@@ -10,6 +10,11 @@
 
 - Issue #26930: Update Windows builds to use OpenSSL 1.0.2h.
 
+IDLE
+----
+
+- Issue #27365: Fix about dialog.
+
 
 What's New in Python 2.7.12 release candidate 1?
 ================================================