Skip to content

Comments

test(chapterview): split critical flows + add inline edit#20

Merged
anantham merged 1 commit intomainfrom
test/chapterview-inline-edit
Dec 27, 2025
Merged

test(chapterview): split critical flows + add inline edit#20
anantham merged 1 commit intomainfrom
test/chapterview-inline-edit

Conversation

@anantham
Copy link
Owner

Summary

Split ChapterView critical-flow tests to keep jsdom coverage focused and add a dedicated inline-edit flow test.

Changes

  • tests/integration/ChapterView.critical-flows.test.tsx: keep diff-marker flow only.
  • tests/integration/ChapterView.inline-edit.test.tsx: add inline edit + persistence coverage.

Testing

collaborative-novel-translator@0.0.0 test
vitest tests/integration/ChapterView.critical-flows.test.tsx tests/integration/ChapterView.inline-edit.test.tsx --run

�[1m�[46m RUN �[49m�[22m �[36mv4.0.16 �[39m�[90m/Users/aditya/Documents/Ongoing Local/LexiconForge�[39m

�[32m✓�[39m tests/integration/ChapterView.inline-edit.test.tsx �[2m(�[22m�[2m1 test�[22m�[2m)�[22m�[33m 363�[2mms�[22m�[39m
�[33m�[2m✓�[22m�[39m edits a selected chunk and persists via TranslationPersistenceService �[33m 362�[2mms�[22m�[39m
�[32m✓�[39m tests/integration/ChapterView.critical-flows.test.tsx �[2m(�[22m�[2m2 tests�[22m�[2m)�[22m�[32m 63�[2mms�[22m�[39m

�[2m Test Files �[22m �[1m�[32m2 passed�[39m�[22m�[90m (2)�[39m
�[2m Tests �[22m �[1m�[32m3 passed�[39m�[22m�[90m (3)�[39m
�[2m Start at �[22m 01:17:23
�[2m Duration �[22m 989ms�[2m (transform 523ms, setup 147ms, import 555ms, tests 427ms, environment 662ms)�[22m

@vercel
Copy link

vercel bot commented Dec 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
lexicon-forge Ready Ready Preview, Comment Dec 26, 2025 7:47pm

@anantham anantham merged commit 6f3e0f7 into main Dec 27, 2025
3 checks passed
@anantham anantham deleted the test/chapterview-inline-edit branch December 27, 2025 16:26
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