Apply review comments

Name the loop variable after what it holds in test_save_all_saves_files,
say what RMenuTest covers, and give Dummy_grep the simplifications it
makes and a comment for each class variable.

Dummy_grep keeps engine and close, which grep_it calls.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: serhiy-storchaka

Lib/idlelib/idle_test/test_config.py

@@ -774,8 +774,8 @@ def test_save_default(self):  # Cover 2nd and 3rd false branches.
     def test_save_all_saves_files(self):
         eq = self.assertEqual
         changes = self.changes
-        for parser in testcfg.values():
-            parser.Save = Func()
+        for config in testcfg.values():
+            config.Save = Func()
         try:
             # 'main', 'highlight' and 'keys' are saved even if unchanged.
             self.assertFalse(changes.save_all())
@@ -787,8 +787,8 @@ def test_save_all_saves_files(self):
             self.assertTrue(changes.save_all())
             eq(testcfg['extensions'].Save.called, 1)
         finally:
-            for parser in testcfg.values():
-                del parser.Save
+            for config in testcfg.values():
+                del config.Save
             userextn.remove_section('Esec')
 
     def test_delete_section(self):

Lib/idlelib/idle_test/test_editor.py

@@ -212,6 +212,9 @@ def test_searcher(self):
 
 
 class RMenuTest(unittest.TestCase):
+    # Test selection-rclick interaction in right_click_event and
+    # rmenu_check_copy(cut) status settings.  These are part of the rmenu
+    # functions common to all text windows with context windows.
 
     @classmethod
     def setUpClass(cls):

Lib/idlelib/idle_test/test_grep.py

@@ -31,12 +31,16 @@ def getprog(self):
 
 
 class Dummy_grep:
-    # Methods tested
-    default_command = grep.GrepDialog.default_command
+    # Simplifications: 1. Don't initialize superclass SearchEngineBase
+    # with searchengine.  2. Use directly set class vars instead of
+    # instance vars set from the GUI.
+    globvar = Var('')  # File name.
+    recvar = Var(False)  # Recurse down directories?
+    engine = searchengine  # Pattern and flags.
+    # Methods tested.
     grep_it = grep.GrepDialog.grep_it
-    # Other stuff needed
-    recvar = Var(False)
-    engine = searchengine
+    default_command = grep.GrepDialog.default_command  # Uses grep_it.
+
     def close(self):  # gui method
         pass