fix: test_image.sh (#696)

algolia · Dec 8, 2022 · 3caa65b · 3caa65b
1 parent c5e497c
commit 3caa65b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/scripts/test_image.sh b/scripts/test_image.sh
@@ -19,6 +19,13 @@ done
 logs=$(docker logs renderscript_test 2>&1)
 echo $logs
 
+if echo $logs | grep -q '"svc":"brws","msg":"Ready"'; then
+  echo "Browser ready"
+else
+  echo "Browser not ready"
+  exit 1
+fi
+
 curl --silent --request POST \
   --url http://localhost:3000/render \
   --header 'Content-Type: application/json' \
@@ -34,17 +41,12 @@ curl --silent --request POST \
 logs=$(docker logs renderscript_test 2>&1)
 echo $logs
 
-launched=$(echo $logs | grep '"svc":"brws","msg":"Ready"')
-if [ -z "$launched" ]; then
-  echo "Not ready"
-  exit 1
-fi
-
-rendered=$(echo $logs | grep '"msg":"Done","data":')
-if [ -z "$rendered" ]; then
+if echo $logs | grep -q '"msg":"Done","data":'; then
+  echo "Rendered"
+else
   echo "Not rendered"
   exit 1
 fi
 
-echo "Ready"
+echo "Image OK"
 docker stop renderscript_test && docker rm renderscript_test
diff --git a/src/lib/browser/Browser.ts b/src/lib/browser/Browser.ts
@@ -76,7 +76,7 @@ export class Browser {
     });
 
     this.#ready = true;
-    log.info('Browser ready', { id: this.#id, browser: this.#engine });
+    log.info('Ready', { id: this.#id, browser: this.#engine });
   }
 
   async stop(): Promise<void> {