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

Tests With docker-compose #327

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

Conversation

RichardKnop
Copy link

I have added a platform agnostic way to run tests with docker-compose:

(docker-compose -f docker-compose.test.yml -p sqlx_ci up --build -d) && (docker logs -f sqlx_sut &) && (docker wait sqlx_sut)

This should make development easier as you can run tests from any machine with docker engine, you don't have to have a proper development setup locally.

Since the command is long, I have added a Makefile with alias to run tests:

make ci

@RichardKnop
Copy link
Author

@jmoiron what do you think?

Also, the build badge/label on home page is broken right now, leading to 404 page at drone.io. Should we fix it?

@jmoiron
Copy link
Owner

jmoiron commented Feb 28, 2018

Hi Richard. After a while away, I've added travis support for CI; I honestly don't know enough about the docker ecosystem to comment on the rest.

@jmoiron jmoiron closed this Feb 28, 2018
@jmoiron jmoiron reopened this Feb 28, 2018
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.

3 participants