Телеграмм чат группы prophp7 страница 9697

https://pastebin.com/PnecjsMw

сверху предлагаемый вариант, снизу второй вариант

Pastebin

[PHP] function foo(bool $doMoreStuff) int {
$a = 10;
if ($doMoreStuff) { - Pastebin.com

16:19пожаловаться #1

Maksim Masiukevich in PHP

https://github.com/php-service-bus/service-bus/blob/v4.0/tests/Application/Kernel/ServiceBusKernelTest.php#L158
аще наплевать, какая там цепочка вызовов. Ну, тип, что мне даст знание о том, какие методы дёрнулись, если мне важно узнать, было ли выполнено ожидаемое действие?)

16:19пожаловаться #2

@evgrom вот так будет лучше.

16:19пожаловаться #3

еще раз - это все исходит из предположения, что я знаю, что я делаю в bar
это то самое завязывание на реализацию - но на стабильную реализацию aka метод, который читает что-то из базы - я знаю, в каком формате я получу ответ

16:20пожаловаться #4

knopkod4v in PHP

Sergey Protko

как его можно проверить? оч просто:

// precondition (цепочка рефералов на 10 человек например)

$calculateFee(['user-9', 'user-8', 'user-7', 'user-6', 'user-5'])->shouldBeCalled()->willReturn([]);

$referal->forReferalChain($calculateFee);

но можно же проверить сумму, которую возвращает forReferalChain, что она соответствует сумме только первых 5 реферралов 🤔

16:20пожаловаться #5

ЕР

@evgrom вот так будет лучше.

Это не отвечает на вопрос, куда делся if

16:21пожаловаться #6

Maksim Masiukevich in PHP

пока отвечал на вопрос, его стёрли..

16:21пожаловаться #7

Maksim Masiukevich in PHP

в примере фесора мне похуй, дрёнулся метод калькулятора, или нет. Есть результат

16:21пожаловаться #8

Евгений Ромашкан

Это не отвечает на вопрос, куда делся if

if поднялся на уровень выше в стеке вызовов

стандартный кэйс для моков, с той же проблемой что раскрывается внутренняя реализация

где в моем примере мок раскрывает детали реализации? тот факт что твой класс зависит от зависимости? ты ее в аргументы передаешь, так что в общем и целом все ок

16:21пожаловаться #10

ЕР

Хотя на самом деле очевидно, что он будет выше

16:21пожаловаться #11

я и не говорил, что его не будет вообще

16:21пожаловаться #12

ЕР

if поднялся на уровень выше в стеке вызовов

Тестить ты его будешь, не?

16:22пожаловаться #13

буду

16:22пожаловаться #14

но я не буду проверять, вызвал ли я функцию

16:22пожаловаться #15

в этом случае мне плевать

16:22пожаловаться #16

(да, в bar должна быть ссылка как входящий аргумент и void, пока писал - забыл об этом)

16:22пожаловаться #17

Dmitry S in PHP

Sergey Protko

потому что проще передать реальный сервис и сравнить с тем какой ты ожидаешь получить

16:22пожаловаться #18

ЕР

буду

А if который

if($doMoreStuff)

16:23пожаловаться #19

Sergey Protko in PHP

Dmitry S

стандартный кэйс для моков, с той же проблемой что раскрывается внутренняя реализация

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

И важно тут то что этот конракт довольно стабилен. Если я захочу дополнительную логику добавить в общем и целом мне не придется "чинить тесты". Чинить тесты придется только если поведение поменялось