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

Migrate RoutingExamplesIT to Testcontainers #1215

Merged
merged 1 commit into from
May 10, 2022

Conversation

injectives
Copy link
Contributor

No description provided.

Copy link
Contributor

@michael-simons michael-simons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm… This isn't doing a real cluster connection anymore. Is that on purpose?

If so, ignore my feedback and move on.
If not, we have a JUNit extension that actually brings up a cluster: https://github.com/michael-simons/junit-jupiter-causal-cluster-testcontainer-extension

@injectives
Copy link
Contributor Author

It is a single node cluster. This should be sufficient for these tests and will make build a bit lighter.

@injectives
Copy link
Contributor Author

I have seen the exciting extension that we might use when we need a multinode cluster. 👍

@injectives injectives force-pushed the feature/example branch 2 times, most recently from 9dfe34a to 0f79ba2 Compare May 10, 2022 08:40
@injectives injectives merged commit dfb609a into neo4j:5.0 May 10, 2022
@injectives injectives deleted the feature/example branch May 10, 2022 12:19
injectives added a commit to injectives/neo4j-java-driver that referenced this pull request Nov 22, 2022
injectives added a commit that referenced this pull request Nov 23, 2022
* Migrate RoutingExamplesIT to Testcontainers (#1215)

* Migrate java-driver-rx-tck (#1224)

* Migrate java-driver-rx-tck

This update migrates verification tests from `java-driver-rx-tck` to this project.

* Update surefire and failsafe to 3.0.0-M6

* Migrate tests depending on Boltkit to use Docker (#1256)

Boltkit has been deprecated and requires additional setup on the host to run tests. This update migrates tests depending on Boltkit to use Docker that is needed for Testkit testing as well.

* Skip tests incompatible with 5.x server
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