-
Notifications
You must be signed in to change notification settings - Fork 12
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 docker-based integration test infrastructure #14
Comments
@heynickc Not necessary. We have AppVeyor CI and it worked for integration tests before. I have to realize why it stopped working |
@alexvaluyskiy this is an additional feature, not a bug fix. this is how we test our other plugins against live databases to allow integration testing against different versions of the databases |
@alexvaluyskiy if you would like to discontinue testing our persistence plugins this way, please bring it up with the core team to reach a consensus and we will discontinue testing this way across all plugins |
@alexvaluyskiy better off standardizing on Docker |
disregard, since this is a contrib package now |
To be consistent with SqlServer, Postgres, Cassandra plugins, test against a container running MySql
The text was updated successfully, but these errors were encountered: