Skip to content

Add MVP synthetic integration test #1590

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

earhart
Copy link
Contributor

@earhart earhart commented Jul 11, 2025

Context

We want to have a comprehensive integration test we can run as an action against various deployment environments.

Fixes #1586

What

This change just adds the initial form of the integration test - it's relatively simplistic, but it drives the server through its state changes, it allows the caller to vary the load on the server, and it reports some useful test metrics; it should be sufficient for building the GHA.

Testing

Ran it against a local server, and observed the generated xunit2 report.

Contribution Checklist

  • If a Python package was changed, please run make fmt in the package directory.
  • If the server was changed, please run make fmt in server/.
  • Make sure all PR Checks are passing.

@earhart earhart requested review from diptanu, eabatalov and j3m7 July 11, 2025 19:36
@earhart earhart self-assigned this Jul 11, 2025
@j3m7
Copy link
Member

j3m7 commented Jul 11, 2025

Not seeing either instructions how to run this during dev or an associated GHA that invokes this. Could you add the GHA's since it's called out on the ticket.

@earhart
Copy link
Contributor Author

earhart commented Jul 11, 2025

Can I run a GHA against a pending PR? (I was planning to add the GHA once this was in - should be pretty straightforward, except maybe for the xunit2 reporting.)

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.

MVP Scheduler Correctness Test Suite
2 participants