Skip to content

test: add vi fixtures#51

Open
naoNao89 wants to merge 1 commit into
caret-collective:mainfrom
naoNao89:feat/vietnamese-fixtures
Open

test: add vi fixtures#51
naoNao89 wants to merge 1 commit into
caret-collective:mainfrom
naoNao89:feat/vietnamese-fixtures

Conversation

@naoNao89

@naoNao89 naoNao89 commented Jun 10, 2026

Copy link
Copy Markdown

add Vietnamese fixtures for segmentation tests
closes #11

covers:

  • Vietnamese letters with tone marks
  • đ ă â ê ô ơ ư
  • punctuation
  • sentences
  • paragraphs

@twocaretcat twocaretcat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! The tests are mostly correct, but the inputs don't match what is used for other languages. Sorry if that requirement was unclear. If you're able to replace the input text and regenerate the fixture outputs that would be great.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This input uses different source text from the other languages. Each language must translate the same input so the tests remain comparable. Please use “Xin chúc mừng!”, matching the English fixture. Copy its source/license comments, then regenerate the fixture outputs.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This input uses different source text from the other languages. Each language must translate the same input so the tests remain comparable. Please use “Tôi nhớ bạn”, matching the English fixture. Copy its source/license comments, then regenerate the fixture outputs.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This input uses different source text and contains two sentences, while this case should contain one sentence with punctuation. Each language must translate the same input so the tests remain comparable. Please use the corresponding Vietnamese translation, matching the English fixture. Copy its source/license comments, then regenerate the fixture outputs.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This input uses different source text from the other languages. Each language must translate the same input so the tests remain comparable. There's no existing translation for this one, so please translate the text in the English fixture into Vietnamese yourself. Copy its source/license comments, then regenerate the fixture outputs.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This input uses different source text from the other languages. Each language must translate the same input so the tests remain comparable. Please use Articles 3 and 4 from the official Vietnamese UDHR translation, matching the English fixture. Keep the blank line between the articles, copy the English file’s source comment, then regenerate the fixture outputs.

@twocaretcat twocaretcat changed the title add vi fixtures test: add vi fixtures Jun 15, 2026
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.

Add unit tests for Vietnamese

2 participants