changeset: 101660:8090931ba850 branch: 2.7 parent: 101646:55886a6aed4b user: Terry Jan Reedy date: Fri Jun 03 13:33:09 2016 -0400 files: Lib/idlelib/idle_test/README.txt Lib/idlelib/idle_test/test_autocomplete.py Lib/idlelib/idle_test/test_autoexpand.py Lib/idlelib/idle_test/test_configdialog.py Lib/idlelib/idle_test/test_formatparagraph.py Lib/idlelib/idle_test/test_textview.py Lib/idlelib/idle_test/test_widgetredir.py description: Issue 20567: Revise idle_test/README.txt and some tests to match new advice. diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/README.txt --- a/Lib/idlelib/idle_test/README.txt Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/README.txt Fri Jun 03 13:33:09 2016 -0400 @@ -51,17 +51,20 @@ This will typically be setUpClass. To avoid interfering with other gui tests, all gui objects must be destroyed and -deleted by the end of the test. Widgets, such as a Tk root, created in a setUpX -function, should be destroyed in the corresponding tearDownX. Module and class -widget attributes should also be deleted.. +deleted by the end of the test. The Tk root created in a setUpX function should +be destroyed in the corresponding tearDownX and the module or class attribute +deleted. Others widgets should descend from the single root and the attributes +deleted BEFORE root is destroyed. See https://bugs.python.org/issue20567. @classmethod def setUpClass(cls): requires('gui') cls.root = tk.Tk() + cls.text = tk.Text(root) @classmethod def tearDownClass(cls): + del cls.text cls.root.destroy() del cls.root @@ -69,13 +72,15 @@ See https://bugs.python.org/issue18910 Requires('gui') causes the test(s) it guards to be skipped if any of -a few conditions are met: +these conditions are met: - The tests are being run by regrtest.py, and it was started without enabling the "gui" resource with the "-u" command line option. - The tests are being run on Windows by a service that is not allowed to interact with the graphical environment. + + - The tests are being run on Linux and X window is not available. - The tests are being run on Mac OSX in a process that cannot make a window manager connection. diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/test_autocomplete.py --- a/Lib/idlelib/idle_test/test_autocomplete.py Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/test_autocomplete.py Fri Jun 03 13:33:09 2016 -0400 @@ -33,9 +33,8 @@ @classmethod def tearDownClass(cls): + del cls.editor, cls.text cls.root.destroy() - del cls.text - del cls.editor del cls.root def setUp(self): diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/test_autoexpand.py --- a/Lib/idlelib/idle_test/test_autoexpand.py Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/test_autoexpand.py Fri Jun 03 13:33:09 2016 -0400 @@ -25,10 +25,10 @@ @classmethod def tearDownClass(cls): + del cls.text, cls.auto_expand if hasattr(cls, 'tk'): cls.tk.destroy() del cls.tk - del cls.text, cls.auto_expand def tearDown(self): self.text.delete('1.0', 'end') diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/test_configdialog.py --- a/Lib/idlelib/idle_test/test_configdialog.py Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/test_configdialog.py Fri Jun 03 13:33:09 2016 -0400 @@ -24,9 +24,8 @@ del cls.root def test_dialog(self): - d=ConfigDialog(self.root, 'Test', _utest=True) + d = ConfigDialog(self.root, 'Test', _utest=True) d.remove_var_callbacks() - d.destroy() if __name__ == '__main__': diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/test_formatparagraph.py --- a/Lib/idlelib/idle_test/test_formatparagraph.py Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/test_formatparagraph.py Fri Jun 03 13:33:09 2016 -0400 @@ -276,10 +276,9 @@ @classmethod def tearDownClass(cls): + del cls.text, cls.formatter cls.root.destroy() del cls.root - del cls.text - del cls.formatter def test_short_line(self): self.text.insert('1.0', "Short line\n") diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/test_textview.py --- a/Lib/idlelib/idle_test/test_textview.py Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/test_textview.py Fri Jun 03 13:33:09 2016 -0400 @@ -23,10 +23,9 @@ @classmethod def tearDownClass(cls): + del cls.TV cls.root.destroy() - TV = cls.TV - del cls.root, cls.TV - del TV.transient, TV.grab_set, TV.wait_window + del cls.root def setUp(self): TV = self.TV diff -r 55886a6aed4b -r 8090931ba850 Lib/idlelib/idle_test/test_widgetredir.py --- a/Lib/idlelib/idle_test/test_widgetredir.py Thu Jun 02 23:51:22 2016 -0700 +++ b/Lib/idlelib/idle_test/test_widgetredir.py Fri Jun 03 13:33:09 2016 -0400 @@ -14,14 +14,14 @@ @classmethod def setUpClass(cls): requires('gui') - cls.tk = Tk() - cls.text = Text(cls.tk) + cls.root = Tk() + cls.text = Text(cls.root) @classmethod def tearDownClass(cls): - cls.text.destroy() - cls.tk.destroy() - del cls.text, cls.tk + del cls.text + cls.root.destroy() + del cls.root def test_init(self): redir = WidgetRedirector(self.text) @@ -43,14 +43,14 @@ @classmethod def setUpClass(cls): requires('gui') - cls.tk = Tk() - cls.text = Text(cls.tk) + cls.root = Tk() + cls.text = Text(cls.root) @classmethod def tearDownClass(cls): - cls.text.destroy() - cls.tk.destroy() - del cls.text, cls.tk + del cls.text + cls.root.destroy() + del cls.root def setUp(self): self.redir = WidgetRedirector(self.text) @@ -108,13 +108,13 @@ def test_command_dispatch(self): # Test that .__init__ causes redirection of tk calls # through redir.dispatch - self.tk.call(self.text._w, 'insert', 'hello') + self.root.call(self.text._w, 'insert', 'hello') self.assertEqual(self.func.args, ('hello',)) self.assertEqual(self.text.get('1.0', 'end'), '\n') # Ensure that called through redir .dispatch and not through # self.text.insert by having mock raise TclError. self.func.__init__(TclError()) - self.assertEqual(self.tk.call(self.text._w, 'insert', 'boo'), '') + self.assertEqual(self.root.call(self.text._w, 'insert', 'boo'), '')