Skip to content

test(byok): keep the operator-key leak test on the HTTP path under replay - #1052

Merged
yeongseon merged 1 commit into
mainfrom
fix/leak-test-under-replay
Oct 6, 2026
Merged

yeongseon merged 1 commit into
mainfrom
fix/leak-test-under-replay

Conversation

@yeongseon

Copy link
Copy Markdown
Collaborator

문제

#1051 을 머지한 뒤 main 의 Cross-repo contract 가 빨갛다 (run 37410221619, ecc75f5).

FAILED tests/unit/test_operator_key_never_leaves.py::test_the_operators_key_is_in_no_request_log_or_file
FAILED tests/unit/test_operator_key_never_leaves.py::test_the_measurement_sees_the_operators_key_when_the_switch_is_off
kpubdata.exceptions.InvalidRequestError: replay 매칭 실패: datago.apt_trade (... 'lawd_cd': '11680', 'deal_ymd': '202401')

그 잡은 KPUBDATA_REPLAY_DIR 을 준다. 그러면 kpubdata 가 HTTP 대신 기록된 fixture 를 재생하므로, 이 테스트가 바꿔 끼운 소켓(httpx.Client.send)에 닿지 않는다. 이 워크플로는 경로 필터 때문에 #1051 의 PR 에서는 돌지 않았고, 내가 승인할 때 그 점을 보지 않았다.

변경

fixture 에서 KPUBDATA_REPLAY_DIR 을 지운다 — test_credential_hosts.py, test_canary_leak_gate.py 가 같은 이유로 하는 것과 같다. 한 줄과 주석.

검증

  • ruff check, ruff format --check 통과.
  • 로컬에서 pytest 는 돌리지 못했다. 이 브랜치에서 Cross-repo contract 를 수동 실행했고 결과를 아래 코멘트에 붙인다.

🤖 Generated with Claude Code

…play

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@yeongseon

Copy link
Copy Markdown
Collaborator Author

이 브랜치(fix/leak-test-under-replay)에서 Cross-repo contract 를 수동 실행한 결과: run 37412728787 → success. 같은 워크플로가 main(ecc75f5)에서는 run 37410221619 로 실패했습니다.

@yeongseon
yeongseon merged commit 84a9d3a into main Oct 6, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant