Skip to content

Commit 51339ee

Browse files
CopilotChaffelson
andauthored
docs: Clarify test comment about ValueError from resolve_entity validation (#399)
* Initial plan * docs: Clarify test comment about ValueError from resolve_entity validation Co-authored-by: Chaffelson <11668664+Chaffelson@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Chaffelson <11668664+Chaffelson@users.noreply.github.com>
1 parent c666c4b commit 51339ee

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

tests/test_versioning_registry.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,8 @@ def test_revert_flow_ver(fix_ver_flow):
155155
r1 = versioning.revert_flow_ver(fix_ver_flow.pg)
156156
assert isinstance(r1, nifi.VersionedFlowUpdateRequestEntity)
157157
# TODO: Add Tests for flows with data loss on reversion
158-
# Test non-existent PG - now raises ValueError (not found) instead of AssertionError
158+
# Test non-existent PG - raises ValueError because resolve_entity validates
159+
# entity existence (previously raised AssertionError from isinstance check)
159160
with pytest.raises(ValueError, match="Not found"):
160161
_ = versioning.revert_flow_ver('NotAPg')
161162

0 commit comments

Comments
 (0)