X1244 number stored items

cypress/e2e/pages/store.cy.ts

@@ -16,12 +16,12 @@ describe('Store', () => {
     context('when the location is found', () => {
       before(() => {
         cy.visit('/store');
-        cy.findByLabelText('Find Location:').should('be.visible').type('STO-001F{enter}');
+        cy.findByLabelText('Find Location:').should('be.visible').type('STO-001{enter}');
       });
 
       it("takes you to that location's page", () => {
-        cy.location('pathname').should('eq', '/locations/STO-001F');
-        cy.findByText('Location STO-001F could not be found').should('not.exist');
+        cy.location('pathname').should('eq', '/locations/STO-001');
+        cy.findByText('Location STO-001 could not be found').should('not.exist');
       });
     });
 

src/graphql/fragments/LinkedLocationFields.graphql

@@ -3,4 +3,6 @@ fragment LinkedLocationFields on LinkedLocation {
   fixedName
   customName
   address
+  numStored
+  leaf
 }

src/graphql/fragments/LocationFields.graphql

@@ -4,6 +4,8 @@ fragment LocationFields on Location {
   customName
   address
   direction
+  numStored
+  leaf
   parent {
     barcode
     fixedName
@@ -18,9 +20,6 @@ fragment LocationFields on Location {
     address
   }
   children {
-    barcode
-    fixedName
-    customName
-    address
+    ...LinkedLocationFields
   }
 }

src/lib/factories/locationFactory.ts

@@ -17,7 +17,9 @@ const locationFactory = Factory.define<Location>(({ sequence, params, afterBuild
     size: null,
     direction: params.direction ?? null,
     parent: params.parent == null ? null : (params.parent as LinkedLocation),
-    qualifiedNameWithFirstBarcode: params.qualifiedNameWithFirstBarcode ?? `FakeParent / ${barcode}`
+    qualifiedNameWithFirstBarcode: params.qualifiedNameWithFirstBarcode ?? `FakeParent / ${barcode}`,
+    numStored: params.numStored ?? 0,
+    leaf: params.leaf ?? false
   };
 
   if (params.size) {
@@ -55,6 +57,8 @@ export function buildLinkedLocation(location: Location): LinkedLocation {
     barcode: location.barcode,
     fixedName: location.fixedName,
     customName: location.customName,
-    address: location.address
+    address: location.address,
+    numStored: location.numStored,
+    leaf: location.leaf ?? false
   };
 }

src/mocks/handlers/locationHandlers.ts

@@ -170,7 +170,9 @@ function addLocationHierarchy(barcode: string, linkedLocation: LinkedLocationFie
     barcode: location.barcode,
     address: location.address,
     customName: location.customName,
-    fixedName: location.fixedName
+    fixedName: location.fixedName,
+    numStored: location.numStored,
+    leaf: location.leaf
   });
   if (location.parent) {
     addLocationHierarchy(location.parent.barcode, linkedLocation);
@@ -185,6 +187,8 @@ export function locationResponse(location: Location): LocationFieldsFragment {
     customName: location.customName,
     address: location.address,
     direction: location.direction,
+    numStored: location.numStored,
+    leaf: location.leaf,
     parent: location.parent
       ? {
           __typename: 'LinkedLocation',
@@ -211,7 +215,9 @@ export function locationResponse(location: Location): LocationFieldsFragment {
         barcode: child.barcode,
         fixedName: child.fixedName,
         customName: child.customName,
-        address: child.address
+        address: child.address,
+        numStored: child.numStored,
+        leaf: child.leaf
       };
     })
   };

src/mocks/repositories/locationRepository.ts

@@ -106,7 +106,7 @@ class LocationRepository implements Repository<Location> {
 
 const locationRepository = new LocationRepository();
 
-const room: Location = locationFactory.build({ customName: 'Room 1234' });
+const room: Location = locationFactory.build({ customName: 'Room 1234', leaf: true });
 
 const freezers: Location[] = [];
 
@@ -143,7 +143,8 @@ racks.forEach((rack, index) => {
     locationFactory.build({
       customName: `Box 1 in ${rack.customName}`,
       direction: GridDirection.RightDown,
-      parent: buildLinkedLocation(rack)
+      parent: buildLinkedLocation(rack),
+      leaf: true
     })
   );
   /***

src/pages/Location.tsx

@@ -504,6 +504,11 @@ const Location = () => {
                             >
                               {child.customName ?? child.fixedName ?? child.barcode}
                             </StyledLink>
+                            {child.leaf && (
+                              <span className="font-semibold" data-testid={'storedItemsCount'}>
+                                {` - Number of Stored items : ${child.numStored}`}
+                              </span>
+                            )}
                           </li>
                         );
                       })}

tests/e2e/pages/location.spec.tsx

@@ -8,6 +8,7 @@ import { locationResponse } from '../../../src/mocks/handlers/locationHandlers';
 import { enableMapSet } from 'immer';
 import React from 'react';
 import '@testing-library/jest-dom';
+
 jest.mock('../../../src/pages/location/ItemsGrid', () => {
   return {
     __esModule: true,
@@ -204,6 +205,12 @@ describe('Load location with children ', () => {
       });
       expect(screen.queryByText('Stored Items')).not.toBeInTheDocument();
     });
+    it('display number of stored items', () => {
+      act(() => {
+        renderLocation('STO-005');
+      });
+      expect(screen.queryAllByTestId('storedItemsCount').length).toBeGreaterThan(0);
+    });
   });
 });
 

tests/unit/pages/location/itemsGrid.spec.tsx

@@ -24,7 +24,9 @@ const mocklocationParentContextValue: LocationParentContextType = {
       numRows: 2,
       numColumns: 2
     },
-    children: []
+    children: [],
+    numStored: 0,
+    leaf: false
   },
   addressToItemMap: new Map([
     ['A1', { barcode: 'STAN-3111' }],

tests/unit/pages/location/itemsList.spec.tsx

@@ -23,6 +23,8 @@ const mocklocationParentContextValue: LocationParentContextType = {
       numRows: 2,
       numColumns: 2
     },
+    numStored: 0,
+    leaf: false,
     children: []
   },
   addressToItemMap: new Map([