-
Notifications
You must be signed in to change notification settings - Fork 9
mgmt flow functions #726
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mgmt flow functions #726
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds management flow execution functionality to the Descope Python SDK, allowing flows to be programmatically executed with optional input parameters, preview mode, and tenant specification.
Key Changes:
- Added
run_flowmethod to the Flow management class for executing flows with configurable options - Introduced
FlowRunOptionsclass to encapsulate flow execution parameters (input, preview, tenant) - Added comprehensive test coverage including failure scenarios, no options, dict options, and FlowRunOptions object options
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
descope/management/flow.py |
Implements the run_flow method with support for both dict and FlowRunOptions parameter handling |
descope/management/common.py |
Defines the FlowRunOptions class and adds the flow run API path constant |
descope/__init__.py |
Exports the new FlowRunOptions class for public API usage |
tests/management/test_flow.py |
Adds comprehensive test coverage for the new run_flow functionality with multiple test scenarios |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Coverage reportThe coverage rate went from
Diff Coverage details (click to unfold)descope/management/flow.py
descope/management/common.py
|
Related Issues
Fixes https://github.com/descope/etc/issues/13459