Skip to content

Commit 4fd3687

Browse files
fix(cli): Fix help text assertion test by using 'validation errors' instead of 'NOT fixable'
Co-Authored-By: AJ Steers <[email protected]>
1 parent 41b84b9 commit 4fd3687

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

airbyte_cdk/cli/airbyte_cdk/_manifest.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,10 +55,14 @@ def validate_manifest(manifest_path: Path, strict: bool) -> None:
5555
If validation fails, it will suggest running the migrate command if needed.
5656
5757
Exit codes:
58+
5859
\\b
5960
0: Manifest is valid and up to date
61+
\\b
6062
1: Manifest has issues that are fixable via migration
63+
\\b
6164
2: Manifest has validation errors that are NOT fixable via migration
65+
\\b
6266
3: General errors (file not found, invalid YAML, etc.)
6367
"""
6468
try:

unit_tests/cli/test_manifest_cli.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ def test_validate_command_help(self) -> None:
395395
assert "codes:" in result.output
396396
assert "0:" in result.output and "valid and up to date" in result.output
397397
assert "1:" in result.output and "fixable via migration" in result.output
398-
assert "2:" in result.output and "NOT fixable via migration" in result.output
398+
assert "2:" in result.output and "validation errors" in result.output
399399
assert "3:" in result.output and "General errors" in result.output
400400

401401
def test_migrate_command_help(self) -> None:

0 commit comments

Comments
 (0)