de1dcc586a
- node.go: node 핵심 로직 수정 - node_test.go: unit 테스트 업데이트 - parser_test.go: 파서 테스트 업데이트 - session_test.go: 세션 테스트 업데이트 - agent-task 로그 파일 추가
122 lines
5.5 KiB
Text
122 lines
5.5 KiB
Text
<!-- task=edge_node_unit_tests plan=0 tag=TEST -->
|
|
|
|
# Code Review Reference - TEST
|
|
|
|
## 개요
|
|
|
|
date=2026-05-02
|
|
task=edge_node_unit_tests, plan=0, tag=TEST
|
|
|
|
## 이 파일을 읽는 리뷰 에이전트에게
|
|
|
|
각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
|
|
리뷰 완료 후 반드시 아래 순서로 아카이브하세요.
|
|
|
|
1. `CODE_REVIEW.md` → `code_review_0.log` (N = 기존 code_review_*.log 수)
|
|
2. `PLAN.md` → `plan_0.log` (M = 기존 plan_*.log 수)
|
|
3. PASS인 경우 `complete.log` 작성 후 종료. WARN/FAIL인 경우 새 `PLAN.md` + `CODE_REVIEW.md` 스텁 작성.
|
|
|
|
---
|
|
|
|
## 구현 항목별 완료 여부
|
|
|
|
| 항목 | 완료 여부 |
|
|
|------|---------|
|
|
| [TEST-1] apps/node/internal/transport/parser_test.go | [x] |
|
|
| [TEST-2] apps/node/internal/transport/session_test.go | [x] |
|
|
| [TEST-3] apps/node/internal/node/node_test.go | [x] |
|
|
|
|
## 계획 대비 변경 사항
|
|
|
|
- 계획대로 `apps/node/internal/transport/parser_test.go`, `apps/node/internal/transport/session_test.go`, `apps/node/internal/node/node_test.go`를 신규 작성했다.
|
|
- `parser_test.go`의 RunRequest 검증은 계획의 RunId·Adapter에 더해 Model도 함께 확인했다.
|
|
- `node_test.go`의 `fixedRouter`는 요청의 Model·Workspace·Policy·Input·TimeoutSec·Metadata를 `ExecutionSpec`으로 전달하도록 작성했다.
|
|
|
|
## 주요 설계 결정
|
|
|
|
- `parser_test.go`는 `package transport`를 사용해 unexported `nodeParserMap()`을 직접 검증했다.
|
|
- `session_test.go`는 `&transport.Session{}` 제로값을 사용해 TcpClient 없이 cancel 등록·호출·해제 흐름만 검증했다.
|
|
- `node_test.go`는 `sink.Emit`을 호출하지 않는 `noEmitAdapter`와 in-memory SQLite store를 사용해 외부 연결 없이 Node 핵심 경로를 검증했다.
|
|
|
|
## 리뷰어를 위한 체크포인트
|
|
|
|
- `parser_test.go`가 `package transport`로 선언되어 `nodeParserMap()` 접근 가능한지
|
|
- 3개 메시지 타입(RunRequest, CancelRequest, CapabilityRequest) 파서 round-trip 검증 여부
|
|
- `session_test.go`에서 `&transport.Session{}` 제로값 사용 (TcpClient nil) 확인
|
|
- `TestSession_RegisterCancel_CancelRun`: cancel 호출 여부 확인
|
|
- `TestSession_DeregisterCancel`: 해제 후 미호출 확인
|
|
- `node_test.go`의 `noEmitAdapter`가 `sink.Emit`을 호출하지 않는지 확인 (TcpClient nil 안전)
|
|
- `makeNode`에서 `:memory:` SQLite 사용 여부
|
|
- `TestOnCapabilityRequest`: NodeId·AdapterName·Models 필드 검증
|
|
- `TestOnRunRequest_RouterError`: 에러 메시지에 `"node: resolve:"` 접두사 포함
|
|
- `TestOnRunRequest_AdapterNotFound`: 에러 메시지에 `"not found after routing"` 포함
|
|
- `TestOnRunRequest_Success`: nil 반환 확인
|
|
- `TestOnCancel_CallsCancelFn`: cancel 함수 호출 확인
|
|
- `go test ./apps/node/...` 전체 pass
|
|
|
|
## 검증 결과
|
|
|
|
_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._
|
|
|
|
### TEST-1 중간 검증
|
|
```
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go test ./apps/node/internal/transport/...
|
|
ok iop/apps/node/internal/transport 0.002s
|
|
```
|
|
|
|
### TEST-2 중간 검증
|
|
```
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go test ./apps/node/internal/transport/...
|
|
ok iop/apps/node/internal/transport 0.002s
|
|
```
|
|
|
|
### TEST-3 중간 검증
|
|
```
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go test ./apps/node/internal/node/...
|
|
ok iop/apps/node/internal/node 0.004s
|
|
```
|
|
|
|
### 최종 검증
|
|
```
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go test ./apps/node/...
|
|
? iop/apps/node/cmd/iop-node [no test files]
|
|
? iop/apps/node/internal/adapters [no test files]
|
|
? iop/apps/node/internal/adapters/cli [no test files]
|
|
? iop/apps/node/internal/adapters/mock [no test files]
|
|
? iop/apps/node/internal/adapters/ollama [no test files]
|
|
? iop/apps/node/internal/adapters/vllm [no test files]
|
|
? iop/apps/node/internal/bootstrap [no test files]
|
|
ok iop/apps/node/internal/node (cached)
|
|
? iop/apps/node/internal/router [no test files]
|
|
? iop/apps/node/internal/runtime [no test files]
|
|
? iop/apps/node/internal/store [no test files]
|
|
ok iop/apps/node/internal/transport (cached)
|
|
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go test ./...
|
|
? iop/apps/control-plane/cmd/iop-control-plane [no test files]
|
|
? iop/apps/edge/cmd/iop-edge [no test files]
|
|
? iop/apps/edge/internal/bootstrap [no test files]
|
|
ok iop/apps/edge/internal/node (cached)
|
|
? iop/apps/edge/internal/transport [no test files]
|
|
? iop/apps/node/cmd/iop-node [no test files]
|
|
? iop/apps/node/internal/adapters [no test files]
|
|
? iop/apps/node/internal/adapters/cli [no test files]
|
|
? iop/apps/node/internal/adapters/mock [no test files]
|
|
? iop/apps/node/internal/adapters/ollama [no test files]
|
|
? iop/apps/node/internal/adapters/vllm [no test files]
|
|
? iop/apps/node/internal/bootstrap [no test files]
|
|
ok iop/apps/node/internal/node (cached)
|
|
? iop/apps/node/internal/router [no test files]
|
|
? iop/apps/node/internal/runtime [no test files]
|
|
? iop/apps/node/internal/store [no test files]
|
|
ok iop/apps/node/internal/transport (cached)
|
|
? iop/apps/worker/cmd/iop-worker [no test files]
|
|
? iop/packages/auth [no test files]
|
|
? iop/packages/config [no test files]
|
|
? iop/packages/jobs [no test files]
|
|
? iop/packages/metadata [no test files]
|
|
? iop/packages/observability [no test files]
|
|
? iop/packages/policy [no test files]
|
|
? iop/packages/version [no test files]
|
|
? iop/proto/gen/iop [no test files]
|
|
```
|