search

v-dem / queasy-db

QuEasy PHP Framework - Database
GNU Lesser General Public License v3.0
4 stars 2 forks source link

Make Db constructor compatible with PDO constructor #31

Closed v-dem closed 4 years ago

Compolomus commented 4 years ago

Gj

Compolomus commented 4 years ago

Ты пробовал другие тестилки?

v-dem commented 4 years ago

В смысле?

v-dem commented 4 years ago

Кстати сегодня на проблему нарвался. Простой код, который сам по себе работает и вручную проверен, в phpunit виснет наглухо, хотя никакой особой магии внутри нет.

v-dem commented 4 years ago

` public function testRunSelect() { $db = new Db(['connection' => ['path' => 'tests/resources/test.sqlite']]);

    $statement = $db->run('
        SELECT  count(*)
        FROM    `user_roles`');
}

`

Compolomus commented 4 years ago

Я ещё месяц назад кидал ссылки

Compolomus commented 4 years ago

Ну а баги надо тестами покрывать

v-dem commented 4 years ago

Баг где-то в phpunit. Код рабочий, я добавил логгер и в тесте он делает вывод в самом конце тестового метода. Из tearDown убрал всё, остальные тесты закомментил. И зависает, причем именно на этом тесте.

v-dem commented 4 years ago

Т.е. уже после выхода из теста где-то виснет.

v-dem commented 4 years ago

$db->run() это фактически обертка для $pdo->prepare().. bindValue().. execute(), ну и как я говорил после этого вызова логгер в журнал пишет, больше в тестовом методе ничего нет, и... всё, висит.

Compolomus commented 4 years ago

Вечером гляну

v-dem commented 4 years ago

Так уже разобрался. Кеширование Statement мешало (так-то оно работает, но в PHPUnit нет), это вообще сомнительная идея была. Я наверное сам класс Statement уберу, по крайней мере не вижу как можно с достаточной пользой расширить PDOStatement

Compolomus commented 4 years ago

Можно не биндить, а закидывать в execute все забиндится как строки

Compolomus commented 4 years ago

https://scrutinizer-ci.com/ https://codeclimate.com/ Бэйджики можешь у меня в репах глянуть

Compolomus commented 4 years ago

Видал какую хрень замутил https://github.com/Compolomus/IniObject

v-dem commented 4 years ago

Без примеров сложно заценить. А бейджики пусть пока как есть будут, мне сейчас главное тесты для db, ну и еще кое-какую функциональность добавить, и переходить к собственно фреймворку поскорее.

v-dem commented 4 years ago

Readme бы минимальный

Compolomus commented 4 years ago

Ну сделай папку doc в неё положи примеры в мд файл и ссылку на него в основном ридми

v-dem commented 4 years ago

Да я про твой IniObject :)

Compolomus commented 4 years ago

Я его ещё сам не тестил гг, написал, может он вообще не работает, там сохранение в файл прикольное