Skip to content

Commit 04c4bb1

Browse files
committed
suppress forbidden api usage
1 parent 624a365 commit 04c4bb1

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

x-pack/plugin/logsdb/src/javaRestTest/java/org/elasticsearch/xpack/logsdb/LogsdbSnapshotRestoreIT.java

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
import org.elasticsearch.common.settings.Settings;
1616
import org.elasticsearch.common.time.DateFormatter;
1717
import org.elasticsearch.common.time.FormatNames;
18+
import org.elasticsearch.core.SuppressForbidden;
1819
import org.elasticsearch.repositories.fs.FsRepository;
1920
import org.elasticsearch.test.cluster.ElasticsearchCluster;
2021
import org.elasticsearch.test.cluster.local.distribution.DistributionType;
@@ -45,7 +46,7 @@ public class LogsdbSnapshotRestoreIT extends ESRestTestCase {
4546

4647
private static ElasticsearchCluster cluster = ElasticsearchCluster.local()
4748
.distribution(DistributionType.DEFAULT)
48-
.setting("path.repo", () -> repoDirectory.getRoot().getPath())
49+
.setting("path.repo", () -> getRepoPath())
4950
.setting("xpack.security.enabled", "false")
5051
.setting("xpack.license.self_generated.type", "trial")
5152
.build();
@@ -162,17 +163,15 @@ public void cleanup() throws Exception {
162163
deleteDataStream("logs-my-test");
163164
}
164165

166+
@SuppressForbidden(reason = "TemporaryFolder only has io.File methods, not nio.File")
165167
static void snapshotAndRestore(String sourceMode, String arrayType, boolean sourceOnly) throws IOException {
166168
String dataStreamName = "logs-my-test";
167169
String repositoryName = "my-repository";
168170
if (sourceOnly) {
169-
var repositorySettings = Settings.builder()
170-
.put("delegate_type", "fs")
171-
.put("location", repoDirectory.getRoot().getPath())
172-
.build();
171+
var repositorySettings = Settings.builder().put("delegate_type", "fs").put("location", getRepoPath()).build();
173172
registerRepository(repositoryName, "source", true, repositorySettings);
174173
} else {
175-
var repositorySettings = Settings.builder().put("location", repoDirectory.getRoot().getPath()).build();
174+
var repositorySettings = Settings.builder().put("location", getRepoPath()).build();
176175
registerRepository(repositoryName, FsRepository.TYPE, true, repositorySettings);
177176
}
178177

@@ -214,13 +213,10 @@ static void snapshotAndFail(String sourceMode, String arrayType, boolean sourceO
214213
String dataStreamName = "logs-my-test";
215214
String repositoryName = "my-repository";
216215
if (sourceOnly) {
217-
var repositorySettings = Settings.builder()
218-
.put("delegate_type", "fs")
219-
.put("location", repoDirectory.getRoot().getPath())
220-
.build();
216+
var repositorySettings = Settings.builder().put("delegate_type", "fs").put("location", getRepoPath()).build();
221217
registerRepository(repositoryName, "source", true, repositorySettings);
222218
} else {
223-
var repositorySettings = Settings.builder().put("location", repoDirectory.getRoot().getPath()).build();
219+
var repositorySettings = Settings.builder().put("location", getRepoPath()).build();
224220
registerRepository(repositoryName, FsRepository.TYPE, true, repositorySettings);
225221
}
226222

@@ -338,4 +334,9 @@ public static void assertDocCount(RestClient client, String indexName, long docC
338334
);
339335
}
340336

337+
@SuppressForbidden(reason = "TemporaryFolder only has io.File methods, not nio.File")
338+
private static String getRepoPath() {
339+
return repoDirectory.getRoot().getPath();
340+
}
341+
341342
}

0 commit comments

Comments
 (0)