Uh oh!
There was an error while loading. Please reload this page.
feat: Add create project tool - #66
Conversation
- Add get-project, get-projects tool - Test code are updated and passing
- Add create-project tool - Tests are updated ans passing
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new create_project tool to the identity tools module, enabling users to create OpenStack projects with optional parent project hierarchies.
- Implements a new
create_projectmethod in theIdentityToolsclass - Registers the new tool with the FastMCP framework
- Adds comprehensive test coverage for various creation scenarios and error conditions
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/openstack_mcp_server/tools/identity_tools.py | Implements the create_project method and registers it as a tool |
| tests/tools/test_identity_tools.py | Adds comprehensive test cases covering success scenarios and various error conditions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
choieastsea
commented
Sep 2, 2025
@S0okJu sdk의 동작 자체를 테스트하거나, 발생가능한 여러 오류들을 모두 모킹하여 테스트할 필요는 없을 것 같아요. |
S0okJu
commented
Sep 2, 2025
@choieastsea 좋은 의견 감사합니다. |
Uh oh!
There was an error while loading. Please reload this page.
* feat(identity): Add get-project, get-projects tool(#58) - Add get-project, get-projects tool - Test code are updated and passing * chore(identity): ruff format(#58) * chore(identity): ruff format(#58) * fix(identity): Remove _version.py(#58) * feat(identity): Add create project tool(#58) - Add create-project tool - Tests are updated ans passing * fix(identity): Fix unnecessary test codes(#?58)
* feat(identity): Add get-project, get-projects tool(#58) - Add get-project, get-projects tool - Test code are updated and passing * chore(identity): ruff format(#58) * chore(identity): ruff format(#58) * fix(identity): Remove _version.py(#58) * feat(identity): Add create project tool(#58) - Add create-project tool - Tests are updated ans passing * fix(identity): Fix unnecessary test codes(#?58)
Overview
Key Changes
Related Issues
Additional context