Skip to content

Commit

Permalink
fix: use a timezone in tests that doesn't have DST
Browse files Browse the repository at this point in the history
Using CET will fail half of the year since daylight savings kicks in.
Not sure why this didn't surface until now, though.
  • Loading branch information
jbergstroem committed Apr 5, 2024
1 parent f7c813d commit b1aa248
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/config.sh
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,10 @@ test_custom_charset_collation() {

test_custom_timezone() {
local name="${suite_name}_timezone"
ip=$(create "${name}" false "-e SKIP_INNODB=1 -e MYSQL_DATABASE=bar -e TZ=CET")
ip=$(create "${name}" false "-e SKIP_INNODB=1 -e MYSQL_DATABASE=bar -e TZ=TRT")
# timezone is set properly
ret=$(${CLIENT} -h "${ip}" -N --database=bar -e "SELECT GLOBAL_VALUE FROM INFORMATION_SCHEMA.SYSTEM_VARIABLES WHERE VARIABLE_NAME LIKE 'system_time_zone';")
assert_equals "CET" "${ret}"
assert_equals "TRT" "${ret}"
stop "${name}"
}

Expand Down

0 comments on commit b1aa248

Please sign in to comment.