Skip to content

Add comprehensive unit tests and documentation for workflows service#154

Merged
privateip merged 1 commit into
itential:develfrom
privateip:workflow-tests
Sep 9, 2025
Merged

Add comprehensive unit tests and documentation for workflows service#154
privateip merged 1 commit into
itential:develfrom
privateip:workflow-tests

Conversation

@privateip

Copy link
Copy Markdown
Contributor
  • Add docstrings for WorkflowService struct and 6 methods (NewWorkflowService, GetById, ExportById, Clear, Update)
  • Add 15 new test cases covering previously untested methods and error scenarios
  • Improve test coverage for GetById, ExportById, Clear, Update, and NewWorkflow functions
  • Add edge case tests for multiple workflows with same name and not found scenarios
  • All 24 workflow tests now pass with comprehensive success and error case coverage

- Add docstrings for WorkflowService struct and 6 methods (NewWorkflowService, GetById, ExportById, Clear, Update)
- Add 15 new test cases covering previously untested methods and error scenarios
- Improve test coverage for GetById, ExportById, Clear, Update, and NewWorkflow functions
- Add edge case tests for multiple workflows with same name and not found scenarios
- All 24 workflow tests now pass with comprehensive success and error case coverage
@privateip privateip merged commit 2bf89bc into itential:devel Sep 9, 2025
1 check passed
@privateip privateip deleted the workflow-tests branch September 9, 2025 16:45
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.

1 participant