TDD кто сможет поучить?

[craz]

А зачем нужен такой класс? Предполагается, что снаружи будет FtpWorker->getConnection и работа ftp-функциями? Нафиг не нужен такой класс =)

Банальный враппер тестировать смысла особо нет. Все равно не протестируешь. А вот когда здесь будут функции put(), get(), list() и подобные - то вот там, где ты его будешь использовать, сделаешь мок а этот воркер.

Вот и я про че и спрашиваю, он видишь воркер я его назвал, то есть я буду брать файлики и их парсить(ну тут я другие классы напишу), а все работу с файлами типа их сортировки хранения и т.п. я оставлю ему.

То есть пока не на что писать тесты, так как это врапер?

 

[Вурдалак]

Из деструктора нельзя выбрасывать исключение.

 

[craz]

Из деструктора нельзя выбрасывать исключение.

ок пасяб(чето я забыл))))), а как быть тогда?

 

[Вурдалак]

Просто не делать так.

 

[craz]

Просто не делать так.

Да чето я перемудрил это не исключительная же ситуация... Спасибо еще раз.

 

[AmdY]

Не бросай стандартный \Exception, бросай либо специализированный. Было время, что phpunit не работал с \Exception, как сейчас не проверял.
Используй PSR, а то уже подчёркивание глаз режет. Лучше пройтись по проекту PHP-CS-Fixer.

>>На что в таком классе надо было сначала написать тест?
Если бы ты писал тест, то поймал бы баг с деструктором и не использовал бы вызов $this->connect($serverAddress)->ftpLogin("app", "app");

 

[fixxxer]

AmdY
разве PSR регламентирует имена атрибутов?

я согласен, что подчерки не нужны, просто не понимаю, чем PSR compliance тулзы помогут

UPD

4.2. Properties

This guide intentionally avoids any recommendation regarding the use of $StudlyCaps, $camelCase, or $under_score property names.

Whatever naming convention is used SHOULD be applied consistently within a reasonable scope. That scope may be vendor-level, package-level, class-level, or method-level.

 

[cDLEON]

Используй PSR, а то уже подчёркивание глаз режет.

На вкус и цвет фломастеры разные. Мне глаз режет serverAddress

 

[craz]

На вкус и цвет фломастеры разные. Мне глаз режет serverAddress

а как не режет?

 

[Вурдалак]

а как не режет?

Грузин у роддома:
— Эй, жена, у тебя сын?!
— Нет!
— А кто?!