Skip to content

Commit 2a1dc17

Browse files
committed
Clarify function
1 parent 98a1b6b commit 2a1dc17

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

server/src/test/java/org/elasticsearch/action/admin/cluster/settings/SettingsUpdaterTests.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030

3131
import java.util.Set;
3232
import java.util.concurrent.atomic.AtomicReference;
33+
import java.util.function.BiFunction;
3334
import java.util.function.Function;
3435
import java.util.stream.Collectors;
3536
import java.util.stream.Stream;
@@ -169,12 +170,12 @@ public void testDeprecationLogging() {
169170
}
170171

171172
public void testUpdateWithUnknownAndSettings() {
172-
runUpdateWithUnknownAndInvalidSettingTest(builder -> builder::persistentSettings, MetaData::persistentSettings);
173-
runUpdateWithUnknownAndInvalidSettingTest(builder -> builder::transientSettings, MetaData::transientSettings);
173+
runUpdateWithUnknownAndInvalidSettingTest(MetaData.Builder::persistentSettings, MetaData::persistentSettings);
174+
runUpdateWithUnknownAndInvalidSettingTest(MetaData.Builder::transientSettings, MetaData::transientSettings);
174175
}
175176

176177
private void runUpdateWithUnknownAndInvalidSettingTest(
177-
final Function<MetaData.Builder, Function<Settings, MetaData.Builder>> function,
178+
final BiFunction<MetaData.Builder, Settings, MetaData.Builder> metaDataSettingsBuilder,
178179
final Function<MetaData, Settings> settingsToTest) {
179180
final Setting<String> dynamicSetting = Setting.simpleString("dynamic.setting", Property.Dynamic, Property.NodeScope);
180181
final Setting<String> invalidSetting = Setting.simpleString(
@@ -196,7 +197,7 @@ private void runUpdateWithUnknownAndInvalidSettingTest(
196197
final ClusterState clusterState =
197198
ClusterState
198199
.builder(new ClusterName("cluster"))
199-
.metaData(function.apply(MetaData.builder()).apply(settings).build())
200+
.metaData(metaDataSettingsBuilder.apply(MetaData.builder(), settings).build())
200201
.build();
201202
final Settings toApply = Settings.builder().put("dynamic.setting", "value").build();
202203
final boolean applyTransient = randomBoolean();

0 commit comments

Comments
 (0)