Merge pull request #765 from sanger/x1238-decoding-consumables-lot-nu…

…mber add a decoding consumables lot number to the Xenium Analyser operation
sanger · Sep 18, 2024 · a809d7b · a809d7b
2 parents 4316580 + 27cc770
commit a809d7b
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 6 deletions.
diff --git a/src/pages/XeniumAnalyser.tsx b/src/pages/XeniumAnalyser.tsx
@@ -53,6 +53,7 @@ type LabwareSamples = {
 type AnalyserLabwareForm = {
   labware: LabwareFlaggedFieldsFragment;
   workNumber: string;
+  decodingConsumablesLot?: string;
   position?: CassettePosition;
   samples: Array<SampleRoi>;
   analyserScanData?: AnalyserScanDataFieldsFragment;
@@ -178,7 +179,10 @@ const XeniumAnalyser = () => {
       .of(
         Yup.object().shape({
           workNumber: Yup.string().required().label('SGP Number'),
-          position: Yup.string().required().label('Cassette Position'),
+          position: Yup.string().required(),
+          decodingConsumablesLot: Yup.string()
+            .optional()
+            .matches(/^\d{6}$/, 'Consumables lot number should be a 6-digit number'),
           samples: Yup.array()
             .of(
               Yup.object().shape({
@@ -237,6 +241,7 @@ const XeniumAnalyser = () => {
                     labware,
                     workNumber: values.workNumberAll,
                     position: undefined,
+                    decodingConsumablesLot: undefined,
                     samples: [],
                     analyserScanData: res.analyserScanData
                   });
@@ -254,6 +259,7 @@ const XeniumAnalyser = () => {
                 labware,
                 workNumber: values.workNumberAll,
                 position: undefined,
+                decodingConsumablesLot: undefined,
                 samples: []
               });
               return { ...prev };
@@ -309,6 +315,7 @@ const XeniumAnalyser = () => {
                   return {
                     barcode: lw.labware.barcode,
                     workNumber: lw.workNumber,
+                    decodingConsumablesLot: lw.decodingConsumablesLot,
                     position: lw.position?.toLowerCase() === 'left' ? CassettePosition.Left : CassettePosition.Right,
                     samples: labwareSample
                       ? labwareSample.samples.map((sample) => {
@@ -434,11 +441,17 @@ const XeniumAnalyser = () => {
                           label={'SGP Number'}
                           name={'workNumberAll'}
                           dataTestId={'workNumberAll'}
-                          onWorkNumberChange={(workNumber) => {
-                            setFieldValue('workNumberAll', workNumber);
-                            values.labware.forEach((lw, index) =>
-                              setFieldValue(`labware.${index}.workNumber`, workNumber)
-                            );
+                          onWorkNumberChange={async (workNumber) => {
+                            await setValues((prev) => {
+                              return {
+                                ...prev,
+                                workNumberAll: workNumber,
+                                labware: prev.labware.map((lw) => ({
+                                  ...lw,
+                                  workNumber
+                                }))
+                              };
+                            });
                           }}
                           requiredField={false}
                         />
@@ -449,6 +462,7 @@ const XeniumAnalyser = () => {
                             <tr>
                               <TableHeader>Barcode</TableHeader>
                               <TableHeader>SGP Number</TableHeader>
+                              <TableHeader>Decoding consumables lot number</TableHeader>
                               <TableHeader>Cassette Position</TableHeader>
                               <TableHeader>Samples</TableHeader>
                             </tr>
@@ -469,6 +483,15 @@ const XeniumAnalyser = () => {
                                   />
                                   <FormikErrorMessage name={`labware.${lwIndex}.workNumber`} />
                                 </TableCell>
+                                <TableCell className={'w-20'}>
+                                  <div className={'flex flex-col'}>
+                                    <FormikInput
+                                      label={''}
+                                      name={`labware.${lwIndex}.decodingConsumablesLot`}
+                                      data-testid={`labware.${lwIndex}.decodingConsumablesLot`}
+                                    />
+                                  </div>
+                                </TableCell>
                                 <TableCell>
                                   <CustomReactSelect
                                     options={objectKeys(CassettePosition).map((val) => {

diff --git a/tests/unit/pages/lab/xeniumAnalyser.spec.tsx b/tests/unit/pages/lab/xeniumAnalyser.spec.tsx
@@ -157,6 +157,15 @@ describe('Xenium analyser', () => {
           });
         });
       });
+      describe('Decoding Consumables Lot Number', () => {
+        it('validates the number if specified', () => {
+          waitFor(async () => {
+            await userEvent.type(screen.getByTestId('labware.0.decodingConsumablesLot'), '12234567');
+            await userEvent.tab();
+            expect(screen.getByText('Consumables lot number should be a 6-digit number')).toBeVisible();
+          });
+        });
+      });
       describe('Save button enabling', () => {
         it('should enable Save button when all fields are filled in ', () => {
           waitFor(async () => {