Skip to content

Add integration tests and dataset generation tooling#19

Merged
cpdata merged 1 commit intodevfrom
codex/update-setup-files-and-documentation
Oct 17, 2025
Merged

Add integration tests and dataset generation tooling#19
cpdata merged 1 commit intodevfrom
codex/update-setup-files-and-documentation

Conversation

@cpdata
Copy link
Owner

@cpdata cpdata commented Oct 17, 2025

Summary

  • add meshmind/tests/test_integration_live.py with pytest markers and skip logic so Memgraph/Neo4j/Redis can be exercised when docker-compose services are up
  • generate scripts/generate_synthetic_dataset.py for large JSONL/CSV corpora and document the workflow across README/SETUP/docs
  • regenerate uv.lock, pin .python-version to 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 policy

Testing

  • pytest
  • pytest -m integration

https://chatgpt.com/codex/tasks/task_b_68f18ccfc1948321b9e527090b44b094

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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

Comment on lines +30 to +31
[tool.pytest.ini_options]
addopts = "-m 'not integration'"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge 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 👍 / 👎.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@codex please fix this

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary

  • Added explicit pytest configuration in pyproject.toml without 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 as tiktoken and openai are not installed in the execution environment)

View task →

@cpdata cpdata merged commit f12ff59 into dev Oct 17, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant