test: Edge Cases#4
Conversation
| @@ -0,0 +1,5 @@ | |||
| const regular = 'ascii'; | |||
| const japanese = '修正済み'; | |||
| const russian = 'Привет'; | |||
There was a problem hiding this comment.
[EC-04] 🎉 修正済み Привет مرحبا - Unicode/emoji rendering test
| @@ -0,0 +1,5 @@ | |||
| const regular = 'ascii'; | |||
There was a problem hiding this comment.
[EC-05] Comment at first character of file
There was a problem hiding this comment.
Pull request overview
This PR adds test files to verify edge case handling for the CodjiFlo E2E test suite. The changes introduce test data files covering various unusual but valid scenarios including Unicode characters, extremely long lines, and empty files.
- Adds Unicode character test data including Japanese, Russian, Arabic, and emoji characters
- Adds an extremely long line test file to verify line length handling
- Adds an empty file to test empty file scenarios
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/unicode-test.ts | Introduces TypeScript constants containing various Unicode test strings (ASCII, Japanese, Russian, Arabic, emoji) |
| src/long-line.txt | Creates a text file with an extremely long line (1000+ characters) for edge case testing |
| src/empty-file.txt | Adds an empty file for testing empty file handling |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const regular = 'ascii'; | ||
| const japanese = '修正済み'; | ||
| const russian = 'Привет'; | ||
| const arabic = 'مرحبا'; | ||
| const emoji = '🎉✨🚀'; |
There was a problem hiding this comment.
These constants are declared but never exported or used. To make this test data accessible, consider exporting these constants so they can be imported and used in tests. Without exports, these values cannot be referenced outside this file.
| const regular = 'ascii'; | |
| const japanese = '修正済み'; | |
| const russian = 'Привет'; | |
| const arabic = 'مرحبا'; | |
| const emoji = '🎉✨🚀'; | |
| export const regular = 'ascii'; | |
| export const japanese = '修正済み'; | |
| export const russian = 'Привет'; | |
| export const arabic = 'مرحبا'; | |
| export const emoji = '🎉✨🚀'; |
pedropaulovc
force-pushed
the
test/edge-cases
branch
from
bc5788b to
f173489
Compare
| @@ -0,0 +1,5 @@ | |||
| const regular = 'ascii'; | |||
| const japanese = '修正済み'; | |||
| const russian = 'Привет'; | |||
There was a problem hiding this comment.
[EC-04] 🎉 修正済み Привет مرحبا - Unicode/emoji rendering test
| @@ -0,0 +1,5 @@ | |||
| const regular = 'ascii'; | |||
There was a problem hiding this comment.
[EC-05] Comment at first character of file
Test PR for unusual but valid scenarios