Skip to content

Commit 0e59899

Browse files
Rob TjalmaRob Tjalma
authored andcommitted
Added Filename field to Item
Signed-off-by: Rob Tjalma <[email protected]>
1 parent 804aee1 commit 0e59899

File tree

4 files changed

+29
-5
lines changed

4 files changed

+29
-5
lines changed

app/src/test/java/org/lfenergy/compas/scl/data/rest/v1/CompasSclDataResourceTest.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,11 @@ class CompasSclDataResourceTest {
3737
void list_WhenCalled_ThenItemResponseRetrieved() {
3838
var type = SclType.SCD;
3939
var uuid = UUID.randomUUID();
40+
var filename = "Filename";
4041
var version = "1.0.0";
4142

42-
when(compasSclDataService.list(type)).thenReturn(Collections.singletonList(new Item(uuid.toString(), version)));
43+
when(compasSclDataService.list(type))
44+
.thenReturn(Collections.singletonList(new Item(uuid.toString(), filename, version)));
4345

4446
var response = given()
4547
.pathParam(TYPE_PATH_PARAM, type)
@@ -51,6 +53,7 @@ void list_WhenCalled_ThenItemResponseRetrieved() {
5153

5254
var xmlPath = response.xmlPath();
5355
assertEquals(uuid.toString(), xmlPath.get("ListResponse.Item[0].Id"));
56+
assertEquals(filename, xmlPath.get("ListResponse.Item[0].Filename"));
5457
assertEquals(version, xmlPath.get("ListResponse.Item[0].Version"));
5558
verify(compasSclDataService, times(1)).list(type);
5659
}
@@ -59,9 +62,11 @@ void list_WhenCalled_ThenItemResponseRetrieved() {
5962
void listVersionsByUUID_WhenCalled_ThenItemResponseRetrieved() {
6063
var type = SclType.SCD;
6164
var uuid = UUID.randomUUID();
65+
var filename = "Filename";
6266
var version = "1.0.0";
6367

64-
when(compasSclDataService.listVersionsByUUID(type, uuid)).thenReturn(Collections.singletonList(new Item(uuid.toString(), version)));
68+
when(compasSclDataService.listVersionsByUUID(type, uuid))
69+
.thenReturn(Collections.singletonList(new Item(uuid.toString(), filename, version)));
6570

6671
var response = given()
6772
.pathParam(TYPE_PATH_PARAM, type)
@@ -74,6 +79,7 @@ void listVersionsByUUID_WhenCalled_ThenItemResponseRetrieved() {
7479

7580
var xmlPath = response.xmlPath();
7681
assertEquals(uuid.toString(), xmlPath.get("ListResponse.Item[0].Id"));
82+
assertEquals(filename, xmlPath.get("ListResponse.Item[0].Filename"));
7783
assertEquals(version, xmlPath.get("ListResponse.Item[0].Version"));
7884
verify(compasSclDataService, times(1)).listVersionsByUUID(type, uuid);
7985
}

repository/src/main/java/org/lfenergy/compas/scl/data/model/Item.java

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,11 @@ public class Item {
1616
required = true
1717
)
1818
private String id;
19+
@XmlElement(
20+
name = "Filename",
21+
required = true
22+
)
23+
private String filename;
1924
@XmlElement(
2025
name = "Version",
2126
required = true
@@ -25,8 +30,9 @@ public class Item {
2530
public Item() {
2631
}
2732

28-
public Item(String id, String version) {
33+
public Item(String id, String filename, String version) {
2934
this.id = id;
35+
this.filename = filename;
3036
this.version = version;
3137
}
3238

@@ -38,6 +44,14 @@ public void setId(String id) {
3844
this.id = id;
3945
}
4046

47+
public String getFilename() {
48+
return filename;
49+
}
50+
51+
public void setFilename(String filename) {
52+
this.filename = filename;
53+
}
54+
4155
public String getVersion() {
4256
return version;
4357
}

repository/src/test/java/org/lfenergy/compas/scl/data/model/ItemTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,13 @@ protected Class<?> getClassToBeTested() {
1818
@Test
1919
void constructor_WhenCalledWithParameters_ThenValuesAreFilled() {
2020
var id = UUID.randomUUID().toString();
21+
var filename = "Filename";
2122
var version = "1.0.0";
2223

23-
var item = new Item(id, version);
24+
var item = new Item(id, filename, version);
2425

2526
assertEquals(id, item.getId());
27+
assertEquals(filename, item.getFilename());
2628
assertEquals(version, item.getVersion());
2729
}
2830
}

repository/src/test/java/org/lfenergy/compas/scl/data/util/JaxbMarshallerTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,16 @@ class JaxbMarshallerTest {
1515
@Test
1616
void unmarshal_WhenItemXMLPassed_ThenItemObjectReturned() throws JAXBException {
1717
var id = "ID";
18+
var filename = "FILENAME";
1819
var version = "VERSION";
19-
var xml = "<Item><Id>" + id + "</Id><Version>" + version + "</Version></Item>";
20+
var xml = "<Item><Id>" + id + "</Id><Filename>" + filename + "</Filename><Version>" + version + "</Version></Item>";
2021

2122
JaxbMarshaller marshaller = new JaxbMarshaller();
2223
Item item = marshaller.unmarshal(xml);
2324

2425
assertNotNull(item);
2526
assertEquals(id, item.getId());
27+
assertEquals(filename, item.getFilename());
2628
assertEquals(version, item.getVersion());
2729
}
2830
}

0 commit comments

Comments
 (0)