Adjust test to updated default of index checksum on merge

danielmitterdorfer · danielmitterdorfer · commit c5a85ceafbd5 · 2016-07-26T09:12:11.000+02:00
In elastic#19114 we set the default for index checksum on merge to true but forgot to update the assertions in InternalEngineSettingsTest. With this commit, the test is adjusted to the new defaults.
diff --git a/src/test/java/org/elasticsearch/index/engine/InternalEngineSettingsTest.java b/src/test/java/org/elasticsearch/index/engine/InternalEngineSettingsTest.java
@@ -41,11 +41,11 @@ public void testSettingsUpdate() {
         assertThat(engine.getCurrentIndexWriterConfig().getUseCompoundFile(), is(true));
 
         // INDEX_CHECKSUM_ON_MERGE
-        assertThat(engine.getCurrentIndexWriterConfig().getCheckIntegrityAtMerge(), is(false));
-        client().admin().indices().prepareUpdateSettings("foo").setSettings(ImmutableSettings.builder().put(EngineConfig.INDEX_CHECKSUM_ON_MERGE, true).build()).get();
         assertThat(engine.getCurrentIndexWriterConfig().getCheckIntegrityAtMerge(), is(true));
         client().admin().indices().prepareUpdateSettings("foo").setSettings(ImmutableSettings.builder().put(EngineConfig.INDEX_CHECKSUM_ON_MERGE, false).build()).get();
         assertThat(engine.getCurrentIndexWriterConfig().getCheckIntegrityAtMerge(), is(false));
+        client().admin().indices().prepareUpdateSettings("foo").setSettings(ImmutableSettings.builder().put(EngineConfig.INDEX_CHECKSUM_ON_MERGE, true).build()).get();
+        assertThat(engine.getCurrentIndexWriterConfig().getCheckIntegrityAtMerge(), is(true));
 
         // VERSION MAP SIZE
         long indexBufferSize = engine.config().getIndexingBufferSize().bytes();
