search

scylladb / java-driver

ScyllaDB Java Driver for ScyllaDB and Apache Cassandra, based on the DataStax Java Driver
Apache License 2.0
61 stars 37 forks source link

More MockResolver tests #337

Closed Bouncheck closed 3 weeks ago

Bouncheck commented 4 weeks ago

Adds one more integartion test method and unit tests for ResolverProvider Applied on top of #335

dkropachev commented 3 weeks ago

@Bouncheck , let's rebase it and continue working on it

dkropachev commented 3 weeks ago

@Bouncheck , ping

Bouncheck commented 3 weeks ago

Looks like with some cluster versions the test manages to restart cluster 17-27 times before it runs out of time to reproduce the socket replacement issue. Maybe I should remove @Test from it and just have it like that as a reproducer for later?

dkropachev commented 3 weeks ago

@Bouncheck , let's remove cannot_reconnect_with_resolved_socket from this PR, and create another PR to address this particular issue, that is going include fix for it, and mark test to be skipped with message that describes what test does and why we need it there.

Bouncheck commented 3 weeks ago

Removed