test: add new e2e tests - #1065
Open
vasylk-crowdin wants to merge 68 commits into
Open
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1065 +/- ##
==========================================
- Coverage 92.21% 92.19% -0.01%
==========================================
Files 142 142
Lines 11500 11574 +74
==========================================
+ Hits 10604 10670 +66
- Misses 896 904 +8 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
bhdnb
force-pushed
the
next_tests
branch
2 times, most recently
from
July 30, 2026 11:52
0372fca to
1633cad
Compare
vasylk-crowdin
marked this pull request as ready for review
September 1, 2026 07:32
bhdnb
force-pushed
the
next_tests
branch
3 times, most recently
from
September 4, 2026 14:14
e0e0d7e to
c6a9d86
Compare
…ase path test snapshot
b522ff7 reworked groupKey() so that content lines share one sortable group (PLAIN) instead of each keying on itself. Bare `--output plain` path listings therefore sort alphabetically now instead of keeping the download-completion order, but two snapshots were not regenerated: - download sources ... with --output plain - download translations --all ... with plain output Both diffs are a pure re-sort: same lines, same count, new order. The same commit also landed multilingual-csv.test.ts.snap empty, dropping all 13 of its snapshots; restore them, including the `Directory <name> created` -> `Directory '<name>'` message change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
… path
The suite was written 2026-07-24 and never revisited, so three assertions
outlived the behavior they described:
- `requires --engine-id` and `warns when --auto-approve-option ...` both
read result.stdout, but CliError and output.warning are diagnostics and
d1dac57 (2026-08-20) routed those to stderr in every output format so
stdout carries only the result document.
- `uploads sources` expected `File 'sources/1_android.xml'`, the local
path. Success lines report the project path, and the fixture sets
`preserve_hierarchy: false`, which strips the shared `sources/` parent --
exactly as the comment on the following test already spelled out.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…e one The `docxMatches.length === 1` check could not pass under any CLI behavior: the same test also asserts stdout contains `File 'Folder/sample.docx'` and that `/sources/sample.docx` still exists and still carries the sampleV2 SRX segmentation. Those two imply at least two `sample.docx` files, so the count assertion contradicted them rather than marking a regression. Its stated justification was Java parity, but v4's ProjectFilesUtils.fileLookup is logically identical to fileLookup here, so a dest change duplicates on v4 too - the duplication is a pre-existing product wart, not a port regression, and cleanup is `--delete-obsolete`. Assert both project paths instead, which is what actually happens. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.