Skip to content

test: preserve linebreak text on HWPX roundtrip#210

Closed
yl-star7 wants to merge 2 commits into
edwardkim:mainfrom
yl-star7:test/hwpx-linebreak-roundtrip
Closed

test: preserve linebreak text on HWPX roundtrip#210
yl-star7 wants to merge 2 commits into
edwardkim:mainfrom
yl-star7:test/hwpx-linebreak-roundtrip

Conversation

@yl-star7

Copy link
Copy Markdown

Summary

  • add a roundtrip regression test for paragraphs containing line breaks
  • verify paragraph text and char_offsets are preserved after serialize and reparse

Test plan

  • cargo test --manifest-path "/Users/jskang/Projects/rhwp/Cargo.toml" --test hwpx_roundtrip_integration task177_linebreak_preserved_on_roundtrip_ref_mixed -- --nocapture

강정석 and others added 2 commits April 20, 2026 11:41
Add a roundtrip regression test that verifies paragraphs containing line breaks keep their text and char offsets after serialize and reparse.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Keep HWPX paragraph offsets and char shape positions aligned with the original text/control order so linebreak and embedded control roundtrips remain stable.

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

Copy link
Copy Markdown
Owner

라운드트립 테스트는 AI 가 작성해준 테스트 케이스를 절대 믿으시면 않됩니다. 반드시 꼼꼼히 정말 라운드 트립인지 아니면 0=0 논리인지 확인 사살해주셔야 합니다.

@edwardkim

Copy link
Copy Markdown
Owner

PR #213 과 동일 브랜치의 main 타깃 중복 PR. #213 은 devel 반영 완료 처리되어 본 PR 도 close.

@edwardkim edwardkim closed this Apr 20, 2026
edwardkim added a commit that referenced this pull request Apr 20, 2026
2026-04-20~21 사이클 정리:

PR 처리 9건:
- admin merge 5: #209 #214 #215 #221 #224
- cherry-pick + close 2: #213 (+중복 #210 close), #181 (+golden 재생성)
- dependabot close 2: #211 #212 (devel 수동 bump + target-branch=devel 설정)
- 보류 1: #165 skia (별도 사이클)

이슈 close 7: #173 #195 #202 #205 #207 #210 #222
신규 이슈 등록 1: #204 (표 Undo/Redo)

Chrome Web Store / Edge Add-ons v0.2.1 심사 통과 (2026-04-21).

local/task205 폐기: PR #209+#214 가 중복 처리하여 잔여 고유 기여 분리 곤란.

기여자 7명 감사 기록.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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