From 0408d7f75ee6079ef1e4feaa498c55caf3f2033a Mon Sep 17 00:00:00 2001
From: Mira Leung <miraleung@google.com>
Date: Thu, 19 Nov 2020 12:02:38 -0800
Subject: [PATCH] build: speed up git precommit checks

---
 .githooks/pre-commit | 89 +-------------------------------------------
 1 file changed, 1 insertion(+), 88 deletions(-)

diff --git a/.githooks/pre-commit b/.githooks/pre-commit
index a3acd78d97..04b029dee6 100755
--- a/.githooks/pre-commit
+++ b/.githooks/pre-commit
@@ -92,98 +92,11 @@ then
   fi
 fi
 
-# Check that everything builds, but only if there are Java or Bazel changes.
-if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] || [ $NUM_BAZEL_FILES_CHANGED -gt 0 ]
-then
-  echo_status "Checking the build..."
-  bazel --batch build --disk_cache="$BAZEL_CACHE_DIR" //src/...
-  BUILD_STATUS=$?
-  if [ $BUILD_STATUS != 0 ]
-  then
-    echo_error "Build failed." "Please fix it and try again."
-  fi
-fi
-
 # Check unit tests.
 if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] || [ $NUM_UNIT_GOLDEN_FILES_CHANGED -gt 0 ]
 then
   echo_status "Checking unit tests..."
-  bazel --batch test --disk_cache="$BAZEL_CACHE_DIR" //src/test/...
-  TEST_STATUS=$?
-  if [ $TEST_STATUS != 0 ]
-  then
-    echo_error "Unit tests failed." "Please fix them and try again."
-    exit 1
-  fi
-fi
-
-# Check integration tests.
-if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_GOLDEN_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_BAZEL_FILES_CHANGED -gt 0 ]
-then
-  echo_status "Checking integration tests..."
-  bazel --batch test --disk_cache="$BAZEL_CACHE_DIR" //test/integration/...
-  TEST_STATUS=$?
-  if [ $TEST_STATUS != 0 ]
-  then
-    echo_error "Integration tests failed." "Please fix them and try again."
-    exit 1
-  fi
-fi
-
-# Check integration tests.
-if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_GOLDEN_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_BAZEL_FILES_CHANGED -gt 0 ]
-then
-  echo_status "Checking integration tests..."
-  bazel --batch test --disk_cache="$BAZEL_CACHE_DIR" //test/integration/...
-  TEST_STATUS=$?
-  if [ $TEST_STATUS != 0 ]
-  then
-    echo_error "Tests failed." "Please fix them and try again."
-    exit 1
-  fi
-fi
-
-# Check integration tests.
-if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_GOLDEN_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_BAZEL_FILES_CHANGED -gt 0 ]
-then
-  echo_status "Checking integration tests..."
-  bazel --batch test --disk_cache="$BAZEL_CACHE_DIR" //test/integration/...
-  TEST_STATUS=$?
-  if [ $TEST_STATUS != 0 ]
-  then
-    echo_error "Tests failed." "Please fix them and try again."
-    exit 1
-  fi
-fi
-
-# Check integration tests.
-if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_GOLDEN_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_BAZEL_FILES_CHANGED -gt 0 ]
-then
-  echo_status "Checking integration tests..."
-  bazel --batch test --disk_cache="$BAZEL_CACHE_DIR" //test/integration/...
-  TEST_STATUS=$?
-  if [ $TEST_STATUS != 0 ]
-  then
-    echo_error "Tests failed." "Please fix them and try again."
-    exit 1
-  fi
-fi
-
-# Check integration tests.
-if [ $NUM_JAVA_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_GOLDEN_FILES_CHANGED -gt 0 ] \
-  || [ $NUM_INTEGRATION_BAZEL_FILES_CHANGED -gt 0 ]
-then
-  echo_status "Checking integration tests..."
-  bazel --batch test --disk_cache="$BAZEL_CACHE_DIR" //test/integration/...
+  bazel --batch test $(bazel query "src/test/..." | grep "Test$") --disk_cache="$BAZEL_CACHE_DIR"
   TEST_STATUS=$?
   if [ $TEST_STATUS != 0 ]
   then