Skip to content

Commit 61ed074

Browse files
authored
Merge pull request #330 from AAFC-BICoE/28321_Update_dina-base_0.91
28321 Update dina-base 0.91
2 parents 14243ae + 927823e commit 61ed074

File tree

2 files changed

+23
-32
lines changed

2 files changed

+23
-32
lines changed

pom.xml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
<parent>
1515
<groupId>org.springframework.boot</groupId>
1616
<artifactId>spring-boot-starter-parent</artifactId>
17-
<version>2.6.6</version>
17+
<version>2.7.2</version>
1818
</parent>
1919

2020
<properties>
@@ -32,18 +32,16 @@
3232
<pdfbox.version>2.0.25</pdfbox.version>
3333
<imageio.version>3.8.2</imageio.version>
3434

35-
<spotbugs-maven-plugin.version>4.5.3.0</spotbugs-maven-plugin.version>
36-
<spotbugs.version>4.5.3</spotbugs.version>
35+
<spotbugs-maven-plugin.version>4.6.0.0</spotbugs-maven-plugin.version>
36+
<spotbugs.version>4.7.1</spotbugs.version>
3737
<jcip-annotations.version>1.0</jcip-annotations.version>
38-
<dependency-check.version>6.5.3</dependency-check.version>
38+
<dependency-check.version>7.1.1</dependency-check.version>
3939
<jacoco-maven-plugin.version>0.8.5</jacoco-maven-plugin.version>
4040
<maven-checkstyle-plugin.version>3.1.2</maven-checkstyle-plugin.version>
41-
<checkstyle.version>8.45.1</checkstyle.version>
41+
<checkstyle.version>9.3</checkstyle.version>
4242
<asciidoctor-maven-plugin.version>2.1.0</asciidoctor-maven-plugin.version>
4343
<asciidoctorj.diagram.version>2.0.2</asciidoctorj.diagram.version>
44-
<dina-base-api.version>0.86</dina-base-api.version>
45-
46-
<liquibase.version>4.9.0</liquibase.version>
44+
<dina-base-api.version>0.91</dina-base-api.version>
4745

4846
<!-- Override to avoid NoClassDefFoundError: org/apache/maven/doxia/siterenderer/DocumentContent -->
4947
<maven-site-plugin.version>3.7.1</maven-site-plugin.version>

src/test/java/ca/gc/aafc/objectstore/api/openapi/DerivativeOpenApiIT.java

Lines changed: 17 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,16 @@
11
package ca.gc.aafc.objectstore.api.openapi;
22

3-
import java.net.MalformedURLException;
4-
import java.net.URISyntaxException;
5-
import java.net.URL;
63
import java.time.OffsetDateTime;
4+
import java.util.List;
75
import java.util.UUID;
8-
import java.util.Map;
96

107
import javax.inject.Inject;
118
import javax.persistence.criteria.CriteriaBuilder;
129
import javax.persistence.criteria.CriteriaDelete;
1310
import javax.persistence.criteria.Root;
1411
import javax.transaction.Transactional;
1512

16-
import org.apache.http.client.utils.URIBuilder;
13+
import ca.gc.aafc.dina.testsupport.jsonapi.JsonAPIRelationship;
1714
import org.junit.jupiter.api.AfterEach;
1815
import org.junit.jupiter.api.BeforeEach;
1916
import org.junit.jupiter.api.Test;
@@ -85,17 +82,18 @@ public void setup() {
8582

8683
ObjectStoreMetadataDto osMetadata = buildObjectStoreMetadataDto();
8784

88-
metadataUuid = sendPost("metadata", JsonAPITestHelper.toJsonAPIMap("metadata", JsonAPITestHelper.toAttributeMap(osMetadata), null, null)).extract().body().jsonPath().get("data.id");
85+
metadataUuid = JsonAPITestHelper.extractId(sendPost("metadata",
86+
JsonAPITestHelper.toJsonAPIMap("metadata", JsonAPITestHelper.toAttributeMap(osMetadata), null, null)));
8987

9088
DerivativeDto derivative = buildDerivativeDto(objectUpload_generatedFrom.getFileIdentifier());
9189

92-
derivativeUuid = sendPost(RESOURCE_UNDER_TEST, JsonAPITestHelper.toJsonAPIMap(
93-
RESOURCE_UNDER_TEST,
94-
JsonAPITestHelper.toAttributeMap(derivative),
95-
Map.of(
96-
"acDerivedFrom", getRelationshipType("metadata", metadataUuid)),
97-
null)).extract().body().jsonPath().get("data.id");
98-
90+
derivativeUuid = JsonAPITestHelper.extractId(sendPost(RESOURCE_UNDER_TEST, JsonAPITestHelper.toJsonAPIMap(
91+
DerivativeDto.TYPENAME,
92+
JsonAPITestHelper.toAttributeMap(derivative),
93+
JsonAPITestHelper.toRelationshipMap(
94+
JsonAPIRelationship.of("acDerivedFrom", "metadata", metadataUuid)
95+
),
96+
null)));
9997
}
10098

10199
/**
@@ -116,20 +114,15 @@ public void tearDown() {
116114
void derivative_SpecValid() {
117115
DerivativeDto derivativeDto = buildDerivativeDto(objectUpload.getFileIdentifier());
118116
OpenAPI3Assertions.assertRemoteSchema(OpenAPIConstants.OBJECT_STORE_API_SPECS_URL, SCHEMA_NAME,
117+
119118
sendPost(RESOURCE_UNDER_TEST, JsonAPITestHelper.toJsonAPIMap(
120119
RESOURCE_UNDER_TEST,
121120
JsonAPITestHelper.toAttributeMap(derivativeDto),
122-
Map.of(
123-
"acDerivedFrom", getRelationshipType("metadata", metadataUuid),
124-
"generatedFromDerivative", getRelationshipType("derivative", derivativeUuid)),
125-
null))
126-
.extract().asString());
127-
}
128-
129-
private Map<String, Object> getRelationshipType(String type, String uuid) {
130-
return Map.of("data", Map.of(
131-
"id", uuid,
132-
"type", type));
121+
JsonAPITestHelper.toRelationshipMap(
122+
List.of(
123+
JsonAPIRelationship.of("acDerivedFrom", "metadata", metadataUuid),
124+
JsonAPIRelationship.of("generatedFromDerivative", "derivative", derivativeUuid))),
125+
null)).extract().asString());
133126
}
134127

135128
private DerivativeDto buildDerivativeDto(UUID fileIdentifier) {

0 commit comments

Comments
 (0)