My sql cast fix #127
My sql cast fix #127
Conversation
|
Может тесты добавить? |
|
Та ну что тут тестировать |
|
используется, мало используется "кастование" в нем ) Не подрывался к стати ни разу |
|
Тесты нужны, чтобы в след. раз это не править. Прохождение теста будет нам гарантировать, что баг исправлен навсегда. |
There was a problem hiding this comment.
Есть конфиг файл с адресами баз в каком-то виде. А тут вообще жёстко так захардкожено.
|
А зачем вообще в таком тесте использовать базу? |
|
@AlexeyDsov Я тоже считаю что не нужно выполнять запрос, а достаточно сравнить с ожидаемой sql строкой. Только вот в toDialectString() делается mysql_real_escape_string, которому нужно подключение к базе |
|
Тьфу ты. Надо добраться до переписывания диалектов что бы они явно использовали базы, т.к. сейчас это происходит неявно, некоторые методы используют первое попавшееся соединение с базой - что в MyDialect, что в PostgresDialect. |
|
Завел PullRequest (#135) с переработанными тестами, после него будет удобней сделать тест с использованием заданного в конфиге DB. |
В MyDialect была ошибся в синтаксисе CAST. Нужно писать "CAST(", а не "CAST ("
Собственно исправил