Skip to content

Conversation

@ihyeeun
Copy link
Contributor

@ihyeeun ihyeeun commented Oct 26, 2025

issue #97

수정 전 수정 후
image image
(54/63)로 테스트 케이스는 통과하지만
UI는 실패로 떠있어서 어떤 케이스가 틀린지 알 수 없었음.
라이브러리 결과와 PDF 점자가 다른 케이스만 표시되었음.

기존 테스트 케이스가 일부 통과(54/63)하더라도, UI 상에서는 실패로 표시되는 문제가 있었습니다.
확인 결과, 일부 테스트 케이스 데이터가 실제 규정 PDF와 일치하지 않았습니다.

  • 띄어쓰기가 누락된 케이스 존재
  • 테스트 케이스 내용이 PDF 규정 원문과 다른 케이스 존재

위 문제상황을 규정 PDF와 기존 테스트 케이스를 일일이 대조하여, PDF 점자에 맞게 테스트 케이스 데이터를 수정했습니다.
이제 라이브러리 로직의 문제로 인해 실패하는 케이스만 명확하게 식별할 수 있어 수정 및 유지보수가 용이해질 것으로 기대합니다.

resolved #97

@changeset-bot
Copy link

changeset-bot bot commented Oct 26, 2025

⚠️ No Changeset found

Latest commit: 9a8d22c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@owjs3901 owjs3901 merged commit eefa143 into dev-five-git:main Oct 27, 2025
2 checks passed
@ihyeeun ihyeeun deleted the rule_49 branch October 27, 2025 06:57
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.

[fix] testcase 49항 예제 점자 출력 수정

2 participants