Skip to content

Commit

Permalink
github test: use lxml==5.2 to fix jmdict test
Browse files Browse the repository at this point in the history
  • Loading branch information
ilius committed Sep 2, 2024
1 parent 1f095ad commit eac286b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
beautifulsoup4 \
biplist html5lib \
python-idzip \
lxml \
lxml==5.2 \
marisa-trie \
mistune
- name: Remove test cache
Expand Down
2 changes: 2 additions & 0 deletions tests/g_jmdict_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ def convert_jmdict_txt(self, fname, fname2, **convertArgs):
**convertArgs,
)

# for some reason, with lxml==5.3.0, for "bword://{word}", `word` is not unicode-escaped by lxml
# while lxml < 5.3.0 does escape these unicode characters
def test_convert_jmdict_txt_1(self):
self.convert_jmdict_txt(
"050-JMdict-English",
Expand Down

0 comments on commit eac286b

Please sign in to comment.