Skip to content

Commit ccf50a9

Browse files
committed
BasicLoadBalancingPolicyPreferredRemoteDcsTest adjustments
1 parent dbf4224 commit ccf50a9

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

core/src/test/java/com/datastax/oss/driver/internal/core/loadbalancing/BasicLoadBalancingPolicyPreferredRemoteDcsTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ public void should_prioritize_single_replica() {
5353
when(request.getRoutingKeyspace()).thenReturn(KEYSPACE);
5454
when(request.getRoutingKey()).thenReturn(ROUTING_KEY);
5555
when(tokenMap.getReplicas(KEYSPACE, ROUTING_KEY)).thenReturn(ImmutableSet.of(node3));
56+
when(tokenMap.getReplicas(eq(KEYSPACE), any(), eq(ROUTING_KEY)))
57+
.thenReturn(ImmutableSet.of(node3));
5658

5759
// node3 always first, round-robin on the rest
5860
assertThat(policy.newQueryPlan(request, session))
@@ -81,6 +83,8 @@ public void should_prioritize_and_shuffle_replicas() {
8183
when(request.getRoutingKey()).thenReturn(ROUTING_KEY);
8284
when(tokenMap.getReplicas(KEYSPACE, ROUTING_KEY))
8385
.thenReturn(ImmutableSet.of(node1, node2, node3, node6, node9));
86+
when(tokenMap.getReplicas(eq(KEYSPACE), any(), eq(ROUTING_KEY)))
87+
.thenReturn(ImmutableSet.of(node1, node2, node3, node6, node9));
8488

8589
// node 6 and 9 being in a remote DC, they don't get a boost for being a replica
8690
assertThat(policy.newQueryPlan(request, session))

core/src/test/java/com/datastax/oss/driver/internal/core/metadata/DefaultTopologyMonitorTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,7 @@ public void should_get_peer_address_info_peers_v2() {
502502
AdminRow peer1 = mockPeersV2Row(2, node1.getHostId());
503503
topologyMonitor.isSchemaV2 = true;
504504
topologyMonitor.stubQueries(
505-
new StubbedQuery("SELECT * FROM system.local", mockResult(local)),
505+
new StubbedQuery("SELECT * FROM system.local WHERE key='local'", mockResult(local)),
506506
new StubbedQuery("SELECT * FROM system.peers_v2", mockResult(peer2, peer1)));
507507
when(context.getSslEngineFactory()).thenReturn(Optional.empty());
508508

@@ -534,7 +534,7 @@ public void should_get_peer_address_info_peers_dse() {
534534
AdminRow peer1 = mockPeersRowDse(2, node1.getHostId());
535535
topologyMonitor.isSchemaV2 = true;
536536
topologyMonitor.stubQueries(
537-
new StubbedQuery("SELECT * FROM system.local", mockResult(local)),
537+
new StubbedQuery("SELECT * FROM system.local WHERE key='local'", mockResult(local)),
538538
new StubbedQuery("SELECT * FROM system.peers_v2", Maps.newHashMap(), null, true),
539539
new StubbedQuery("SELECT * FROM system.peers", mockResult(peer2, peer1)));
540540
when(context.getSslEngineFactory()).thenReturn(Optional.empty());
@@ -567,7 +567,7 @@ public void should_get_peer_address_info_peers_dse_with_ssl() {
567567
AdminRow peer1 = mockPeersRowDseWithSsl(2, node1.getHostId());
568568
topologyMonitor.isSchemaV2 = true;
569569
topologyMonitor.stubQueries(
570-
new StubbedQuery("SELECT * FROM system.local", mockResult(local)),
570+
new StubbedQuery("SELECT * FROM system.local WHERE key='local'", mockResult(local)),
571571
new StubbedQuery("SELECT * FROM system.peers_v2", Maps.newHashMap(), null, true),
572572
new StubbedQuery("SELECT * FROM system.peers", mockResult(peer2, peer1)));
573573
when(context.getSslEngineFactory()).thenReturn(Optional.of(sslEngineFactory));

0 commit comments

Comments
 (0)