Skip to content

fix(core): COLORREF JSON {r,g,b} 복원 및 테스트 컴파일/기대값 수정#78

Merged
ohah merged 2 commits intomainfrom
fix/hwp-core-tests-and-colorref-json
Mar 1, 2026
Merged

fix(core): COLORREF JSON {r,g,b} 복원 및 테스트 컴파일/기대값 수정#78
ohah merged 2 commits intomainfrom
fix/hwp-core-tests-and-colorref-json

Conversation

@ohah
Copy link
Copy Markdown
Owner

@ohah ohah commented Mar 1, 2026

변경 요약

  • COLORREF 직렬화: JSON에서 {"r", "g", "b"} 객체 형태로 출력/입력 (이전 스냅샷 호환)
  • 테스트 컴파일: markdown ctrl_header convert_* pub 노출 및 테스트 import 수정, ObjectAttribute 등 Default 추가, shape_object_test String 타입 수정
  • bodytext_test: FileHeader 256바이트 요구에 맞게 minimal_file_header_bytes() 사용
  • outline_test: format_outline_number / number_to_circled 실제 동작에 맞게 기대값 수정
  • html 뷰어: round_to_2dp 및 SHWPUNIT 타입 정리 (position, page, document)
  • docs: CONTRIBUTING - Colima/Apple Silicon Linux 크로스 빌드 설명 보강 (Intel·Apple Silicon 모두 Colima 사용 가능 명시, Apple Silicon에서 cross 시 rustup target/툴체인 등록 단계 추가)

테스트

  • cargo test -p hwp-core 통과
  • 스냅샷(insta)은 COLORREF 변경으로 JSON 스냅이 {"r","g","b"} 형식으로 유지됨

- COLORREF: custom Serialize/Deserialize for {"r","g","b"} (snapshot compatible)
- ObjectAttribute + related enums: add Default for test fixtures
- bodytext_test: use 256-byte FileHeader (minimal_file_header_bytes)
- outline_test: align expectations with format_outline_number/number_to_circled
- markdown ctrl_header: pub convert_* and test imports (super::super::)
- shape_object_test: use String for expected values
- html viewer: round_to_2dp and SHWPUNIT type fixes (position, page, document)

Made-with: Cursor
@ohah ohah self-assigned this Mar 1, 2026
@ohah ohah added bug Something isn't working enhancement New feature or request labels Mar 1, 2026
- macOS(Intel·Apple Silicon) 모두 Colima 사용 가능 명시
- Apple Silicon에서 cross 사용 시 rustup target/툴체인 등록 단계 추가

Made-with: Cursor
@ohah ohah merged commit 362aa84 into main Mar 1, 2026
ohah added a commit that referenced this pull request Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant