Changing messaging for ILlegalArgumentException on duplicate model gr… (

#1294) (#1299) * Changing messaging for ILlegalArgumentException on duplicate model group creation. Signed-off-by: Nate Boot <nateboot@amazon.com> * Adjusting the test. `./gradlew test` passes. Signed-off-by: Nate Boot <nateboot@amazon.com> --------- Signed-off-by: Nate Boot <nateboot@amazon.com> (cherry picked from commit a9687fc) Co-authored-by: Nate B <96254688+nateynateynate@users.noreply.github.com>
opensearch-project · Sep 8, 2023 · 979ac68 · 979ac68
1 parent 99679bc
commit 979ac68
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/plugin/src/main/java/org/opensearch/ml/model/MLModelGroupManager.java b/plugin/src/main/java/org/opensearch/ml/model/MLModelGroupManager.java
@@ -75,11 +75,7 @@ public void createModelGroup(MLRegisterModelGroupInput input, ActionListener<Str
                             listener
                                 .onFailure(
                                     new IllegalArgumentException(
-                                        "The name you provided is already being used by another model with ID: "
-                                            + id
-                                            + ". Please provide a different name or add \"model_group_id\": \""
-                                            + id
-                                            + "\" to request body"
+                                        "The name you provided is already being used by a model group with ID: " + id + "."
                                     )
                                 );
                         }

diff --git a/plugin/src/test/java/org/opensearch/ml/model/MLModelGroupManagerTests.java b/plugin/src/test/java/org/opensearch/ml/model/MLModelGroupManagerTests.java
@@ -143,7 +143,7 @@ public void test_ModelGroupNameNotUnique() throws IOException {//
         ArgumentCaptor<Exception> argumentCaptor = ArgumentCaptor.forClass(Exception.class);
         verify(actionListener).onFailure(argumentCaptor.capture());
         assertEquals(
-            "The name you provided is already being used by another model with ID: model_group_ID. Please provide a different name or add \"model_group_id\": \"model_group_ID\" to request body",
+            "The name you provided is already being used by a model group with ID: model_group_ID.",
             argumentCaptor.getValue().getMessage()
         );