diff --git a/src/test/java/org/opensearch/knn/integ/PainlessScriptFieldsIT.java b/src/test/java/org/opensearch/knn/integ/PainlessScriptFieldsIT.java index c68ca2a6fb..8f9e1ed298 100644 --- a/src/test/java/org/opensearch/knn/integ/PainlessScriptFieldsIT.java +++ b/src/test/java/org/opensearch/knn/integ/PainlessScriptFieldsIT.java @@ -30,7 +30,7 @@ // it is clear if similarity method is supported by script_score, then same is applicable for script_fields // provided script_fields context is supported. Hence, we test for one similarity method to verify that script_fields // context is supported by this plugin. -public class PainlessScriptFieldsIT extends KNNRestTestCase { +public final class PainlessScriptFieldsIT extends KNNRestTestCase { private static final String NUMERIC_INDEX_FIELD_NAME = "price"; diff --git a/src/test/java/org/opensearch/knn/integ/PainlessScriptHelper.java b/src/test/java/org/opensearch/knn/integ/PainlessScriptHelper.java index 800f1a1ec0..a5001c7dd5 100644 --- a/src/test/java/org/opensearch/knn/integ/PainlessScriptHelper.java +++ b/src/test/java/org/opensearch/knn/integ/PainlessScriptHelper.java @@ -15,8 +15,7 @@ import java.util.List; import java.util.Objects; -public class PainlessScriptHelper { - +public final class PainlessScriptHelper { /** * Utility to create a Index Mapping with multiple fields */ @@ -45,7 +44,7 @@ public static String createMapping(List properties) throws IOEx return xContentBuilder.toString(); } - static class MappingProperty { + final static class MappingProperty { private final String name; private final String type; diff --git a/src/test/java/org/opensearch/knn/integ/PainlessScriptScoreIT.java b/src/test/java/org/opensearch/knn/integ/PainlessScriptScoreIT.java index 03e8f68a25..3f96bac9c9 100644 --- a/src/test/java/org/opensearch/knn/integ/PainlessScriptScoreIT.java +++ b/src/test/java/org/opensearch/knn/integ/PainlessScriptScoreIT.java @@ -36,7 +36,7 @@ import static org.opensearch.knn.common.KNNConstants.METHOD_HNSW; import static org.opensearch.knn.integ.PainlessScriptHelper.createMapping; -public class PainlessScriptScoreIT extends KNNRestTestCase { +public final class PainlessScriptScoreIT extends KNNRestTestCase { public static final int AGGREGATION_FIELD_NAME_MIN_LENGTH = 2; public static final int AGGREGATION_FIELD_NAME_MAX_LENGTH = 5;