Skip to content

Commit 5922a29

Browse files
Updated the API readed to make simplier
1 parent bc2d667 commit 5922a29

File tree

4 files changed

+11
-12
lines changed

4 files changed

+11
-12
lines changed

src/main/java/com/uid2/operator/reader/ApiStoreReader.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,13 @@ public ApiStoreReader(DownloadCloudStorage fileStreamProvider, StoreScope scope,
2121
super(fileStreamProvider, scope, parser, dataTypeName);
2222
}
2323

24-
@Override
25-
public long loadContent(JsonObject contents, String dataType) throws Exception {
24+
public long loadContent(JsonObject contents) throws Exception {
2625
if (contents == null) {
27-
throw new IllegalArgumentException(String.format("No contents provided for loading data type %s, cannot load content", dataType));
26+
throw new IllegalArgumentException(String.format("No contents provided for loading data type %s, cannot load content", dataTypeName));
2827
}
2928

3029
try {
31-
JsonArray dataArray = contents.getJsonArray(dataType);
30+
JsonArray dataArray = contents.getJsonArray(dataTypeName);
3231
if (dataArray == null) {
3332
throw new IllegalArgumentException("No array found in the contents");
3433
}

src/main/java/com/uid2/operator/reader/RotatingCloudEncryptionKeyApiProvider.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public CloudPath getMetadataPath() {
3535

3636
@Override
3737
public long loadContent(JsonObject metadata) throws Exception {
38-
return apiStoreReader.loadContent(metadata, "cloudEncryptionKeys");
38+
return apiStoreReader.loadContent(metadata);
3939
}
4040

4141
@Override

src/test/java/com/uid2/operator/ApiStoreReaderTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,15 +51,15 @@ void getMetadataPathReturnsPathFromScope() {
5151

5252
@Test
5353
void loadContentThrowsExceptionWhenContentsAreNull() {
54-
assertThatThrownBy(() -> reader.loadContent(null, dataType))
54+
assertThatThrownBy(() -> reader.loadContent(null))
5555
.isInstanceOf(IllegalArgumentException.class)
5656
.hasMessageContaining("No contents provided for loading data type");
5757
}
5858

5959
@Test
6060
void loadContentThrowsExceptionWhenArrayNotFound() {
6161
JsonObject contents = new JsonObject();
62-
assertThatThrownBy(() -> reader.loadContent(contents, dataType))
62+
assertThatThrownBy(() -> reader.loadContent(contents))
6363
.isInstanceOf(IllegalArgumentException.class)
6464
.hasMessageContaining("No array found in the contents");
6565
}
@@ -73,7 +73,7 @@ void loadContentSuccessfullyLoadsData() throws Exception {
7373
when(mockParser.deserialize(any(InputStream.class)))
7474
.thenReturn(new ParsingResult<>(expectedData, expectedData.size()));
7575

76-
long count = reader.loadContent(contents, dataType);
76+
long count = reader.loadContent(contents);
7777

7878
assertThat(count).isEqualTo(2);
7979
assertThat(reader.getSnapshot()).isEqualTo(expectedData);

src/test/java/com/uid2/operator/RotatingCloudEncryptionKeyApiProviderTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,11 @@ void testGetMetadataPath() {
6161
@Test
6262
void testLoadContentWithMetadata() throws Exception {
6363
JsonObject metadata = new JsonObject();
64-
when(mockApiStoreReader.loadContent(metadata, "cloudEncryptionKeys")).thenReturn(1L);
64+
when(mockApiStoreReader.loadContent(metadata)).thenReturn(1L);
6565

6666
long version = rotatingCloudEncryptionKeyApiProvider.loadContent(metadata);
6767
assertEquals(1L, version);
68-
verify(mockApiStoreReader).loadContent(metadata, "cloudEncryptionKeys");
68+
verify(mockApiStoreReader).loadContent(metadata);
6969
}
7070

7171
@Test
@@ -94,10 +94,10 @@ void testGetAllWithNullSnapshot() {
9494
void testLoadContent() throws Exception {
9595
JsonObject metadata = new JsonObject().put("version", 1L);
9696
when(mockApiStoreReader.getMetadata()).thenReturn(metadata);
97-
when(mockApiStoreReader.loadContent(metadata, "cloudEncryptionKeys")).thenReturn(1L);
97+
when(mockApiStoreReader.loadContent(metadata)).thenReturn(1L);
9898

9999
rotatingCloudEncryptionKeyApiProvider.loadContent();
100100
verify(mockApiStoreReader).getMetadata();
101-
verify(mockApiStoreReader).loadContent(metadata, "cloudEncryptionKeys");
101+
verify(mockApiStoreReader).loadContent(metadata);
102102
}
103103
}

0 commit comments

Comments
 (0)