Add integration tests and dataset generation tooling#19
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| [tool.pytest.ini_options] | ||
| addopts = "-m 'not integration'" |
There was a problem hiding this comment.
Remove default marker filter to allow integration suite to run
The new pytest configuration unconditionally injects -m 'not integration'. When someone follows the documented instructions and runs pytest -m integration, pytest combines both -m expressions with logical AND, so the expression becomes integration and not integration and zero tests are collected. As written, there is no straightforward way to execute the integration suite without overriding addopts, which defeats the purpose of adding these tests.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Summary
- Added explicit pytest configuration in
pyproject.tomlwithout the default-m "not integration"filter so the integration suite can be requested explicitly while retaining strict markers and warning suppression. pyproject.tomlL26-L33
Testing
⚠️ pytest -m integration(fails because optional dependencies such astiktokenandopenaiare not installed in the execution environment)
Summary
meshmind/tests/test_integration_live.pywith pytest markers and skip logic so Memgraph/Neo4j/Redis can be exercised when docker-compose services are upscripts/generate_synthetic_dataset.pyfor large JSONL/CSV corpora and document the workflow across README/SETUP/docsuv.lock, pin.python-versionto 3.12, and update configuration/planning docs (README, SETUP, docs/*, PROJECT, PLAN, SOT, ROADMAP, RECOMMENDATIONS, TODO, RESUME_NOTES, ENVIRONMENT_NEEDS, NEEDED_FOR_TESTING, DUMMIES, ISSUES) including the new Pydantic 2.x policyTesting
https://chatgpt.com/codex/tasks/task_b_68f18ccfc1948321b9e527090b44b094