Skip to content

fix: handle null ErrorObject in raise_callable_error#87

Merged
1 commit merged intomainfrom
null-error-object
Oct 22, 2025
Merged

fix: handle null ErrorObject in raise_callable_error#87
1 commit merged intomainfrom
null-error-object

Conversation

@rarepolz
Copy link
Contributor

@rarepolz rarepolz commented Oct 22, 2025

Update raise_callable_error method to accept optional message parameter and raise CallableRuntimeError when error is None.

Issue #, if available:
closes #35
Description of changes:

  • Updated raise_callable_error() method
    • Now accepts optional msg parameter and raises CallableRuntimeError instead of DurableExecutionsError when ErrorObject is None
  • Added null-safe error handling
    • Method provides default "Unknown error" message or uses custom message when no ErrorObject exists
  • Updated tests
    • Modified existing test and added new test case for custom message parameter functionality

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Update raise_callable_error method to accept optional message
parameter and raise CallableRuntimeError when error is None.
@rarepolz rarepolz marked this pull request as ready for review October 22, 2025 18:30
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ghost ghost merged commit 91ff1e3 into main Oct 22, 2025
12 of 17 checks passed
@wangyb-A wangyb-A deleted the null-error-object branch December 9, 2025 22:29
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ErrorObject Can Be Null on Operation

1 participant