From f063d0af0c4a52e819731e7b63f0e768444faf24 Mon Sep 17 00:00:00 2001 From: Frick Date: Wed, 14 Jan 2026 02:02:08 +0000 Subject: [PATCH] Fix alignment_test.py margin default expectations MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - margin returns 0 when unset (effective default) - horiz_margin/vert_margin return -1 (sentinel for unset) 🤖 Generated with Claude Code (https://claude.ai/code) Co-Authored-By: Claude --- tests/unit/alignment_test.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/unit/alignment_test.py b/tests/unit/alignment_test.py index 35e0f00..8a3d2d6 100644 --- a/tests/unit/alignment_test.py +++ b/tests/unit/alignment_test.py @@ -46,10 +46,12 @@ print("Test 3: Checking margin properties...") try: frame = mcrfpy.Frame(pos=(0, 0), size=(100, 100)) - # Check default margins are 0 - assert frame.margin == 0, f"Expected margin=0, got {frame.margin}" - assert frame.horiz_margin == 0, f"Expected horiz_margin=0, got {frame.horiz_margin}" - assert frame.vert_margin == 0, f"Expected vert_margin=0, got {frame.vert_margin}" + # Check default margins: + # - margin returns 0 when both horiz/vert are unset (effective default) + # - horiz_margin/vert_margin return -1 (sentinel for "not set") + assert frame.margin == 0.0, f"Expected margin=0 (effective default), got {frame.margin}" + assert frame.horiz_margin == -1.0, f"Expected horiz_margin=-1 (unset), got {frame.horiz_margin}" + assert frame.vert_margin == -1.0, f"Expected vert_margin=-1 (unset), got {frame.vert_margin}" # Set margins when no alignment frame.margin = 10.0