Skip to content

Commit 940dbb9

Browse files
committed
Allow retrieving bucket prefix from storage configuration katta-server.
1 parent fe21ab4 commit 940dbb9

File tree

3 files changed

+48
-22
lines changed

3 files changed

+48
-22
lines changed

hub/src/main/java/cloud/katta/model/StorageProfileDtoWrapper.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,8 +115,7 @@ else if(proxy.getActualInstance() instanceof StorageProfileS3STSDto) {
115115

116116
public String getBucketPrefix() {
117117
if(proxy.getActualInstance() instanceof StorageProfileS3StaticDto) {
118-
// only STS
119-
return null;
118+
return ((StorageProfileS3StaticDto) proxy.getActualInstance()).getBucketPrefix();
120119
}
121120
else if(proxy.getActualInstance() instanceof StorageProfileS3STSDto) {
122121
return ((StorageProfileS3STSDto) proxy.getActualInstance()).getBucketPrefix();

hub/src/test/java/cloud/katta/workflows/AbstractHubSynchronizeTest.java

Lines changed: 46 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,57 @@
44

55
package cloud.katta.workflows;
66

7-
import ch.cyberduck.core.*;
7+
import ch.cyberduck.core.AlphanumericRandomStringService;
8+
import ch.cyberduck.core.AttributedList;
9+
import ch.cyberduck.core.DisabledConnectionCallback;
10+
import ch.cyberduck.core.DisabledListProgressListener;
11+
import ch.cyberduck.core.DisabledLoginCallback;
12+
import ch.cyberduck.core.DisabledPasswordCallback;
13+
import ch.cyberduck.core.ListService;
14+
import ch.cyberduck.core.OAuthTokens;
15+
import ch.cyberduck.core.Path;
16+
import ch.cyberduck.core.SimplePathPredicate;
17+
import ch.cyberduck.core.UUIDRandomStringService;
818
import ch.cyberduck.core.exception.AccessDeniedException;
919
import ch.cyberduck.core.exception.NotfoundException;
10-
import ch.cyberduck.core.features.*;
20+
import ch.cyberduck.core.features.AttributesFinder;
21+
import ch.cyberduck.core.features.Delete;
22+
import ch.cyberduck.core.features.Directory;
23+
import ch.cyberduck.core.features.Find;
24+
import ch.cyberduck.core.features.Home;
25+
import ch.cyberduck.core.features.Move;
26+
import ch.cyberduck.core.features.Vault;
27+
import ch.cyberduck.core.features.Write;
1128
import ch.cyberduck.core.transfer.TransferStatus;
1229
import ch.cyberduck.core.vault.VaultCredentials;
1330
import ch.cyberduck.core.vault.VaultRegistry;
31+
32+
import org.apache.commons.lang3.RandomUtils;
33+
import org.apache.commons.lang3.StringUtils;
34+
import org.apache.logging.log4j.LogManager;
35+
import org.apache.logging.log4j.Logger;
36+
import org.junit.jupiter.api.MethodOrderer;
37+
import org.junit.jupiter.api.TestMethodOrder;
38+
import org.junit.jupiter.params.ParameterizedTest;
39+
import org.junit.jupiter.params.provider.MethodSource;
40+
import org.openapitools.jackson.nullable.JsonNullableModule;
41+
42+
import java.util.Collections;
43+
import java.util.EnumSet;
44+
import java.util.HashMap;
45+
import java.util.List;
46+
import java.util.Properties;
47+
import java.util.UUID;
48+
import java.util.stream.Collectors;
49+
1450
import cloud.katta.client.ApiClient;
1551
import cloud.katta.client.ApiException;
1652
import cloud.katta.client.api.StorageProfileResourceApi;
17-
import cloud.katta.client.model.*;
53+
import cloud.katta.client.model.S3SERVERSIDEENCRYPTION;
54+
import cloud.katta.client.model.S3STORAGECLASSES;
55+
import cloud.katta.client.model.StorageProfileDto;
56+
import cloud.katta.client.model.StorageProfileS3STSDto;
57+
import cloud.katta.client.model.StorageProfileS3StaticDto;
1858
import cloud.katta.crypto.uvf.UvfMetadataPayload;
1959
import cloud.katta.crypto.uvf.VaultMetadataJWEAutomaticAccessGrantDto;
2060
import cloud.katta.crypto.uvf.VaultMetadataJWEBackendDto;
@@ -26,23 +66,12 @@
2666
import cloud.katta.testsetup.AbstractHubTest;
2767
import cloud.katta.testsetup.HubTestConfig;
2868
import cloud.katta.testsetup.HubTestUtilities;
29-
import static cloud.katta.testsetup.HubTestUtilities.getAdminApiClient;
3069
import cloud.katta.testsetup.MethodIgnorableSource;
3170
import com.fasterxml.jackson.annotation.JsonInclude;
3271
import com.fasterxml.jackson.databind.ObjectMapper;
33-
import org.apache.commons.lang3.RandomUtils;
34-
import org.apache.commons.lang3.StringUtils;
35-
import org.apache.logging.log4j.LogManager;
36-
import org.apache.logging.log4j.Logger;
37-
import static org.junit.jupiter.api.Assertions.*;
38-
import org.junit.jupiter.api.MethodOrderer;
39-
import org.junit.jupiter.api.TestMethodOrder;
40-
import org.junit.jupiter.params.ParameterizedTest;
41-
import org.junit.jupiter.params.provider.MethodSource;
42-
import org.openapitools.jackson.nullable.JsonNullableModule;
4372

44-
import java.util.*;
45-
import java.util.stream.Collectors;
73+
import static cloud.katta.testsetup.HubTestUtilities.getAdminApiClient;
74+
import static org.junit.jupiter.api.Assertions.*;
4675

4776
@TestMethodOrder(MethodOrderer.MethodName.class)
4877
abstract class AbstractHubSynchronizeTest extends AbstractHubTest {
@@ -225,8 +254,7 @@ void test03AddVault(final HubTestConfig config) throws Exception {
225254
log.info("Creating vault in {}", hubSession);
226255
final UUID vaultId = UUID.fromString(new UUIDRandomStringService().random());
227256

228-
final Path bucket = new Path(null == storageProfileWrapper.getBucketPrefix() ? "katta-test-" + vaultId : storageProfileWrapper.getBucketPrefix() + vaultId,
229-
EnumSet.of(Path.Type.volume, Path.Type.directory));
257+
final Path bucket = new Path(storageProfileWrapper.getBucketPrefix() + vaultId, EnumSet.of(Path.Type.volume, Path.Type.directory));
230258
final HubStorageLocationService.StorageLocation location = new HubStorageLocationService.StorageLocation(storageProfileWrapper.getId().toString(), storageProfileWrapper.getRegion(),
231259
storageProfileWrapper.getName());
232260
final UvfMetadataPayload vaultMetadata = UvfMetadataPayload.create()

hub/src/test/java/cloud/katta/workflows/AbstractHubWorkflowTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,7 @@ void testHubWorkflow(final HubTestConfig config) throws Exception {
111111
.filter(p -> p.getId().toString().equals(config.vault.storageProfileId.toLowerCase())).findFirst().get();
112112

113113
final UUID vaultId = UUID.fromString(new UUIDRandomStringService().random());
114-
final Path bucket = new Path(null == storageProfileWrapper.getBucketPrefix() ? "katta-test-" + vaultId : storageProfileWrapper.getBucketPrefix() + vaultId,
115-
EnumSet.of(Path.Type.volume, Path.Type.directory));
114+
final Path bucket = new Path(storageProfileWrapper.getBucketPrefix() + vaultId, EnumSet.of(Path.Type.volume, Path.Type.directory));
116115
final HubStorageLocationService.StorageLocation location = new HubStorageLocationService.StorageLocation(storageProfileWrapper.getId().toString(), storageProfileWrapper.getRegion(),
117116
storageProfileWrapper.getName());
118117
final UvfMetadataPayload vaultMetadata = UvfMetadataPayload.create()

0 commit comments

Comments
 (0)