Skip to content

Conversation

@syedazeez337
Copy link
Contributor

Summary

This PR enables integration tests to run in GitHub Actions as part of PR checks.

Changes

  • Enabled Ko setup step in .github/workflows/test.yml for building Docker images
  • Enabled integration test step to run task test-integration
  • Removed TODO comments as the integration tests are now working

What the Integration Tests Cover

The ./test/integration-test.sh script tests all three MCP transports:

  1. STDIO Transport - Tests via Docker container with stdio
  2. SSE Transport - Tests Server-Sent Events on port 8080
  3. Streamable HTTP Transport - Tests HTTP transport on port 8081

Each transport is tested for:

  • Server connection and info retrieval
  • Tool listing
  • Tool calling with the echo tool

Technical Details

  • Ko is used to build the Docker image (task build-image)
  • Docker is already available on GitHub Actions ubuntu-latest runners
  • Tests run after unit tests pass

Test Plan

The integration tests will run automatically on this PR to verify the setup works correctly in CI.

Resolves #4

@syedazeez337 syedazeez337 force-pushed the enable-integration-tests branch from 0b2f154 to d4c7b6d Compare December 2, 2025 05:17
@syedazeez337 syedazeez337 force-pushed the enable-integration-tests branch from d4c7b6d to 854cd8c Compare December 9, 2025 08:39
- Enable Ko setup step for building Docker images
- Enable integration test step to run task test-integration
- Fix image name mismatch: align KO_DOCKER_REPO and IMAGE_NAME
  to use ghcr.io/stackloklabs/yardstick (without /server suffix)
  to match the default set by setup-ko action

The integration tests validate all three MCP transports:
- STDIO transport via Docker container
- SSE transport on port 8080
- Streamable HTTP transport on port 8081

Resolves StacklokLabs#4

Signed-off-by: Azeez Syed <[email protected]>
@syedazeez337 syedazeez337 force-pushed the enable-integration-tests branch from 854cd8c to 4ddb1d1 Compare December 9, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get integration tests running and working in Github Actions

1 participant