test: double-decorated identifier

Author: elifarley

tests/corpus/move.double-decorated-method/1.py

@@ -0,0 +1,28 @@
+import functools
+
+
+class SpyderKernel(IPythonKernel):
+    """Spyder kernel for Jupyter."""
+
+    shell_class = SpyderShell
+    @comm_handler
+    def safe_exec(self, filename):
+        """Safely execute a file using IPKernelApp._exec_file."""
+        self.parent._exec_file(filename)
+
+    @comm_handler
+    @functools.lru_cache(32)
+    def get_fault_text(self, fault_filename, main_id, ignore_ids):
+        """Get fault text from old run."""
+        # Read file
+        try:
+            with open(fault_filename, 'r') as f:
+                fault = f.read()
+        except FileNotFoundError:
+            return
+        return text
+
+    def get_system_threads_id(self):
+        """Return the list of system threads id."""
+        ignore_threads = [
+        ]

tests/corpus/move.double-decorated-method/chat.xml

@@ -0,0 +1,20 @@
+<no-train>
+```CEDARScript
+-- 1. Move the `get_fault_text` method from the `SpyderKernel` class to be a top-level function
+UPDATE METHOD "SpyderKernel.get_fault_text"
+FROM FILE "1.py"
+MOVE WHOLE
+INSERT BEFORE CLASS "SpyderKernel"
+RELATIVE INDENTATION 0;
+
+-- 2. Update the copied function to remove references to `self`
+UPDATE FUNCTION "get_fault_text"
+FROM FILE r"1.py"
+REPLACE WHOLE WITH CASE
+WHEN REGEX r'''def get_fault_text\(''' THEN SUB
+r'''def get_fault_text\(self, fault_filename, main_id, ignore_ids\):'''
+r'''def get_fault_text(fault_filename, main_id, ignore_ids):'''
+END;
+
+```
+</no-train>

tests/corpus/move.double-decorated-method/expected.1.py

@@ -0,0 +1,28 @@
+import functools
+
+
+@comm_handler
+@functools.lru_cache(32)
+def get_fault_text(fault_filename, main_id, ignore_ids):
+    """Get fault text from old run."""
+    # Read file
+    try:
+        with open(fault_filename, 'r') as f:
+            fault = f.read()
+    except FileNotFoundError:
+        return
+    return text
+class SpyderKernel(IPythonKernel):
+    """Spyder kernel for Jupyter."""
+
+    shell_class = SpyderShell
+    @comm_handler
+    def safe_exec(self, filename):
+        """Safely execute a file using IPKernelApp._exec_file."""
+        self.parent._exec_file(filename)
+
+
+    def get_system_threads_id(self):
+        """Return the list of system threads id."""
+        ignore_threads = [
+        ]

tests/test_corpus.py

@@ -7,6 +7,7 @@
 
 from cedarscript_editor import find_commands, CEDARScriptEditor
 
+_no_windows = '!nowindows'
 
 def get_test_cases() -> list[str]:
     """Get all test cases from tests/corpus directory.
@@ -41,9 +42,8 @@ def editor(tmp_path_factory):
 @pytest.mark.parametrize('test_case', get_test_cases())
 def test_corpus(editor: CEDARScriptEditor, test_case: str):
     """Test CEDARScript commands from chat.xml files in corpus."""
-    if test_case.casefold().endswith('!nowindows'):
-        if sys.platform == 'win32':
-            pytest.skip(f"Cannot run under Windows: {test_case.removesuffix('!nowindows')}")
+    if test_case.casefold().endswith(_no_windows) and sys.platform == 'win32':
+        pytest.skip(f"Cannot run under Windows: {test_case.removesuffix(_no_windows)}")
 
     try:
         corpus_dir = Path(__file__).parent / 'corpus'
@@ -95,10 +95,10 @@ def check_expected_files(dir_path: Path):
                     continue
                 # Find corresponding expected file in test directory
                 rel_path = path.relative_to(editor.root_path)
-                if str(rel_path).startswith("."):
+                if str(rel_path).startswith(".") or str(rel_path).endswith("~"):
                     continue
                 expected_file = test_dir / f"expected.{rel_path}"
-                assert expected_file.exists(), f"'expected.*' file not found: {expected_file}"
+                assert expected_file.exists(), f"'expected.*' file not found: '{expected_file}'"
 
                 expected_content = file_to_lines(expected_file, rel_path)
                 actual_content = file_to_lines(path, rel_path)