McRogueFace/tests/issue_12_gridpoint_instantiation_test.py
John McCardle cd0bd5468b Squashed commit of the following: [alpha_streamline_1]
the low-hanging fruit of pre-existing issues and standardizing the
Python interfaces

Special thanks to Claude Code, ~100k output tokens for this merge

    🤖 Generated with [Claude Code](https://claude.ai/code)
    Co-Authored-By: Claude <noreply@anthropic.com>

commit 99f301e3a0
Author: John McCardle <mccardle.john@gmail.com>
Date:   Sat Jul 5 16:25:32 2025 -0400

    Add position tuple support and pos property to UI elements

    closes #83, closes #84

    - Issue #83: Add position tuple support to constructors
      - Frame and Sprite now accept both (x, y) and ((x, y)) forms
      - Also accept Vector objects as position arguments
      - Caption and Entity already supported tuple/Vector forms
      - Uses PyVector::from_arg for flexible position parsing

    - Issue #84: Add pos property to Frame and Sprite
      - Added pos getter that returns a Vector
      - Added pos setter that accepts Vector or tuple
      - Provides consistency with Caption and Entity which already had pos properties
      - All UI elements now have a uniform way to get/set positions as Vectors

    Both features improve API consistency and make it easier to work with positions.

commit 2f2b488fb5
Author: John McCardle <mccardle.john@gmail.com>
Date:   Sat Jul 5 16:18:10 2025 -0400

    Standardize sprite_index property and add scale_x/scale_y to UISprite

    closes #81, closes #82

    - Issue #81: Standardized property name to sprite_index across UISprite and UIEntity
      - Added sprite_index as the primary property name
      - Kept sprite_number as a deprecated alias for backward compatibility
      - Updated repr() methods to use sprite_index
      - Updated animation system to recognize both names

    - Issue #82: Added scale_x and scale_y properties to UISprite
      - Enables non-uniform scaling of sprites
      - scale property still works for uniform scaling
      - Both properties work with the animation system

    All existing code using sprite_number continues to work due to backward compatibility.

commit 5a003a9aa5
Author: John McCardle <mccardle.john@gmail.com>
Date:   Sat Jul 5 16:09:52 2025 -0400

    Fix multiple low priority issues

    closes #12, closes #80, closes #95, closes #96, closes #99

    - Issue #12: Set tp_new to NULL for GridPoint and GridPointState to prevent instantiation from Python
    - Issue #80: Renamed Caption.size to Caption.font_size for semantic clarity
    - Issue #95: Fixed UICollection repr to show actual derived types instead of generic UIDrawable
    - Issue #96: Added extend() method to UICollection for API consistency with UIEntityCollection
    - Issue #99: Exposed read-only properties for Texture (sprite_width, sprite_height, sheet_width, sheet_height, sprite_count, source) and Font (family, source)

    All issues have corresponding tests that verify the fixes work correctly.

commit e5affaf317
Author: John McCardle <mccardle.john@gmail.com>
Date:   Sat Jul 5 15:50:09 2025 -0400

    Fix critical issues: script loading, entity types, and color properties

    - Issue #37: Fix Windows scripts subdirectory not checked
      - Updated executeScript() to use executable_path() from platform.h
      - Scripts now load correctly when working directory differs from executable

    - Issue #76: Fix UIEntityCollection returns wrong type
      - Updated UIEntityCollectionIter::next() to check for stored Python object
      - Derived Entity classes now preserve their type when retrieved from collections

    - Issue #9: Recreate RenderTexture when resized (already fixed)
      - Confirmed RenderTexture recreation already implemented in set_size() and set_float_member()
      - Uses 1.5x padding and 4096 max size limit

    - Issue #79: Fix Color r, g, b, a properties return None
      - Implemented get_member() and set_member() in PyColor.cpp
      - Color component properties now work correctly with proper validation

    - Additional fix: Grid.at() method signature
      - Changed from METH_O to METH_VARARGS to accept two arguments

    All fixes include comprehensive tests to verify functionality.

    closes #37, closes #76, closes #9, closes #79
2025-07-05 18:56:02 -04:00

136 lines
No EOL
4.7 KiB
Python

#!/usr/bin/env python3
"""
Test for Issue #12: Forbid GridPoint/GridPointState instantiation
This test verifies that GridPoint and GridPointState cannot be instantiated
directly from Python, as they should only be created internally by the C++ code.
"""
import mcrfpy
import sys
def test_gridpoint_instantiation():
"""Test that GridPoint and GridPointState cannot be instantiated"""
print("=== Testing GridPoint/GridPointState Instantiation Prevention (Issue #12) ===\n")
tests_passed = 0
tests_total = 0
# Test 1: Try to instantiate GridPoint
print("--- Test 1: GridPoint instantiation ---")
tests_total += 1
try:
point = mcrfpy.GridPoint()
print("✗ FAIL: GridPoint() should not be allowed")
except TypeError as e:
print(f"✓ PASS: GridPoint instantiation correctly prevented: {e}")
tests_passed += 1
except Exception as e:
print(f"✗ FAIL: Unexpected error: {e}")
# Test 2: Try to instantiate GridPointState
print("\n--- Test 2: GridPointState instantiation ---")
tests_total += 1
try:
state = mcrfpy.GridPointState()
print("✗ FAIL: GridPointState() should not be allowed")
except TypeError as e:
print(f"✓ PASS: GridPointState instantiation correctly prevented: {e}")
tests_passed += 1
except Exception as e:
print(f"✗ FAIL: Unexpected error: {e}")
# Test 3: Verify GridPoint can still be obtained from Grid
print("\n--- Test 3: GridPoint obtained from Grid.at() ---")
tests_total += 1
try:
grid = mcrfpy.Grid(10, 10)
point = grid.at(5, 5)
print(f"✓ PASS: GridPoint obtained from Grid.at(): {point}")
print(f" Type: {type(point).__name__}")
tests_passed += 1
except Exception as e:
print(f"✗ FAIL: Could not get GridPoint from Grid: {e}")
# Test 4: Verify GridPointState can still be obtained from GridPoint
print("\n--- Test 4: GridPointState obtained from GridPoint ---")
tests_total += 1
try:
# GridPointState is accessed through GridPoint's click handler
# Let's check if we can access point properties that would use GridPointState
if hasattr(point, 'walkable'):
print(f"✓ PASS: GridPoint has expected properties")
print(f" walkable: {point.walkable}")
print(f" transparent: {point.transparent}")
tests_passed += 1
else:
print("✗ FAIL: GridPoint missing expected properties")
except Exception as e:
print(f"✗ FAIL: Error accessing GridPoint properties: {e}")
# Test 5: Try to call the types directly (alternative syntax)
print("\n--- Test 5: Alternative instantiation attempts ---")
tests_total += 1
all_prevented = True
# Try various ways to instantiate
attempts = [
("mcrfpy.GridPoint.__new__(mcrfpy.GridPoint)",
lambda: mcrfpy.GridPoint.__new__(mcrfpy.GridPoint)),
("type(point)()",
lambda: type(point)() if 'point' in locals() else None),
]
for desc, func in attempts:
try:
if func:
result = func()
print(f"✗ FAIL: {desc} should not be allowed")
all_prevented = False
except (TypeError, AttributeError) as e:
print(f" ✓ Correctly prevented: {desc}")
except Exception as e:
print(f" ? Unexpected error for {desc}: {e}")
if all_prevented:
print("✓ PASS: All alternative instantiation attempts prevented")
tests_passed += 1
else:
print("✗ FAIL: Some instantiation attempts succeeded")
# Summary
print(f"\n=== SUMMARY ===")
print(f"Tests passed: {tests_passed}/{tests_total}")
if tests_passed == tests_total:
print("\nIssue #12 FIXED: GridPoint/GridPointState instantiation properly forbidden!")
else:
print("\nIssue #12: Some tests failed")
return tests_passed == tests_total
def run_test(runtime):
"""Timer callback to run the test"""
try:
# First verify the types exist
print("Checking that GridPoint and GridPointState types exist...")
print(f"GridPoint type: {mcrfpy.GridPoint}")
print(f"GridPointState type: {mcrfpy.GridPointState}")
print()
success = test_gridpoint_instantiation()
print("\nOverall result: " + ("PASS" if success else "FAIL"))
except Exception as e:
print(f"\nTest error: {e}")
import traceback
traceback.print_exc()
print("\nOverall result: FAIL")
sys.exit(0)
# Set up the test scene
mcrfpy.createScene("test")
mcrfpy.setScene("test")
# Schedule test to run after game loop starts
mcrfpy.setTimer("test", run_test, 100)