added few tests

Signed-off-by: Sarthak Aggarwal <sarthagg@amazon.com>
opensearch-project · Jul 28, 2023 · 0efd22e · 0efd22e
1 parent e7bfc17
commit 0efd22e
Showing 1 changed file with 29 additions and 2 deletions.
diff --git a/server/src/test/java/org/opensearch/index/codec/CodecTests.java b/server/src/test/java/org/opensearch/index/codec/CodecTests.java
@@ -62,6 +62,7 @@
 import java.util.Collections;
 
 import static org.hamcrest.Matchers.instanceOf;
+import static org.opensearch.index.engine.EngineConfig.INDEX_CODEC_COMPRESSION_LEVEL_SETTING;
 
 @SuppressCodecs("*") // we test against default codec so never get a random one here!
 public class CodecTests extends OpenSearchTestCase {
@@ -114,7 +115,7 @@ public void testZstdNoDictWithCompressionLevel() throws Exception {
 
     public void testBestCompressionWithCompressionLevel() {
         final Settings zstdSettings = Settings.builder()
-            .put(EngineConfig.INDEX_CODEC_COMPRESSION_LEVEL_SETTING.getKey(), randomIntBetween(1, 6))
+            .put(INDEX_CODEC_COMPRESSION_LEVEL_SETTING.getKey(), randomIntBetween(1, 6))
             .put(EngineConfig.INDEX_CODEC_SETTING.getKey(), randomFrom(CodecService.ZSTD_CODEC, CodecService.ZSTD_NO_DICT_CODEC))
             .build();
 
@@ -126,7 +127,7 @@ public void testBestCompressionWithCompressionLevel() {
         zstdIndexScopedSettings.validate(zstdSettings, true);
 
         final Settings settings = Settings.builder()
-            .put(EngineConfig.INDEX_CODEC_COMPRESSION_LEVEL_SETTING.getKey(), randomIntBetween(1, 6))
+            .put(INDEX_CODEC_COMPRESSION_LEVEL_SETTING.getKey(), randomIntBetween(1, 6))
             .put(EngineConfig.INDEX_CODEC_SETTING.getKey(), randomFrom(CodecService.DEFAULT_CODEC, CodecService.BEST_COMPRESSION_CODEC))
             .build();
         final IndexScopedSettings indexScopedSettings = new IndexScopedSettings(settings, IndexScopedSettings.BUILT_IN_INDEX_SETTINGS);
@@ -135,6 +136,32 @@ public void testBestCompressionWithCompressionLevel() {
         assertTrue(e.getMessage().startsWith("Compression level cannot be set"));
     }
 
+    public void testLuceneCodecsWithCompressionLevel() {
+        String codecName = randomFrom(Codec.availableCodecs());
+        Codec codec = Codec.forName(codecName);
+        final Settings settings = Settings.builder()
+            .put(INDEX_CODEC_COMPRESSION_LEVEL_SETTING.getKey(), randomIntBetween(1, 6))
+            .put(EngineConfig.INDEX_CODEC_SETTING.getKey(), codecName)
+            .build();
+        final IndexScopedSettings indexScopedSettings = new IndexScopedSettings(settings, IndexScopedSettings.BUILT_IN_INDEX_SETTINGS);
+
+        if (!(codec instanceof CodecSettings && ((CodecSettings) codec).supports(INDEX_CODEC_COMPRESSION_LEVEL_SETTING))) {
+            final IllegalArgumentException e = expectThrows(
+                IllegalArgumentException.class,
+                () -> indexScopedSettings.validate(settings, true)
+            );
+            assertTrue(e.getMessage().startsWith("Compression level cannot be set"));
+        }
+    }
+
+    public void testZstandardCompressionLevelSupport() throws Exception {
+        CodecService codecService = createCodecService(false);
+        CodecSettings zstdCodec = (CodecSettings) codecService.codec("zstd");
+        CodecSettings zstdNoDictCodec = (CodecSettings) codecService.codec("zstd_no_dict");
+        assertTrue(zstdCodec.supports(INDEX_CODEC_COMPRESSION_LEVEL_SETTING));
+        assertTrue(zstdNoDictCodec.supports(INDEX_CODEC_COMPRESSION_LEVEL_SETTING));
+    }
+
     public void testDefaultMapperServiceNull() throws Exception {
         Codec codec = createCodecService(true).codec("default");
         assertStoredFieldsCompressionEquals(Lucene95Codec.Mode.BEST_SPEED, codec);