Skip to content

Task #704: 통합재정통계 trailing 빈 paragraph 0.84 px borderline — tests/issue_703.rs #[ignore] cleanup (closes #704)#870

Closed
planet6897 wants to merge 2 commits into
edwardkim:develfrom
planet6897:task704
Closed

Task #704: 통합재정통계 trailing 빈 paragraph 0.84 px borderline — tests/issue_703.rs #[ignore] cleanup (closes #704)#870
planet6897 wants to merge 2 commits into
edwardkim:develfrom
planet6897:task704

Conversation

@planet6897
Copy link
Copy Markdown
Contributor

Summary

Issue #704 (통합재정통계 페이지 분할 — TopAndBottom TAC 1×1 wrapper + 각주 환경 0.84 px borderline) 의 본질적 결함은 Task #676 (PR #679, 커밋 52ae6558) 의 trailing empty paragraph 가드 (typeset.rs:1182-1199) 로 이미 정정 완료. 본 PR 은 잔존 tests/issue_703.rs#[ignore = \"Issue #704 ...\"] 2 건 cleanup 만 수행 (소스 코드 무변경).

처리 경위

시점 사건
Task #703 분석 단계 통합재정통계 케이스를 BehindText/InFrontOfText 와 다른 본질로 판단 → #704 분리 등록 + tests/issue_703.rs 두 테스트 #[ignore]
Task #676 (PR #679 merged bd3b63dd) typeset.rs:1182-1199 에 trailing empty paragraph 가드 — is_last_in_section && col_count==1 && empty_para && total_h ≤ available + LAYOUT_DRIFT_SAFETY_PX → height=0 흡수. 이슈 #704 본문 "옵션 B (epsilon 보강)" 와 동일 시멘틱
본 PR (Task #704) tests/issue_703.rs#[ignore] 2 건 강제 실행 GREEN 검증 → cleanup

변경 파일

소스 코드 무변경 — 테스트 cleanup 만 수행.

Test plan

  • cargo test --release --test issue_703 — 3/3 ok (통합재정통계 2 건 정식 GREEN + calendar_year 보존)
  • cargo test --release --test issue_676_trailing_empty_para — 3/3 ok (회귀 0)
  • cargo test --release 전체 — 1231 passed / 1 failed / 2 ignored
    • 1 failure = wasm_api::tests::test_empty_save_analysis 기존 flaky (단독 실행 시 PASS, devel HEAD 단독 PASS 동일 — 본 PR 무관)
  • cargo clippy --all-targets -- -D warnings — 54 errors (devel HEAD src/wasm_api/tests.rs 기존 unused_must_use 위반 — 본 PR 무관, devel HEAD 단독 검증으로 확인 / 후속 별도 이슈 등록 권장)

후속 권장

devel HEAD 의 lib test 54 clippy unused_must_use 위반 정리 — 별도 이슈 등록 권장.

관련

본질 정정은 Task edwardkim#676 (PR edwardkim#679, 커밋 52ae655) 의 trailing empty
paragraph 가드 (typeset.rs:1182-1199) 로 이미 진입.
이슈 edwardkim#704 분리 시점에 #[ignore] 처리된 두 테스트가 현재 GREEN 임을
Stage 1 에서 검증 (`cargo test --test issue_703 -- --ignored`).
Stage 2 에서 #[ignore] 2 건 제거 + 주석을 Task edwardkim#676 reference 로 갱신.

- mydocs/plans/task_m100_704.md (수행계획서)
- mydocs/working/task_m100_704_stage1.md (현황 검증)
- mydocs/working/task_m100_704_stage2.md (cleanup)
- tests/issue_703.rs: #[ignore] 2건 제거 + 주석 갱신

Ref edwardkim#704
cargo test --release 1231 passed (1 fail = 기존 flaky `wasm_api::tests::
test_empty_save_analysis`, 단독 PASS 확인, 본 task 무관).
cargo clippy --all-targets -- -D warnings 54 errors 는 devel HEAD 의
lib test 기존 unused_must_use 위반 (devel HEAD 단독 검증). 본 task 무관 —
후속 별도 이슈 등록 권장.

타겟 통합 테스트 GREEN:
- tests/issue_676_trailing_empty_para.rs: 3/3 ok
- tests/issue_703.rs: 3/3 ok (통합재정통계 2 건 정식 GREEN)

- mydocs/report/task_m100_704_report.md (최종 보고서)
- mydocs/orders/20260513.md (Task edwardkim#704 행 추가)

Ref edwardkim#704
edwardkim pushed a commit that referenced this pull request May 14, 2026
…704)

PR #870 (@planet6897) cherry-pick. mydocs 제외.
소스 코드 무변경 — tests/issue_703.rs #[ignore] 2건 제거 + 주석 갱신만.
본질 정정은 Task #676 (PR #679, typeset.rs trailing empty paragraph 가드)에서 완료.

검증: cargo test --release --test issue_703: 3/3 passed

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@edwardkim
Copy link
Copy Markdown
Owner

검토 + cherry-pick 머지 완료. 감사합니다.

처리 결과

재검증

  • cargo test --release --test issue_703: 3/3 passed (통합재정통계 2건 정식 GREEN + calendar_year 보존)

소스 코드 무변경, Task #676 정정의 정당성을 테스트로 확인한 깔끔한 cleanup입니다. 수고하셨습니다.

@edwardkim edwardkim closed this May 14, 2026
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