Issue #28572: Use system-specific values for configdialog font test

diff --git a/Lib/idlelib/config-main.def b/Lib/idlelib/config-main.def
index 330c0154a7b7e876e976dde8d254e1d315ed2507..16f4b0959cf13cc9f198f0426275bb884dff9cae 100644 (file)
--- a/Lib/idlelib/config-main.def
+++ b/Lib/idlelib/config-main.def
@@ -60,6 +60,8 @@ delete-exitfunc= 1
 width= 80
 height= 40
 font= TkFixedFont
+# For TkFixedFont, the actual size and boldness are obtained from tk
+# and override 10 and 0.  See idlelib.config.IdleConf.GetFont
 font-size= 10
 font-bold= 0
 encoding= none

diff --git a/Lib/idlelib/idle_test/test_configdialog.py b/Lib/idlelib/idle_test/test_configdialog.py
index 81c57e863e531c8e96db756119c31bc1950f08ff..3f94493733ba0b6e5e3db00fc1fbe23a2d35f2d9 100644 (file)
--- a/Lib/idlelib/idle_test/test_configdialog.py
+++ b/Lib/idlelib/idle_test/test_configdialog.py
@@ -54,25 +54,29 @@ class FontTabTest(unittest.TestCase):
         changes.clear()
 
     def test_font(self):
+        # Set values guaranteed not to be defaults.
+        dfont = idleConf.GetFont(root, 'main', 'EditorWindow')
+        dsize = str(dfont[1])
+        dbold = dfont[2] == 'bold'
         configure.fontName.set('Test Font')
         expected = [
             ('main', 'EditorWindow', 'font', 'Test Font'),
-            ('main', 'EditorWindow', 'font-size', '10'),
-            ('main', 'EditorWindow', 'font-bold', False)]
+            ('main', 'EditorWindow', 'font-size', dsize),
+            ('main', 'EditorWindow', 'font-bold', dbold)]
         self.assertEqual(changes, expected)
         changes.clear()
-        configure.fontSize.set(12)
+        configure.fontSize.set(20)
         expected = [
             ('main', 'EditorWindow', 'font', 'Test Font'),
-            ('main', 'EditorWindow', 'font-size', '12'),
-            ('main', 'EditorWindow', 'font-bold', False)]
+            ('main', 'EditorWindow', 'font-size', '20'),
+            ('main', 'EditorWindow', 'font-bold', dbold)]
         self.assertEqual(changes, expected)
         changes.clear()
-        configure.fontBold.set(True)
+        configure.fontBold.set(not dbold)
         expected = [
             ('main', 'EditorWindow', 'font', 'Test Font'),
-            ('main', 'EditorWindow', 'font-size', '12'),
-            ('main', 'EditorWindow', 'font-bold', True)]
+            ('main', 'EditorWindow', 'font-size', '20'),
+            ('main', 'EditorWindow', 'font-bold', not dbold)]
         self.assertEqual(changes, expected)
 
     #def test_sample(self): pass  # TODO