Add test for cross-module getter→setter enum roundtrip

The existing test_cross_module_scoped_enum_imports test had a gap:
it didn't test the scenario where a class in module B uses both
getter and setter methods for an enum defined in module A.

This is the exact pattern that failed in pyOpenMS:
  s.setType(s.getType())  # Cross-module roundtrip

Changes:
- Add StatusTracker class to EnumConsumer with getter/setter for
  enums from EnumProvider (both Task::TaskStatus and Priority)
- Add Runtime Test 8 that validates tracker.setX(tracker.getX())
  works correctly across module boundaries
- Update test assertions to check for _get_scoped_enum_class()
  pattern instead of the old globals().get() pattern

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

Author: timosachsenberg

tests/test_code_generator.py

@@ -861,13 +861,14 @@ def test_cross_module_scoped_enum_imports(tmpdir):
             f"EnumConsumer.pyx content:\n{consumer_pyx}"
         )
 
-        # Test 2: Verify isinstance checks use globals().get() for late binding
-        assert "globals().get('_PyTask_TaskStatus'" in consumer_pyx, (
-            f"Expected isinstance check with globals().get('_PyTask_TaskStatus', int) for wrap-attach enum.\n"
+        # Test 2: Verify isinstance checks use _get_scoped_enum_class() helper for late binding
+        # The helper looks up enums via registry and sys.modules for cross-module support
+        assert "_get_scoped_enum_class('_PyTask_TaskStatus')" in consumer_pyx, (
+            f"Expected isinstance check with _get_scoped_enum_class('_PyTask_TaskStatus') for wrap-attach enum.\n"
             f"EnumConsumer.pyx content:\n{consumer_pyx}"
         )
-        assert "globals().get('Priority'" in consumer_pyx, (
-            f"Expected isinstance check with globals().get('Priority', int) for non-wrap-attach enum.\n"
+        assert "_get_scoped_enum_class('Priority')" in consumer_pyx, (
+            f"Expected isinstance check with _get_scoped_enum_class('Priority') for non-wrap-attach enum.\n"
             f"EnumConsumer.pyx content:\n{consumer_pyx}"
         )
 
@@ -976,6 +977,27 @@ def test_cross_module_scoped_enum_imports(tmpdir):
             except AssertionError as e:
                 assert "wrong type" in str(e)
 
+            # Runtime Test 8: Cross-module getter→setter roundtrip (the pyOpenMS scenario)
+            # This tests that a class in module B can use getX()/setX() with an enum from module A
+            # where setX(getX()) works correctly - this was the missing test case.
+            tracker = EnumConsumer.StatusTracker()
+
+            # Test with class-attached enum (Task::TaskStatus)
+            assert tracker.getStatus() == EnumProvider.Task.TaskStatus.PENDING
+            tracker.setStatus(EnumProvider.Task.TaskStatus.RUNNING)
+            assert tracker.getStatus() == EnumProvider.Task.TaskStatus.RUNNING
+            # THE KEY TEST: getter→setter roundtrip across module boundaries
+            tracker.setStatus(tracker.getStatus())  # This is what failed in pyOpenMS!
+            assert tracker.getStatus() == EnumProvider.Task.TaskStatus.RUNNING
+
+            # Test with standalone enum (Priority)
+            assert tracker.getPriority() == EnumProvider.Priority.LOW
+            tracker.setPriority(EnumProvider.Priority.HIGH)
+            assert tracker.getPriority() == EnumProvider.Priority.HIGH
+            # THE KEY TEST: getter→setter roundtrip across module boundaries
+            tracker.setPriority(tracker.getPriority())  # This is what failed in pyOpenMS!
+            assert tracker.getPriority() == EnumProvider.Priority.HIGH
+
             print("Test passed: Cross-module scoped enum imports work correctly at runtime!")
 
         finally:

tests/test_files/enum_cross_module/EnumConsumer.hpp

@@ -35,4 +35,29 @@ class TaskRunner {
     }
 };
 
+/**
+ * StatusTracker: Tests cross-module getter→setter roundtrip.
+ *
+ * This class lives in EnumConsumer module but uses enums from EnumProvider
+ * for BOTH getter AND setter methods. This tests the scenario where:
+ *   tracker.setStatus(tracker.getStatus())
+ * must work correctly across module boundaries.
+ */
+class StatusTracker {
+public:
+    StatusTracker() : status_(Task::TaskStatus::PENDING), priority_(Priority::LOW) {}
+
+    // Getter and setter for class-attached enum (Task::TaskStatus)
+    void setStatus(Task::TaskStatus s) { status_ = s; }
+    Task::TaskStatus getStatus() const { return status_; }
+
+    // Getter and setter for standalone enum (Priority)
+    void setPriority(Priority p) { priority_ = p; }
+    Priority getPriority() const { return priority_; }
+
+private:
+    Task::TaskStatus status_;
+    Priority priority_;
+};
+
 #endif // ENUM_CONSUMER_HPP

tests/test_files/enum_cross_module/EnumConsumer.pxd

@@ -16,3 +16,13 @@ cdef extern from "EnumConsumer.hpp":
         bool isCompleted(Task_TaskStatus s)
         Priority getDefaultPriority()
         Task_TaskStatus getDefaultStatus()
+
+    # StatusTracker: Tests cross-module getter→setter roundtrip
+    # This class has getter AND setter for enums from EnumProvider,
+    # testing that tracker.setStatus(tracker.getStatus()) works.
+    cdef cppclass StatusTracker:
+        StatusTracker()
+        void setStatus(Task_TaskStatus s)
+        Task_TaskStatus getStatus()
+        void setPriority(Priority p)
+        Priority getPriority()