Skip to content

Commit ea016c6

Browse files
feat: raise ValueError when no stdout source available in from_completed_process
- Replace empty EntrypointOutput creation with ValueError - Add clear error message when both stdout_file and completed_process.stdout are None - Improves error handling and prevents silent failures - Addresses GitHub PR feedback from @aaronsteers Co-Authored-By: AJ Steers <[email protected]>
1 parent 3d1dc9a commit ea016c6

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

airbyte_cdk/test/entrypoint_wrapper.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -306,7 +306,9 @@ def from_completed_process(
306306
messages = completed_process.stdout.splitlines()
307307
instance = cls(messages=messages)
308308
else:
309-
instance = cls()
309+
raise ValueError(
310+
"Either stdout_file should be provided or completed_process.stdout should not be None"
311+
)
310312

311313
return instance
312314

0 commit comments

Comments
 (0)