Fix/ppdsc 2854 flaky tests

scripts/percy_local.sh

@@ -14,12 +14,14 @@ if [[ $BRANCH_NAME == "main" ]]; then
     exit 1
   fi
   echo "Running on main, so baseline images will be updated"
+  read -p "Are you sure you want to update the baselines? (y/n)" updateBaselinesRes
+  if [[ "$updateBaselinesRes" != "y" ]]; then
+    exit 1
+  fi
 else
   export PERCY_BRANCH="local run by ${USERNAME}"
 fi
 
-echo "PERCY_BRANCH=$PERCY_BRANCH"
-
 # ask the user to input their API key
 read -p "Enter the API key for the 'newskit-$1' project (you can find this in the Percy UI): " apiKey
 

src/audio-player-composable/__tests__/audio-player-composable.test.tsx

@@ -1255,10 +1255,15 @@ describe('Audio Player Composable', () => {
         'audio-player-time-display',
       ) as HTMLParagraphElement;
 
+      fireEvent.durationChange(audioElement, {
+        target: {
+          duration: 60,
+        },
+      });
       const seekBar = getByTestId('audio-slider-track') as HTMLDivElement;
       fireEvent.timeUpdate(audioElement);
       expect(audioElement.currentTime).toEqual(50);
-      expect(audioTimeLabel.innerHTML).toEqual('00:50/00:00 ');
+      expect(audioTimeLabel.innerHTML).toEqual('00:50/01:00 ');
       expect(seekBar.getAttribute('values')).toEqual('50');
     });
   });

src/audio-player-composable/audio-player-composable.tsx

@@ -77,12 +77,12 @@ export const AudioPlayerComposable = ({
     // On render onTimeUpdate will be fired and initialTime will be set as a value for currentTime state.
     // I can't set this one to the setCurrentTime state directly as the audioElement time
     // will still be 0, currentTime will be overridden to 0 and the audio will start from 0
-    if (audioRef && audioRef.current) {
+    if (audioRef && audioRef.current && duration) {
       audioRef.current.currentTime = initialTime;
     }
     setCurrentTime(0);
     setDisplayDuration(0);
-  }, [src, initialTime]);
+  }, [src, initialTime, duration]);
 
   const {
     audioEvents,