test-rewatch: show actual error messages on "Error Cleaning/Building Repo" (#7609)

Author: cknitt

rewatch/tests/compile.sh

@@ -5,19 +5,23 @@ cd ../testrepo
 
 bold "Test: It should compile"
 
-if rewatch clean &> /dev/null;
+error_output=$(rewatch clean 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Cleaned"
 else
   error "Error Cleaning Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 
-if rewatch &> /dev/null;
+error_output=$(rewatch 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Built"
 else
   error "Error Building Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 

rewatch/tests/lock.sh

@@ -5,15 +5,17 @@ bold "Test: It should lock - when watching"
 
 sleep 1
 
-if rewatch clean &> /dev/null;
+error_output=$(rewatch clean 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Cleaned"
-else 
+else
   error "Error Cleaning Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 
-exit_watcher() { 
+exit_watcher() {
   # kill watcher by removing lock file
   rm lib/rewatch.lock
 }
@@ -23,11 +25,11 @@ success "Watcher Started"
 
 sleep 2
 
-if rewatch build | grep 'Could not start Rewatch:' &> /dev/null; 
+if rewatch build | grep 'Could not start Rewatch:' &> /dev/null;
 then
   success "Lock is correctly set"
   exit_watcher
-else 
+else
   error "Not setting lock correctly"
   exit_watcher
   exit 1
@@ -41,7 +43,7 @@ success "Watcher Started"
 
 sleep 2
 
-if cat tmp.txt | grep 'Could not start Rewatch:' &> /dev/null; 
+if cat tmp.txt | grep 'Could not start Rewatch:' &> /dev/null;
 then
   error "Lock not removed correctly"
   exit_watcher

rewatch/tests/suffix.sh

@@ -5,22 +5,26 @@ bold "Test: It should support custom suffixes"
 
 # Clean Repo
 sleep 1
-if rewatch clean &> /dev/null;
+error_output=$(rewatch clean 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Cleaned"
 else
   error "Error Cleaning Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 
 # Replace suffix
 replace "s/.mjs/.res.js/g" bsconfig.json
 
-if rewatch build &> /dev/null;
+error_output=$(rewatch build 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Built"
 else
-  error "Error building repo"
+  error "Error Building Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 
@@ -35,22 +39,26 @@ else
   exit 1
 fi
 
-if rewatch clean &> /dev/null;
+error_output=$(rewatch clean 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Cleaned"
 else
   error "Error Cleaning Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 
 # Restore Suffix
 replace "s/.res.js/.mjs/g" bsconfig.json
 
 # Restore original build
-if rewatch build &> /dev/null;
+error_output=$(rewatch build 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Built"
 else
-  error "Error building repo"
+  error "Error Building Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi

rewatch/tests/watch.sh

@@ -3,15 +3,17 @@ cd ../testrepo
 
 bold "Test: It should watch"
 
-if rewatch clean &> /dev/null;
+error_output=$(rewatch clean 2>&1)
+if [ $? -eq 0 ];
 then
   success "Repo Cleaned"
-else 
+else
   error "Error Cleaning Repo"
+  printf "%s\n" "$error_output" >&2
   exit 1
 fi
 
-exit_watcher() { 
+exit_watcher() {
   # kill watcher by removing lock file
   rm lib/rewatch.lock
 }
@@ -23,10 +25,10 @@ echo 'Js.log("added-by-test")' >> ./packages/main/src/Main.res
 
 sleep 2
 
-if node ./packages/main/src/Main.mjs | grep 'added-by-test' &> /dev/null; 
+if node ./packages/main/src/Main.mjs | grep 'added-by-test' &> /dev/null;
 then
   success "Output is correct"
-else 
+else
   error "Output is incorrect"
   exit_watcher
   exit 1
@@ -38,10 +40,10 @@ replace '/Js.log("added-by-test")/d' ./packages/main/src/Main.res;
 
 sleep 1
 
-if git diff --exit-code ./  
+if git diff --exit-code ./
 then
   success "Adding and removing changes nothing"
-else 
+else
   error "Adding and removing changes left some artifacts"
   exit_watcher
   exit 1