Skip to content

Add tests for execute_complete success and error handling in AirbyteTriggerSyncOperator#64119

Open
SameerMesiah97 wants to merge 1 commit intoapache:mainfrom
SameerMesiah97:AirbyteTriggerSyncOperator-Tests
Open

Add tests for execute_complete success and error handling in AirbyteTriggerSyncOperator#64119
SameerMesiah97 wants to merge 1 commit intoapache:mainfrom
SameerMesiah97:AirbyteTriggerSyncOperator-Tests

Conversation

@SameerMesiah97
Copy link
Contributor

@SameerMesiah97 SameerMesiah97 commented Mar 23, 2026

Description

This PR adds unit tests for AirbyteTriggerSyncOperator.execute_complete. The tests cover scenarios where the operator completes successfully on non-error events and where an error event triggers an exception.

Rationale

execute_complete is a key part of the deferrable execution flow but was not previously covered by the test suite. The method contains branching logic based on trigger event status that determines whether execution succeeds or fails.

Notes

Covers behavior where execute_complete returns successfully on non-error events and raises on error events. AirflowException has been replaced with RuntimeError.
@SameerMesiah97
Copy link
Contributor Author

CI failure is unrelated to my changes. It was present in PR #64084, which only changes the spelling of a 'deferrable' in one of the log messages within the execute method of AirbyteTriggerSyncOperator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant