Skip to content

Commit e0dda7d

Browse files
committed
fix: update unit test expectations for set_page_label normalization
The test_set_page_label_valid_config test was expecting the old behavior where open-ended page ranges would get 'end: -1' added automatically. Updated to match the new implementation where 'end' is only included if explicitly provided in the input.
1 parent 15d8cad commit e0dda7d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/unit/test_client.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,10 +159,10 @@ def test_set_page_label_valid_config():
159159

160160
result = client.set_page_label("test.pdf", labels)
161161

162-
# Expected normalized labels (implementation adds 'end': -1 when missing)
162+
# Expected normalized labels (implementation only includes 'end' if explicitly provided)
163163
expected_normalized_labels = [
164164
{"pages": {"start": 0, "end": 3}, "label": "Introduction"},
165-
{"pages": {"start": 3, "end": -1}, "label": "Content"},
165+
{"pages": {"start": 3}, "label": "Content"}, # No 'end' means to end of document
166166
]
167167

168168
# Verify the API call was made with correct parameters

0 commit comments

Comments
 (0)