Skip to content

Commit a949d6e

Browse files
committed
fix tests
Signed-off-by: Julio Jimenez <julio@clickhouse.com>
1 parent 547da5b commit a949d6e

2 files changed

Lines changed: 0 additions & 8 deletions

File tree

lib/sanitize.sh

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -173,11 +173,6 @@ sanitize_database_name() {
173173
local sanitized
174174
sanitized=$(echo "$name" | sed 's/[^a-zA-Z0-9_]//g' | sed 's/^[0-9]/_&/')
175175

176-
# Log a warning if the name was modified
177-
if [[ "$sanitized" != "$name" ]]; then
178-
log_warning "Database name modified to conform to requirements: $sanitized"
179-
fi
180-
181176
echo "$sanitized"
182177
}
183178

test/simple.bats

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -690,23 +690,20 @@ EOF
690690
# Test 76: sanitize_database_name removes dangerous characters
691691
@test "sanitize_database_name removes dangerous characters" {
692692
run sanitize_database_name "test-database.name"
693-
echo "$output"
694693
[ "$status" -eq 0 ]
695694
[[ "$output" == "testdatabasename" ]]
696695
}
697696

698697
# Test 77: sanitize_database_name rejects name with starting with number
699698
@test "sanitize_database_name rejects name starting with number" {
700699
run sanitize_database_name "1test_database"
701-
echo "$output"
702700
[ "$status" -eq 1 ]
703701
[[ "$output" == *"Invalid database name"* ]]
704702
}
705703

706704
# Test 78: sanitize_database_name rejects name with spaces
707705
@test "sanitize_database_name rejects name with spaces" {
708706
run sanitize_database_name "test database"
709-
echo "$output"
710707
[ "$status" -eq 1 ]
711708
[[ "$output" == *"Invalid database name"* ]]
712709
}

0 commit comments

Comments
 (0)