Skip to content

test(core): add integration tests for viewer conversion functions#33

Closed
ohah wants to merge 0 commit intomainfrom
test/viewer-conversion-integration-tests
Closed

test(core): add integration tests for viewer conversion functions#33
ohah wants to merge 0 commit intomainfrom
test/viewer-conversion-integration-tests

Conversation

@ohah
Copy link
Copy Markdown
Owner

@ohah ohah commented Feb 25, 2026

테스트 추가/보강. viewer 모듈의 다양한 변환 함수(to_markdown, to_html, to_pdf)에 대한 통합 테스트를 추가함.

  • toJson() → fileHeaderJson() (파서 타겟이어서 기존에 존재하여 제외)

  • to_markdown() → Markdown 변환 검증

    • 기본 변환 테스트
    • 이미지 출력 디렉토리 옵션 테스트
    • 사용자 정의 옵션 테스트
    • 빈 문서 처리 테스트
  • to_html() → HTML 변환 검증

    • 기본 변환 테스트
    • 사용자 정의 옵션 테스트
    • 빈 문서 처리 테스트
  • to_pdf() → PDF 변환 테스트 (금지됨. 글꼴 필수라 자동으로 skip됨)

  • resolve_display_texts() → 표시 텍스트 해결 메서드 검증

  • 기능: 실제 HWP 파일(hwpjs/fixtures/noori.hwp) 또는 최소 유효 FileHeader 사용

  • 7개 테스트 통과 (1개는 금지됨/사실상 스킵됨)

  • 조건: 실제 HWP 파일이 fixtures에 존재해야 함 (테스트가 해당 경로를 찾으면 실행)

리뷰 후 머지.

ohah added a commit that referenced this pull request Feb 25, 2026
Tests ParagraphPosition, ParagraphRenderContext, ParagraphRenderState,
render_paragraphs_fragment, and helper functions (collect_*).
Covers struct field access and empty edge cases.
@ohah ohah closed this Feb 25, 2026
@ohah ohah force-pushed the test/viewer-conversion-integration-tests branch from 6af5244 to 849b8f0 Compare February 25, 2026 04:46
ohah added a commit that referenced this pull request Feb 25, 2026
Tests ParagraphPosition, ParagraphRenderContext, ParagraphRenderState,
render_paragraphs_fragment, and helper functions (collect_*).
Covers struct field access and empty edge cases.
ohah added a commit that referenced this pull request Feb 25, 2026
Tests ParagraphPosition, ParagraphRenderContext, ParagraphRenderState,
render_paragraphs_fragment, and helper functions (collect_*).
Covers struct field access and empty edge cases.
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