chore(test): improve diff detection for unit tests

daniel-sanche · daniel-sanche · commit e1e04d78fa77 · 2026-08-04T14:57:40.000-07:00
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -57,7 +57,7 @@ jobs:
         TEST_ALL_PACKAGES: ${{ steps.check-label.outputs.is_full_run }}
       run: |
         if [ -n "$TARGET_BRANCH" ]; then
-          git fetch origin "$TARGET_BRANCH" --depth=1 || true
+          git fetch origin "$TARGET_BRANCH" --deepen=200 || true
         fi
         python3 ci/get_package_shards.py
 
diff --git a/.github/workflows/unittest.yml b/.github/workflows/unittest.yml
@@ -72,7 +72,7 @@ jobs:
         PACKAGE_WEIGHTS: ${{ env.PACKAGE_WEIGHTS }}
       run: |
         if [ -n "$TARGET_BRANCH" ]; then
-          git fetch origin "$TARGET_BRANCH" --depth=1 || true
+          git fetch origin "$TARGET_BRANCH" --deepen=200 || true
         fi
         python3 ci/get_package_shards.py
 
@@ -168,8 +168,8 @@ jobs:
           echo "should_evaluate_coverage=true" >> "$GITHUB_OUTPUT"
         else
           TARGET_BRANCH="${TARGET_BRANCH:-main}"
-          git fetch origin "$TARGET_BRANCH" --depth=1 || true
-          num_files_changed=$(git diff --name-only "origin/${TARGET_BRANCH}" -- ${PACKAGE_DIRS} | wc -l | tr -d ' ')
+          git fetch origin "$TARGET_BRANCH" --deepen=200 || true
+          num_files_changed=$(git diff --name-only "origin/${TARGET_BRANCH}..." -- ${PACKAGE_DIRS} | wc -l | tr -d ' ')
           if [[ "${num_files_changed}" -gt 0 ]]; then
             echo "should_evaluate_coverage=true" >> "$GITHUB_OUTPUT"
           else
diff --git a/ci/get_package_shards.py b/ci/get_package_shards.py
@@ -108,7 +108,7 @@ def get_packages_to_test():
         return all_packages
 
     if build_type == 'presubmit':
-        git_diff_arg = f"origin/{target_branch}"
+        git_diff_arg = f"origin/{target_branch}..."
     elif build_type == 'continuous':
         git_diff_arg = "HEAD~1.."
     else:
diff --git a/ci/report_coverage.sh b/ci/report_coverage.sh
@@ -45,7 +45,7 @@ if [[ "${TEST_ALL_PACKAGES}" == "true" ]]; then
     modified_packages=$(for dir in ${PACKAGE_DIRS}; do ls -d ${dir}/*/ 2>/dev/null; done | cut -d/ -f1,2 | sort -u)
 elif [[ "${BUILD_TYPE}" == "presubmit" ]]; then
     # Presubmit build: evaluate coverage only for packages modified relative to the target branch
-    modified_packages=$(git diff --name-only "origin/${TARGET_BRANCH}" -- ${PACKAGE_DIRS} 2>/dev/null | cut -d/ -f1,2 | sort -u)
+    modified_packages=$(git diff --name-only "origin/${TARGET_BRANCH}..." -- ${PACKAGE_DIRS} 2>/dev/null | cut -d/ -f1,2 | sort -u)
 else
     # Continuous build (post-merge on main): evaluate coverage for packages modified in the last commit
     modified_packages=$(git diff --name-only HEAD~1 -- ${PACKAGE_DIRS} 2>/dev/null | cut -d/ -f1,2 | sort -u)
diff --git a/ci/run_conditional_tests.sh b/ci/run_conditional_tests.sh
@@ -54,9 +54,9 @@ elif [[ ${BUILD_TYPE} == "presubmit" ]]; then
     # For presubmit build, we want to know the difference from the
     # common commit in the target branch.
     if [ -n "${TARGET_BRANCH}" ]; then
-        git fetch origin "${TARGET_BRANCH}" --depth=1 || true
+        git fetch origin "${TARGET_BRANCH}" --deepen=200 || true
     fi
-    GIT_DIFF_ARG="origin/${TARGET_BRANCH}"
+    GIT_DIFF_ARG="origin/${TARGET_BRANCH}..."
 
 elif [[ ${BUILD_TYPE} == "continuous" ]]; then
     # For continuous build, we want to know the difference in the last
