Skip to content
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

Rate-limited test routine #6

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

adesjardins-stripe
Copy link
Contributor

This PR introduces an entirely new test routine command.

Aside from a rate-limiter, this routine also includes support for:

  • meta commands
  • pre-test cache warming
  • pre-generated keys
  • expected value comparisons
  • finite test runs
  • deterministic test runs (rng seed parameter)
  • run stats report

I also implemented some basic module support. Going one step further, vendoring the dependencies could be worth considering.

@dormando
Copy link
Member

Ah wow I had completely forgotten about this PR; there's no activity on issues/pr's on this repo so I don't naturally land here much.

Did you ever end up using this?

@adesjardins-stripe
Copy link
Contributor Author

Hey @dormando, it's all good. I would have bumped this if we needed action on it. In the end, we opted for some alternative testing procedures that were integrated within our in-house systems.

@dormando
Copy link
Member

Thanks. That wasn't a great time for me so I wasn't able to keep us all in sync. I've been using mctester for the recent regression finding and need to decide how or whether to move forward with it for automation.

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.

2 participants