From 421ce77be7bf55fa969cd381deda2e970caad593 Mon Sep 17 00:00:00 2001 From: Chris Raible Date: Mon, 31 Mar 2025 21:11:30 -0700 Subject: [PATCH] Reverted change to Tinybird `exec_test` script (#22731) no issue - This commit reverts a change to our Tinybird `exec_test` script, which added an exclusion to the sanity check warnings if the test included `timezone` in its file name - The warnings were fixed by renaming the timezone tests so they don't start with `all`, and therefore this change to the `exec_test` script isn't necessary --- ghost/web-analytics/scripts/exec_test.sh | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/ghost/web-analytics/scripts/exec_test.sh b/ghost/web-analytics/scripts/exec_test.sh index f6f3a57e26..a3ebc0e5fe 100755 --- a/ghost/web-analytics/scripts/exec_test.sh +++ b/ghost/web-analytics/scripts/exec_test.sh @@ -15,9 +15,8 @@ check_sum() { local file=$1 local expected_count=$2 - # Skip check if file starts with "all_" AND contains "timezone" in the name - # TODO: we should handle this for all tests and in a better way - if [[ ! $(basename "$file") =~ ^all_ ]] || [[ $(basename "$file") =~ timezone ]]; then + # Only perform the check if the file starts with "all_" + if [[ ! $(basename "$file") =~ ^all_ ]]; then return 0 fi