Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

複数の変換候補があるエントリから削除したときにシリアライズ候補から削除されていた #239

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

mtgto
Copy link
Owner

@mtgto mtgto commented Oct 27, 2024

#231 でユニットテストが実は落ちていたことで発覚。
複数の変換候補があるエントリから1つの候補を削除したときに、誤って読みのリストから削除してしまっていました。
そのバグは #231 で修正したのですが、FileDictTests#testSerializeにあったユニットテスト自体に複数候補をもつものから1つの変換候補を消したあとのシリアライズした結果の成功判定に問題があり、ここでもエントリ自体が消えることを期待してしまっていました。

@mtgto mtgto merged commit 4b62291 into main Oct 27, 2024
1 check passed
@mtgto mtgto deleted the fix-delete-yomis3 branch October 27, 2024 03:21
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