Test script runs Brakeman

[dxw use Brakeman as a static code scanner](https://github.com/dxw/tech-team-rfcs/blob/main/rfc-024-use-brakeman.md). Run this at the end of the order as it takes a while. Usually we'd like quicker rspec feedback when working with known fixing failing tests. Ignore `coverage/` from linting checks in case Brakeman is run without being output to stdout
dxw · Sep 27, 2021 · 6f11f0d · 6f11f0d
1 parent a6c0fa6
commit 6f11f0d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 0 deletions.
diff --git a/.eslintignore b/.eslintignore
@@ -2,3 +2,4 @@
 /tmp/
 /vendor/
 /public/assets
+/coverage
diff --git a/.prettierignore b/.prettierignore
@@ -25,3 +25,4 @@
 *.txt
 *.gz
 /public/assets/*
+/coverage
diff --git a/script/test b/script/test
@@ -49,4 +49,7 @@ else
 
   echo "==> Running the tests..."
   bundle exec rspec
+
+  echo "==> Running Brakeman"
+  bundle exec brakeman -o /dev/stdout
 fi
-Original file line number
+Diff line change
@@ Expand Up / @@ -25,3 +25,4 @@ @@
     *.txt
     *.gz
     /public/assets/*
+    /coverage