Skip to content

Commit b314a1f

Browse files
committed
tests: use public html_path in deliverable tests
Replace direct calls to private `_base_format_path` with the public `html_path` property in `test_deliverable.py`. Rationale: assert behavior via public API to avoid coupling tests to internal implementation details.
1 parent 6f4715b commit b314a1f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/models/test_deliverable.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ def test_deliverable_base_format_path(deliverable_de: etree._Element):
263263
# docsetid="15-SP6",
264264
# lang="de-de"
265265
)
266-
assert deli._base_format_path('html') == '/de-de/sles/15-SP6/html/foo/'
266+
assert deli.html_path == '/de-de/sles/15-SP6/html/foo/'
267267

268268

269269
def test_deliverable_base_format_path_rootid_none(deliverable_de: etree._Element):
@@ -275,7 +275,7 @@ def test_deliverable_base_format_path_rootid_none(deliverable_de: etree._Element
275275
# lang="de-de"
276276
)
277277
# If rootid is None, fallback to the deliverable's basefile
278-
assert deli._base_format_path('html') == (
278+
assert deli.html_path == (
279279
'/de-de/sles/15-SP6/html/SLES-administration/'
280280
)
281281

0 commit comments

Comments
 (0)