reduce max attempts. remove merge-only flag

yext · Aug 4, 2022 · ed8f0bd · ed8f0bd
1 parent 2042fe6
commit ed8f0bd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/tests/scripts/combined-coverage.sh b/tests/scripts/combined-coverage.sh
@@ -1,14 +1,7 @@
 #!/bin/bash
 
-# Run unit test and visual coverage test by default.
-# Skip this step if merge-only flag is provided.
-echo $1
-if [ "$1" != "merge-only" ]
-then
-  echo 'running individual tests...'
-  npm run test:unit
-  npm run test:visual
-fi
+npm run test:unit
+npm run test:visual
 
 # merge
 mkdir -p coverage/merge

diff --git a/tests/scripts/visual-coverage.sh b/tests/scripts/visual-coverage.sh
@@ -5,7 +5,7 @@ JOB_ID=$(echo $!) #get the background job ID
 
 # wait for a locally served Storybook
 attempt_counter=0
-max_attempts=100
+max_attempts=30
 until $(curl --output /dev/null --silent --head http://localhost:6006)
 do
   if [ ${attempt_counter} -eq ${max_attempts} ];then