tests: foundry: improve and fix test

`--no-commit` is now the default behavior and the flag has been
removed. This also adds a new case to test with an `out` directory
that is different than the default.

Author: elopez

scripts/ci_test_foundry.sh

@@ -1,13 +1,14 @@
 #!/usr/bin/env bash
-
 ### Test foundry integration
+set -Eeuxo pipefail
 
 
-cd /tmp || exit 255
+## test 1 - same folder
 
+cd /tmp || exit 255
 mkdir forge_test
 cd forge_test || exit 255
-forge init --no-commit
+forge init
 
 crytic-compile .
 if [ $? -ne 0 ]
@@ -16,11 +17,35 @@ then
     exit 255
 fi
 
-mkdir /tmp/forge_test/test_2
-rsync -a --exclude='test_2' ./ /tmp/forge_test/test_2/
-crytic-compile ./test_2
+
+## test 2 - same folder, different out dir
+
+cd /tmp || exit 255
+mkdir forge_test2
+cd forge_test2 || exit 255
+forge init
+
+sed -i 's/^out\s*=.*$/out = "foobar"/' foundry.toml
+
+crytic-compile .
 if [ $? -ne 0 ]
 then
     echo "foundry test 2 failed"
     exit 255
-fi
+fi
+
+## test 3 - different folder
+
+cd /tmp || exit 255
+mkdir forge_test3
+cd forge_test3 || exit 255
+forge init
+
+cd /tmp || exit 255
+
+crytic-compile ./forge_test3
+if [ $? -ne 0 ]
+then
+    echo "foundry test 3 failed"
+    exit 255
+fi