Skip to content

Use InvalidOperationException for workflow retry validation (#766)#770

Merged
ziagham merged 1 commit intoflowsynx:masterfrom
mohin-io:issue-766-validate-retry-invalidoperation
Nov 30, 2025
Merged

Use InvalidOperationException for workflow retry validation (#766)#770
ziagham merged 1 commit intoflowsynx:masterfrom
mohin-io:issue-766-validate-retry-invalidoperation

Conversation

@mohin-io
Copy link
Copy Markdown
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

  • replace the generic Exception thrown when retry validation fails with InvalidOperationException to surface a more specific, meaningful error type during workflow validation
  • keep validation messaging intact while aligning with the project's structured exception handling inside WorkflowValidator
  • add a workflow validator unit test covering invalid retry policy configuration and confirming the new exception type/message

Issue reference

Closes #766

Checklist

  • Code compiles correctly
  • Created/updated tests
  • Unit tests passing
  • End-to-end tests passing
  • Extended the documentation / Created issue in flowsynx/website#
  • Specification has been updated / Created issue in flowsynx/website#
  • Provided sample for the feature / Created issue in flowsynx/website#

@mohin-io mohin-io requested review from a team as code owners November 30, 2025 12:25
@sonarqubecloud
Copy link
Copy Markdown

@ziagham ziagham merged commit 8af00ae into flowsynx:master Nov 30, 2025
4 checks passed
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.

Avoid throwing System.Exception in user code in WorkflowValidator

2 participants