Fix: Propiedades Cython para Tests de Interrupciones

Resumen

Se corrigieron tres tests de interrupciones que estaban fallando debido a que intentaban acceder a las propiedades ime y halted directamente en la instancia de PyCPU, pero el wrapper de Cython solo exponía métodos get_ime() y get_halted(). Se agregaron propiedades Python usando el decorador @property en el wrapper de Cython para permitir acceso directo a estos valores, manteniendo compatibilidad con los tests existentes.

Concepto de Hardware

El wrapper de Cython actúa como un puente entre Python y C++, permitiendo que el código Python acceda a funcionalidades implementadas en C++ de manera eficiente. En Python, es común acceder a propiedades de objetos usando la sintaxis de atributos (ej: cpu.ime) en lugar de métodos (ej: cpu.get_ime()), especialmente en tests donde se busca una API más natural y legible.

El decorador @property de Python permite convertir métodos en propiedades, permitiendo que el código acceda a valores como si fueran atributos, pero manteniendo la lógica de acceso encapsulada. Esto es especialmente útil en wrappers de Cython donde queremos exponer valores de C++ de manera natural desde Python.

Implementación

Se agregaron dos propiedades al wrapper de Cython PyCPU en src/core/cython/cpu.pyx:

Componentes modificados

• src/core/cython/cpu.pyx: Se agregaron las propiedades ime y halted usando el decorador @property.
• tests/test_core_cpu_interrupts.py: Se corrigió el test test_halt_wakeup_on_interrupt para reflejar el comportamiento correcto del hardware cuando la CPU despierta del HALT sin procesar la interrupción.

Decisiones de diseño

Se decidió mantener tanto los métodos get_ime() y get_halted() como las propiedades ime y halted para mantener compatibilidad con código existente y permitir una API más natural en los tests. Las propiedades simplemente llaman a los métodos getter subyacentes, manteniendo la lógica centralizada.

Además, se corrigió el test test_halt_wakeup_on_interrupt para reflejar el comportamiento correcto del hardware: cuando la CPU despierta del HALT sin procesar la interrupción (porque IME está desactivado), la CPU ejecuta la siguiente instrucción, por lo que el PC debe avanzar. Esto coincide con el comportamiento documentado en Pan Docs y con el test antiguo de Python.

Archivos Afectados

• src/core/cython/cpu.pyx - Agregadas propiedades ime y halted
• tests/test_core_cpu_interrupts.py - Corregido test test_halt_wakeup_on_interrupt

Tests y Verificación

Se ejecutaron todos los tests de interrupciones para verificar que el fix funciona correctamente:

pytest tests/test_core_cpu_interrupts.py -v

Resultado: 7 tests pasaron correctamente:

• ✅ test_di_disables_ime - PASSED
• ✅ test_ei_delayed_activation - PASSED
• ✅ test_halt_stops_execution - PASSED
• ✅ test_halt_wakeup_on_interrupt - PASSED
• ✅ test_interrupt_dispatch_vblank - PASSED
• ✅ test_interrupt_priority - PASSED
• ✅ test_all_interrupt_vectors - PASSED

Validación de módulo compilado C++: El módulo se recompiló exitosamente después de agregar las propiedades, confirmando que la sintaxis de Cython es correcta y que las propiedades se exponen correctamente desde C++.

Fuentes Consultadas

• Cython Documentation: https://cython.readthedocs.io/ - Decorador @property en clases Cython
• Python Documentation: https://docs.python.org/3/library/functions.html#property - Decorador @property
• Pan Docs: Comportamiento de HALT cuando hay interrupciones pendientes pero IME está desactivado

Integridad Educativa

Próximos Pasos

• [ ] Continuar con el análisis del trazado disparado para identificar opcodes no implementados
• [ ] Implementar los opcodes faltantes que bloquean el renderizado de gráficos