tests:

Support skipping tests that don't run under Windows;
Add test for decorated identifiers

Author: elifarley

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

@@ -0,0 +1,24 @@
+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
+    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.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.decorated-method/expected.1.py

@@ -0,0 +1,24 @@
+@comm_handler
+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/corpus/update.identifier.ed-script-filter/1.py tests/corpus/update.identifier.ed-script-filter!nowindows/1.py

@@ -1,5 +1,6 @@
 import re
 import shutil
+import sys
 import tempfile
 import pytest
 from pathlib import Path
@@ -40,6 +41,10 @@ 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')}")
+
     try:
         corpus_dir = Path(__file__).parent / 'corpus'
         test_dir = corpus_dir / test_case
@@ -93,7 +98,7 @@ def check_expected_files(dir_path: Path):
                 if str(rel_path).startswith("."):
                     continue
                 expected_file = test_dir / f"expected.{rel_path}"
-                assert expected_file.exists(), f"'expecteed.*' 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)