Remove supplier

Andrey Ershov · Andrey Ershov · commit 94f4d6303d78 · 2019-05-23T17:21:36.000-04:00
diff --git a/server/src/test/java/org/elasticsearch/snapshots/SharedClusterSnapshotRestoreIT.java b/server/src/test/java/org/elasticsearch/snapshots/SharedClusterSnapshotRestoreIT.java
@@ -1364,13 +1364,9 @@ public void testGetSnapshotsMultipleRepos() {
             }
         }
 
-        Supplier<String[]> repoNames = () -> randomFrom(new String[]{"_all"},
-                new String[]{"repo*"}, repoList.toArray(new String[0]));
-
-
         logger.info("--> get and verify snapshots");
         GetSnapshotsResponse getSnapshotsResponse = client.admin().cluster()
-                .prepareGetSnapshots(repoNames.get())
+                .prepareGetSnapshots(randomFrom(new String[]{"_all"}, new String[]{"repo*"}, repoList.toArray(new String[0])))
                 .setSnapshots(randomFrom("_all", "*"))
                 .get();
 
@@ -1381,7 +1377,6 @@ public void testGetSnapshotsMultipleRepos() {
             assertEquals(snapshotNames, snapshots.stream().map(s -> s.snapshotId().getName()).collect(Collectors.toList()));
         }
 
-
         logger.info("--> specify all snapshot names with ignoreUnavailable=false");
         GetSnapshotsResponse getSnapshotsResponse2 = client.admin().cluster()
                 .prepareGetSnapshots(randomFrom("_all", "repo*"))

Original file line number	Diff line number	Diff line change
`@@ -1364,13 +1364,9 @@ public void testGetSnapshotsMultipleRepos() {`
`1364`	`1364`	`}`
`1365`	`1365`	`}`
`1366`	`1366`
`1367`		`- Supplier<String[]> repoNames = () -> randomFrom(new String[]{"_all"},`
`1368`		`- new String[]{"repo*"}, repoList.toArray(new String[0]));`
`1369`		`-`
`1370`		`-`
`1371`	`1367`	`logger.info("--> get and verify snapshots");`
`1372`	`1368`	`GetSnapshotsResponse getSnapshotsResponse = client.admin().cluster()`
`1373`		`- .prepareGetSnapshots(repoNames.get())`
	`1369`	`+ .prepareGetSnapshots(randomFrom(new String[]{"_all"}, new String[]{"repo*"}, repoList.toArray(new String[0])))`
`1374`	`1370`	`.setSnapshots(randomFrom("_all", "*"))`
`1375`	`1371`	`.get();`
`1376`	`1372`
`@@ -1381,7 +1377,6 @@ public void testGetSnapshotsMultipleRepos() {`
`1381`	`1377`	`assertEquals(snapshotNames, snapshots.stream().map(s -> s.snapshotId().getName()).collect(Collectors.toList()));`
`1382`	`1378`	`}`
`1383`	`1379`
`1384`		`-`
`1385`	`1380`	`logger.info("--> specify all snapshot names with ignoreUnavailable=false");`
`1386`	`1381`	`GetSnapshotsResponse getSnapshotsResponse2 = client.admin().cluster()`
`1387`	`1382`	`.prepareGetSnapshots(randomFrom("_all", "repo*"))`