Fix corner case string format bug

Signed-off-by: John Mazanec <[email protected]>
opensearch-project · Apr 6, 2023 · 7ecb036 · 7ecb036
1 parent 6b20a9d
commit 7ecb036
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/test/java/org/opensearch/knn/plugin/action/RestSearchModelHandlerIT.java b/src/test/java/org/opensearch/knn/plugin/action/RestSearchModelHandlerIT.java
@@ -87,11 +87,15 @@ public void testSizeValidationFailsInvalidSize() throws IOException {
             Request request = new Request("GET", restURI);
 
             ResponseException ex = expectThrows(ResponseException.class, () -> client().performRequest(request));
+            String messageExpected = String.format(
+                "%s must be between %s and %s inclusive",
+                PARAM_SIZE,
+                SEARCH_MODEL_MIN_SIZE,
+                SEARCH_MODEL_MAX_SIZE
+            );
             assertTrue(
-                ex.getMessage()
-                    .contains(
-                        String.format("%s must be between %d and %d inclusive", PARAM_SIZE, SEARCH_MODEL_MIN_SIZE, SEARCH_MODEL_MAX_SIZE)
-                    )
+                String.format("FAILED - Expected  \"%s\" to have \"%s\"", ex.getMessage(), messageExpected),
+                ex.getMessage().contains(messageExpected)
             );
         }