Skip to content

Commit b580e65

Browse files
[SELC-6390] fix: missing tokenId in OnboardedProductResponse (#57)
1 parent 37047dd commit b580e65

File tree

4 files changed

+24
-1
lines changed

4 files changed

+24
-1
lines changed

apps/institution-ms/app/src/main/docs/openapi.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3528,6 +3528,9 @@
35283528
"type" : "string",
35293529
"enum" : [ "ACTIVE", "DELETED", "PENDING", "REJECTED", "SUSPENDED", "TOBEVALIDATED" ]
35303530
},
3531+
"tokenId" : {
3532+
"type" : "string"
3533+
},
35313534
"updatedAt" : {
35323535
"type" : "string",
35333536
"format" : "date-time"

apps/institution-ms/web/src/main/java/it/pagopa/selfcare/mscore/web/model/onboarding/OnboardedProductResponse.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
@NoArgsConstructor
1414
public class OnboardedProductResponse {
1515
private String productId;
16+
private String tokenId;
1617
private RelationshipState status;
1718
private BillingResponse billing;
1819
private OffsetDateTime createdAt;

apps/institution-ms/web/src/test/java/it/pagopa/selfcare/mscore/web/controller/InstitutionControllerTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ void testRetrieveInstitutionById() throws Exception {
289289
.perform(requestBuilder)
290290
.andExpect(MockMvcResultMatchers.status().isOk())
291291
.andExpect(MockMvcResultMatchers.content().contentType("application/json"))
292-
.andExpect(MockMvcResultMatchers.content().string("{\"id\":\"42\",\"description\":\"description\",\"institutionType\":\"PG\",\"istatCode\":\"istatCode\",\"attributes\":[{\"origin\":null,\"code\":\"code\",\"description\":\"description\"}],\"onboarding\":[{\"productId\":\"example\",\"status\":\"ACTIVE\",\"billing\":{\"vatNumber\":\"example\",\"taxCodeInvoicing\":\"example\",\"recipientCode\":\"example\",\"publicServices\":false},\"createdAt\":null,\"updatedAt\":null,\"isAggregator\":true}],\"imported\":false,\"delegation\":false}"));
292+
.andExpect(MockMvcResultMatchers.content().string("{\"id\":\"42\",\"description\":\"description\",\"institutionType\":\"PG\",\"istatCode\":\"istatCode\",\"attributes\":[{\"origin\":null,\"code\":\"code\",\"description\":\"description\"}],\"onboarding\":[{\"productId\":\"example\",\"tokenId\":\"tokenId\",\"status\":\"ACTIVE\",\"billing\":{\"vatNumber\":\"example\",\"taxCodeInvoicing\":\"example\",\"recipientCode\":\"example\",\"publicServices\":false},\"createdAt\":null,\"updatedAt\":null,\"isAggregator\":true}],\"imported\":false,\"delegation\":false}"));
293293
}
294294

295295
@Test
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
package it.pagopa.selfcare.institution.service;
2+
3+
import io.quarkus.test.junit.QuarkusTest;
4+
import org.junit.jupiter.api.Test;
5+
6+
import static org.junit.jupiter.api.Assertions.assertTrue;
7+
8+
@QuarkusTest
9+
class SwaggerConfigTest {
10+
11+
@Test
12+
void swaggerSpringPlugin() {
13+
// Empty test required to avoid errors on Swagger Detect Rules and Conflict:
14+
// No tests matching pattern "SwaggerConfigTest#swaggerSpringPlugin"
15+
// TODO: consider to add -Dsurefire.failIfNoSpecifiedTests=false to the github action template
16+
assertTrue(true);
17+
}
18+
19+
}

0 commit comments

Comments
 (0)