nexo312/trading-system-docs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tolaria): C5A reason-code closed-set accepts C5D build code

Browse source 
RC_SEARCH_SOURCE_BUILD_FAILURE (added in 5e41915, C5D search-source
pipeline) was missing from test_reason_codes_closed_set, so the C5A
suite falsely FAILed (18 vs 17). Pre-existing defect exposed by the
C5D adoption fresh-checker regression gate; not introduced by 747376e.
Harden the assertion to the real closed set (len 18).
This commit is contained in:
Red Queen 2026-08-26 15:24:03 +00:00
parent 747376ebb0
commit 76512ddd48
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
tolaria/c5-sync-service/test_c5a.py
View file

@ -41,6 +41,7 @@ from rq_c5a import (
RC_UNKNOWN_LEGACY_OBJECT, RC_AUTH_FAILURE, RC_SEARCH_REBUILD_FAILURE,
RC_TOLARIA_UNAVAILABLE, RC_FORGEJO_UNAVAILABLE,
RC_SEARCH_UNAVAILABLE, RC_NETWORK_TIMEOUT, RC_MALFORMED_RESPONSE, RC_INTEGRITY_FAILURE,
RC_SEARCH_SOURCE_BUILD_FAILURE,
REASON_CODES, OPERATIONS, SYNC_STATES, BOOTSTRAP_STATES,
)
@ -330,7 +331,9 @@ def test_reason_codes_closed_set():
assert RC_NETWORK_TIMEOUT in REASON_CODES
assert RC_MALFORMED_RESPONSE in REASON_CODES
assert RC_INTEGRITY_FAILURE in REASON_CODES
assert len(REASON_CODES) == 17
# C5D-Erweiterung (search-source pipeline, commit 5e41915)
assert RC_SEARCH_SOURCE_BUILD_FAILURE in REASON_CODES
assert len(REASON_CODES) == 18
store = _new_store()
try:
store.set_commit_error("c1", "FREIER_STRING", "x")