-
-
Notifications
You must be signed in to change notification settings - Fork 251
Add ClickHouse module #981
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 ClickHouse module #981
Conversation
✅ Deploy Preview for testcontainers-node ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Used this for reference for the wait strategy: https://github.com/testcontainers/testcontainers-dotnet/blob/develop/src/Testcontainers.ClickHouse/ClickHouseBuilder.cs#L91-L92 |
…ient and fix tests
|
@cristianrgreco thanks for the help! I fixed the tests and cleaned up the implementation a bit, so it should be ready for review now 😄 |
… docs inconsistencies
|
@cristianrgreco thanks for the review! I incorporated your changes, let me know if anything else needs tweaking! 😊 |
Adds an initial implementation for a ClickHouse module.
To run the tests
npx vitest run packages/modules/clickhouse/src/clickhouse-container.test.tsThe implementation and tests are heavily inspired by the Postgres module.