Added CI tests. #88
Added CI tests. #88
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes the legacy tests.yml workflow and makes several adjustments to test configurations across the codebase. Key changes include:
- Removal of the .github/workflows/tests.yml file.
- Updates to root directory setup in several llm test files.
- Addition of new API tests for multiple endpoints related to DeepSeek, USPTO, and Pistachio models.
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/variables_test.py | Added new model variables for USPTO and Pistachio. |
| tests/llm-tests/*.py | Updated the root directory setup paths. |
| tests/api-tests/*.py | Added new API tests for various model endpoints. |
| .github/workflows/tests.yml | Removed the legacy tests workflow file. |
| .github/workflows/llm-tests.yml, api-tests.yml | Added new workflow configurations for LLM and API tests. |
Comments suppressed due to low confidence (1)
tests/api-tests/test_api_uspto_retrosynthesis.py:10
- The variable 'CLAUDE_ADV_MODEL' is imported but not defined in tests/variables_test.py. Please add its definition or update the import to reflect the correct variable.
from tests.variables_test import MOLECULE_1, BASE_URL, ENDPOINTS, X_API_KEY, DEEPSEEK_FIREWORKS_MODEL, USPTO_MODEL, CLAUDE_ADV_MODEL
There was a problem hiding this comment.
Pull Request Overview
This PR removes the obsolete tests.yml workflow file and introduces several new and updated test files along with new GitHub Actions workflows for API, LLM, and DeepSeek tests. Key changes include updating root path setups in test files, adding new API test suites with endpoints for retrosynthesis and rerun functionalities, and reorganizing workflow files.
Reviewed Changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tests/variables_test.py | Added AZ model constants and new test variables for API configurations |
| tests/llm-tests/* | Updated rootutils paths for improved test environment setup |
| tests/api-tests/* | New API test suites for different models and endpoints, with assertion updates |
| .github/workflows/* | Removed tests.yml and added new workflow files for API, LLM, and DeepSeek tests |
Uh oh!
There was an error while loading. Please reload this page.