mutation test speedup - skip unused function per build

commaai · Jun 7, 2024 · ee1eeb3 · ee1eeb3
1 parent 5518085
commit ee1eeb3
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 14 deletions.
diff --git a/tests/misra/test_misra.sh b/tests/misra/test_misra.sh
@@ -71,20 +71,23 @@ cppcheck  --project=$PANDA_DIR/compile_commands.json \
 
 
 # Test individual targets for unused functions
-printf "\n${GREEN}** Check unused functions per build - Pandas **${NC}\n"
-cppcheck  --project=$PANDA_DIR/compile_commands.json \
-          --enable=unusedFunction \
-          -i$PANDA_DIR/board/bootstub.c \
-          -i$PANDA_DIR/board/jungle/ \
-          -i$PANDA_DIR/crypto/
-printf "\n${GREEN}Success!${NC} took $SECONDS seconds\n"
+if [ -z "$SKIP_PER_BUILD_CHECKS" ]; then
+  printf "\n${GREEN}** Check unused functions per build - Pandas **${NC}\n"
+  cppcheck  --project=$PANDA_DIR/compile_commands.json \
+            --enable=unusedFunction \
+            -i$PANDA_DIR/board/bootstub.c \
+            -i$PANDA_DIR/board/jungle/ \
+            -i$PANDA_DIR/crypto/
 
-printf "\n${GREEN}** Check unused functions per build - Bootstub **${NC}\n"
-cppcheck  --project=$PANDA_DIR/compile_commands.json \
-          --enable=unusedFunction \
-          -i$PANDA_DIR/board/main.c \
-          -i$PANDA_DIR/board/jungle/ -UPANDA_JUNGLE \
-          -i$PANDA_DIR/crypto/
+  printf "\n${GREEN}** Check unused functions per build - Bootstub **${NC}\n"
+  cppcheck  --project=$PANDA_DIR/compile_commands.json \
+            --enable=unusedFunction \
+            -i$PANDA_DIR/board/main.c \
+            -i$PANDA_DIR/board/jungle/ -UPANDA_JUNGLE \
+            -i$PANDA_DIR/crypto/
+fi
+
+printf "\n${GREEN}Success!${NC} took $SECONDS seconds\n"
 
 # ensure list of checkers is up to date
 cd $DIR

diff --git a/tests/misra/test_mutation.py b/tests/misra/test_mutation.py
@@ -84,6 +84,6 @@ def test_misra_mutation(fn, patch, should_fail):
       assert r == 0
 
     # run test
-    r = subprocess.run("SKIP_TABLES_DIFF=1 tests/misra/test_misra.sh", cwd=tmp, shell=True)
+    r = subprocess.run("SKIP_TABLES_DIFF=1 SKIP_PER_BUILD_CHECKS=1 tests/misra/test_misra.sh", cwd=tmp, shell=True)
     failed = r.returncode != 0
     assert failed == should_fail