-
Notifications
You must be signed in to change notification settings - Fork 2
214 create flask api tests that validate endpoint functions #280
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
base: develop
Are you sure you want to change the base?
214 create flask api tests that validate endpoint functions #280
Conversation
…ng in test environment
…tests-that-validate-endpoint-functions
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 implements a comprehensive refactoring of the Flask API backend into separate functional modules with extensive testing coverage. The main changes organize the codebase into a blueprint-based architecture with dedicated modules for deployment, OSPARC integration, text file operations, sampling operations, and Dakota algorithms. Tests validate endpoint functionality, error handling, and integration points across all modules.
Key Changes:
- Complete Flask API refactoring into blueprint-based modular architecture
- Comprehensive test suite covering all major endpoints and error scenarios
- Addition of Pydantic-based request validation for sampling endpoints
Reviewed Changes
Copilot reviewed 67 out of 74 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| node/src/utils/sampling_utils.ts | Updates sampling endpoint URL paths to use new /sampling/ prefix |
| node/src/utils/function_utils.ts | Updates OSPARC and deployment endpoint URLs to use new prefixes |
| node/src/components/sampling/*.tsx | Updates sampling component endpoint URLs to new paths |
| node/src/components/plots/*.tsx | Updates Dakota plot endpoint URLs to new /dakota/ prefix |
| node/package.json | Adds vitest coverage package dependency |
| flaskapi/tests/*.py | Adds comprehensive test suites for all Flask modules |
| flaskapi/src/mmux_flaskapi/blueprints/*.py | New modular blueprint architecture for Flask app |
| flaskapi/src/mmux_flaskapi/utils/*.py | New utility modules for configuration and helpers |
| flaskapi/pyproject.toml | New Python project configuration with pytest setup |
| flaskapi/main.py | New application entry point |
| flaskapi/Dockerfile | Updates to use new Makefile-based dependency installation |
| docker-compose-local.yml | Removes deprecated version field |
Files not reviewed (1)
- node/package-lock.json: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This PR includes:
close #214
Once this PR is merged, should automatically close or make easier to address: