Skip to content

test: drop the type-ignore comments from two test files - #803

Merged
Eomdahyeon merged 1 commit into
mainfrom
test/drop-type-ignores
Oct 4, 2026
Merged

Eomdahyeon merged 1 commit into
mainfrom
test/drop-type-ignores

Conversation

@Eomdahyeon

Copy link
Copy Markdown
Collaborator

문제

#801 리뷰가 짚은 것과 같은 위반이 오늘 내가 이미 머지한 테스트에 세 곳 더 있다. AGENTS.md 는 type: ignore 를 금지한다 ("Fix the type error instead of silencing it").

변경 내용

  • 문자열 대신 Representation.API_JSON 을 넘긴다.
  • _LazyTransport 를 FakeTransport 의 서브클래스로 만들어 _make_executor 가 받는 타입이 되게 했다.

테스트 동작은 그대로다: 두 파일 13 passed. ruff 통과.

범위

오늘 내가 추가한 파일만 고쳤다. main 의 tests/ 에는 type: ignore 가 그 밖에도 53곳 남아 있다(git grep -c 합계 56 중 이 3곳 제외) — 그것들은 건드리지 않았다.

🤖 Generated with Claude Code

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

@yeongseon yeongseon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰: diff 를 읽었고 걸리는 점이 없습니다. Representation.API_JSON 을 넘기고 _LazyTransport 를 FakeTransport 의 서브클래스로 만들어 ignore 없이 타입이 맞습니다. 테스트 동작은 바뀌지 않았고 CLEAN 입니다. 승인합니다.

남은 53곳을 이 PR 범위 밖으로 둔 것도 맞습니다 — 한 번에 치우려면 별도 이슈와 게이트(ratchet)가 먼저입니다.

@Eomdahyeon
Eomdahyeon merged commit cf3082e into main Oct 4, 2026
23 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.

2 participants