Skip to content

Add separate MCP CI job and clarify SDK job steps#7

Merged
jaakla merged 2 commits into
mainfrom
codex/investigate-ci-test-failures
May 10, 2026
Merged

Add separate MCP CI job and clarify SDK job steps#7
jaakla merged 2 commits into
mainfrom
codex/investigate-ci-test-failures

Conversation

@jaakla
Copy link
Copy Markdown
Owner

@jaakla jaakla commented May 10, 2026

Motivation

  • Ensure the MCP test suite is executed independently in CI so MCP-specific dependencies and tests do not interfere with the SDK job.
  • Clarify and rename steps in the existing SDK job to make intent explicit and keep installation scopes separate.

Description

  • Renamed the SDK job steps from Install dependencies to Install SDK dependencies and from Run tests to Run SDK tests and kept the existing coverage report generation (python scripts_report_method_test_coverage.py).
  • Added a new mcp-tests job running on ubuntu-latest with Python 3.10 that checks out the repo and installs both the SDK and MCP development dependencies using pip install -e ./merit_api and pip install -e ./mcp[dev].
  • The new job runs the MCP test suite with pytest -q mcp/tests so MCP tests run in isolation from the SDK tests.

Testing

  • The SDK test job runs pytest -q against the merit_api working directory and the coverage report generation script python scripts_report_method_test_coverage.py, and these steps completed successfully.
  • The new MCP job runs pytest -q mcp/tests and completed successfully under CI.
  • All CI jobs execute on ubuntu-latest with Python 3.10 and no automated test failures were observed.

Codex Task

@jaakla jaakla merged commit da167be into main May 10, 2026
4 checks passed
@jaakla jaakla deleted the codex/investigate-ci-test-failures branch May 10, 2026 20:40
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