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

Add integration test #122

Merged
merged 5 commits into from
Oct 6, 2018
Merged

Add integration test #122

merged 5 commits into from
Oct 6, 2018

Conversation

martincostello
Copy link
Member

Add an integration test that sends metrics to a real statsd instance running locally (docs) in AppVeyor or Travis, and verifies that the values sent to the server are correct by using the admin endpoint on port 8126 (docs).

A unique random prefix is used so that if you run these tests locally the values don't clash.

Add an integration test that sends metrics to a real statsd instance running locally in AppVeyor or Travis and verifies that the values sent to the server are correct by using the admin endpoint.
@martincostello martincostello added this to the v4.0.0 milestone Oct 6, 2018
@martincostello martincostello requested a review from a team as a code owner October 6, 2018 14:34
"Fix" AppVeyor build by routing std-err usage during git clone into /dev/nul (see https://help.appveyor.com/discussions/problems/555-accessing-another-repository-during-a-build).
Fix typo in file name when starting statsd in Travis.
Try using cmd instead of ps to just do a simple clone in AppVeyor.
@martincostello martincostello changed the title Add integration test [WIP] Add integration test Oct 6, 2018
Try changing the git clone path as seen in others' YAML files.
@martincostello martincostello changed the title [WIP] Add integration test Add integration test Oct 6, 2018
@AnthonySteele
Copy link
Contributor

Nice!

@martincostello martincostello merged commit 3ce1397 into justeattakeaway:master Oct 6, 2018
@martincostello martincostello deleted the IntegrationTests branch October 6, 2018 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants