#4455 - 'Update sequence' modal window: display the number of nucleot…

…ides to be changed
epam · Apr 12, 2024 · f9598ee · f9598ee
1 parent 670ef26
commit f9598ee
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/...cules/src/components/modal/UpdateSequenceInRNABuilder/UpdateSequenceInRNABuilder.test.tsx b/...cules/src/components/modal/UpdateSequenceInRNABuilder/UpdateSequenceInRNABuilder.test.tsx
@@ -38,7 +38,7 @@ describe('UpdateSequenceInRNABuilder modal component', () => {
           <UpdateSequenceInRNABuilder {...mockProps} />,
           {
             rnaBuilder: {
-              sequenceSelection: [labeledNucleotide],
+              sequenceSelection: [labeledNucleotide, labeledNucleotide],
             },
           },
         ),

diff --git a/...omolecules/src/components/modal/UpdateSequenceInRNABuilder/UpdateSequenceInRNABuilder.tsx b/...omolecules/src/components/modal/UpdateSequenceInRNABuilder/UpdateSequenceInRNABuilder.tsx
@@ -39,6 +39,7 @@ const UpdateSequenceInRNABuilder = ({ isModalOpen, onClose }: Props) => {
   const dispatch = useAppDispatch();
   const sequenceSelection = useAppSelector(selectSequenceSelection);
   const editor = useAppSelector(selectEditor);
+  const countOfSequenceSelection = sequenceSelection.length;
   const onCloseCallback = useCallback(() => {
     onClose();
   }, [onClose]);
@@ -69,7 +70,8 @@ const UpdateSequenceInRNABuilder = ({ isModalOpen, onClose }: Props) => {
     >
       <Modal.Content>
         <TextWrapper>
-          You are going to make multiple changes in your sequence. Are you sure?
+          You are going to modify {countOfSequenceSelection} nucleotides. Are
+          you sure?
         </TextWrapper>
       </Modal.Content>
       <Modal.Footer>

diff --git a/...s/modal/UpdateSequenceInRNABuilder/__snapshots__/UpdateSequenceInRNABuilder.test.tsx.snap b/...s/modal/UpdateSequenceInRNABuilder/__snapshots__/UpdateSequenceInRNABuilder.test.tsx.snap
@@ -75,7 +75,9 @@ Object {
             <div
               class="css-pp7hf1"
             >
-              You are going to make multiple changes in your sequence. Are you sure?
+              You are going to modify 
+              2
+               nucleotides. Are you sure?
             </div>
           </div>
           <div