-
Notifications
You must be signed in to change notification settings - Fork 191
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
* Fixes #1137 * Adds missing documentation. Added changelog * Added deserialization test --------- (cherry picked from commit 7b3719b) Signed-off-by: uri.nudelman <uriel.nudelman@offerup.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: uri.nudelman <uriel.nudelman@offerup.com>
- Loading branch information
1 parent
5b632ff
commit 1b13ee2
Showing
5 changed files
with
137 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
53 changes: 53 additions & 0 deletions
53
java-client/src/main/java/org/opensearch/client/util/MissingRequiredPropertiesException.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
/* | ||
* SPDX-License-Identifier: Apache-2.0 | ||
* | ||
* The OpenSearch Contributors require contributions made to | ||
* this file be licensed under the Apache-2.0 license or a | ||
* compatible open source license. | ||
*/ | ||
|
||
package org.opensearch.client.util; | ||
|
||
import java.util.StringJoiner; | ||
|
||
/** | ||
* Thrown by {@link ObjectBuilder#build()} when one of the required properties is missing. | ||
* <p> | ||
* If you think this is an error and that the reported property is actually optional, a workaround is | ||
* available in {@link ApiTypeHelper} to disable checks. Use with caution. | ||
*/ | ||
public class MissingRequiredPropertiesException extends RuntimeException { | ||
private Class<?> clazz; | ||
private String[] properties; | ||
|
||
public MissingRequiredPropertiesException(Object obj, String... properties) { | ||
super( | ||
"Missing at least one required property between " | ||
+ buildPropertiesMsg(properties) | ||
+ " in '" | ||
+ obj.getClass().getSimpleName() | ||
+ "'" | ||
); | ||
this.clazz = obj.getClass(); | ||
this.properties = properties; | ||
} | ||
|
||
/** | ||
* The class where the missing property was found | ||
*/ | ||
public Class<?> getObjectClass() { | ||
return clazz; | ||
} | ||
|
||
public String[] getPropertiesName() { | ||
return properties; | ||
} | ||
|
||
private static String buildPropertiesMsg(String[] properties) { | ||
final StringJoiner sj = new StringJoiner(",", "'", "'"); | ||
for (final String property : properties) { | ||
sj.add(property); | ||
} | ||
return sj.toString(); | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters