cannot use AMQP classes without extentions

onPHP / onphp-framework

onPHP is the mature GPL'ed multi-purpose object-oriented PHP framework.

85 stars 52 forks source link

Closed dovg closed 11 years ago

dovg commented 11 years ago

В конструкторе используются классы из экстеншена. Следовательно при его отсутствии тесты вялятся с фаталом.

ssserj commented 11 years ago

@dovg да, забыли, предлагаю залить

AlexeyDsov commented 11 years ago

Её бы лучше как-нить skiped делать в таком случае. А то иначе у кого нет extension'а будут думать что всё хорошо.

dovg commented 11 years ago

@AlexeyDsov они чуть ниже помечаются как skipped, просто в диффе этого не видно. Если запустить с --verbose, то будет заметно.

AlexeyDsov commented 11 years ago

её б ещё в мастер и 1.1 ветку cherry-pick сделать

dovg commented 11 years ago

В мастере тесты вроде бы как-то по-другому устроены. Не помню, чтобы они у меня падали.

Ок, волью.

dovg commented 11 years ago

А вот не вливается. ;)

В master и 1.1 нет этого констуктора в тесте.

ssserj commented 11 years ago

@dovg @AlexeyDsov парни, я вспомнил, в мастер лежит не совсем новая amqp, надо 1.0 amqp залить в master

stev commented 11 years ago

как на счет того чтобы добавить и проверку для MySQL PgSQL ?