Skip to content

Commit 2f37b65

Browse files
committed
pr comments - renamed basketitem to basket
1 parent e63ea2e commit 2f37b65

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

src/main/java/uk/gov/companieshouse/orders/api/controller/BasketController.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,9 @@ public ResponseEntity<AddToBasketResponseDTO> addItemToBasket(final @Valid @Requ
5555
returnedBasket = basketService.saveBasket(mappedBasket);
5656
}
5757

58-
final AddToBasketResponseDTO basketItemDTO = mapper.basketToAddToBasketDTO(returnedBasket);
58+
final AddToBasketResponseDTO addToBasketResponseDTO = mapper.basketToAddToBasketDTO(returnedBasket);
5959
trace("EXITING addItemToBasket() with " + addToBasketRequestDTO, requestId);
60-
return ResponseEntity.status(HttpStatus.OK).body(basketItemDTO);
60+
return ResponseEntity.status(HttpStatus.OK).body(addToBasketResponseDTO);
6161
}
6262

6363
/**

src/test/java/uk/gov/companieshouse/orders/api/mapper/BasketMapperTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ static class Config {}
3636
private BasketMapper basketMapper;
3737

3838
@Test
39-
public void testAddToBasketItemDTOToBasket(){
39+
public void testAddToBasketRequestDTOToBasket(){
4040
final AddToBasketRequestDTO dto = new AddToBasketRequestDTO();
4141
dto.setItemUri(ITEM_URI);
4242

@@ -47,14 +47,14 @@ public void testAddToBasketItemDTOToBasket(){
4747
}
4848

4949
@Test
50-
public void testBasketItemToBasketItemDTO() {
50+
public void testBasketToAddToBasketResponseDTO() {
5151
Basket basket = new Basket();
5252
Item item = new Item();
5353
item.setItemUri(ITEM_URI);
5454
basket.getData().setItems(Arrays.asList(item));
5555

56-
AddToBasketResponseDTO basketItemDTO = basketMapper.basketToAddToBasketDTO(basket);
56+
AddToBasketResponseDTO addToBasketResponseDTO = basketMapper.basketToAddToBasketDTO(basket);
5757

58-
assertEquals(ITEM_URI, basketItemDTO.getItemUri());
58+
assertEquals(ITEM_URI, addToBasketResponseDTO.getItemUri());
5959
}
6060
}

src/test/java/uk/gov/companieshouse/orders/api/service/BasketServiceTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public class BasketServiceTest {
2828
private static final LocalDateTime CREATED_AT = LocalDateTime.of(2020, 01, 12, 9, 1);
2929

3030
@Test
31-
public void saveBasketItemPopulatesCreatedAtAndUpdatedAtAndSavesItem() {
31+
public void saveBasketPopulatesCreatedAtAndUpdatedAtAndSavesItem() {
3232
final Basket basket = new Basket();
3333
basket.setId(ERIC_IDENTITY_VALUE);
3434

@@ -49,7 +49,7 @@ public void saveBasketItemPopulatesCreatedAtAndUpdatedAtAndSavesItem() {
4949
}
5050

5151
@Test
52-
public void saveBasketItemPopulatesUpdatedAtAndSavesItem() {
52+
public void saveBasketPopulatesUpdatedAtAndSavesItem() {
5353
final Basket basket = new Basket();
5454
basket.setCreatedAt(CREATED_AT);
5555
basket.setId(ERIC_IDENTITY_VALUE);

0 commit comments

Comments
 (0)