Skip to content

Commit

Permalink
Minor test cleanup
Browse files Browse the repository at this point in the history
Signed-off-by: John Mazanec <[email protected]>
  • Loading branch information
jmazanec15 committed Jan 18, 2023
1 parent 4769666 commit 896816c
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/test/java/org/opensearch/knn/index/KNNSettingsTests.java
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public void testGetSettingValueFromConfig() {
long actualKNNCircuitBreakerLimit = ((ByteSizeValue) KNNSettings.state()
.getSettingValue(KNNSettings.KNN_MEMORY_CIRCUIT_BREAKER_LIMIT)).getKb();
mockNode.close();
assertEquals(actualKNNCircuitBreakerLimit, expectedKNNCircuitBreakerLimit);
assertEquals(expectedKNNCircuitBreakerLimit, actualKNNCircuitBreakerLimit);
}

@SneakyThrows
Expand All @@ -58,9 +58,10 @@ public void testGetSettingValueDefault() {
.getSettingValue(KNNSettings.KNN_MEMORY_CIRCUIT_BREAKER_LIMIT)).getKb();
mockNode.close();
assertEquals(
actualKNNCircuitBreakerLimit,
((ByteSizeValue) KNNSettings.dynamicCacheSettings.get(KNNSettings.KNN_MEMORY_CIRCUIT_BREAKER_LIMIT).getDefault(Settings.EMPTY))
.getKb()
.getKb(),
actualKNNCircuitBreakerLimit

);
}

Expand Down

0 comments on commit 896816c

Please sign in to comment.