thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-11 17:02:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Relax check-style condition because in SQL-based tests there's no $"

Browse Source 
This reverts commit 73e56206e2.
This commit is contained in:
Pablo Marcos 2024-04-02 16:48:08 +02:00
parent 73e56206e2
commit 4a65a06600
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
utils/check-style/check-style
View File

@ -168,7 +168,7 @@ tests_with_query_log=( $(
) )
for test_case in "${tests_with_query_log[@]}"; do
grep -qE current_database.*currentDatabase "$test_case" || {
grep -qE 'current_database.*CLICKHOUSE_DATABASE' "$test_case"
grep -qE 'current_database.*\$CLICKHOUSE_DATABASE' "$test_case"
} || echo "Queries to system.query_log/system.query_thread_log does not have current_database = currentDatabase() condition in $test_case"
done
@ -206,12 +206,12 @@ tests_with_database_column=( $(
) )
for test_case in "${tests_with_database_column[@]}"; do
grep -qE database.*currentDatabase "$test_case" || {
grep -qE 'database.*CLICKHOUSE_DATABASE' "$test_case"
grep -qE 'database.*\$CLICKHOUSE_DATABASE' "$test_case"
} || {
# explicit database
grep -qE "database[ ]*=[ ]*'" "$test_case"
} || {
echo "Queries to ${tables_with_database_column[*]} does not have database = currentDatabase()/CLICKHOUSE_DATABASE condition in $test_case"
echo "Queries to ${tables_with_database_column[*]} does not have database = currentDatabase()/\$CLICKHOUSE_DATABASE condition in $test_case"
}
done