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

Короче моя мысль такая. В системе должно быть единственное место, где сокрыто знание "значение должно быть в пределах". Если это место и есть модель, то и тест должен быть там же.

источник

01:49пожаловаться #4

h🐴

h0rsie 🐴 in PHP

Kirill Nesmeyanov

ну семантически - должно быть, технически - нет)

Почему ей технически не пофиг?

источник

01:49пожаловаться #5

Kirill Nesmeyanov in PHP

h0rsie 🐴

Почему ей технически не пофиг?

это API же

источник

01:49пожаловаться #6

Kirill Nesmeyanov in PHP

отсюда зависимость от сторонних реализаций

источник

01:50пожаловаться #7

Kirill Nesmeyanov in PHP

а int !== uint

источник

01:50пожаловаться #8

Kirill Nesmeyanov in PHP

ну это так, с ходу

источник

01:50пожаловаться #9

Kirill Nesmeyanov in PHP

помимо этого могу придумать всякие кейсы, вида: function isNew(): bool => return $this->id > 0;

источник

01:50пожаловаться #10

Kirill Nesmeyanov in PHP

ну придираюсь, конечно

источник

01:50пожаловаться #11

h🐴

h0rsie 🐴 in PHP

Kirill Nesmeyanov

это API же

Ну и пусть нижний слой апи ругается. Модель могла бы потенциально работать и с uuid, к примеру, разве нет?

источник

01:51пожаловаться #12

Kirill Nesmeyanov in PHP

ну вот я про то же

источник

01:51пожаловаться #13

Kirill Nesmeyanov in PHP

что лучше тестить нижний слой в данном случае

источник

01:51пожаловаться #14

Kirill Nesmeyanov in PHP

ну или наоборот, верхний, смотря как посмотреть

источник

01:52пожаловаться #15

Kirill Nesmeyanov in PHP

короче, контроллеры:

} catch (\InvalidArgumentException | \AssertionError $e) {
    throw new UnprocessableEntityHttpException(...);
}

типа такого

источник

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

h🐴

h0rsie 🐴 in PHP

Ещё раз. Проверка должна быть там, где лежит кусок знания из предметной области.

источник

01:53пожаловаться #17

h🐴

h0rsie 🐴 in PHP

Kirill Nesmeyanov

короче, контроллеры:

} catch (\InvalidArgumentException | \AssertionError $e) {
    throw new UnprocessableEntityHttpException(...);
}

типа такого

Ууу валидация это всегда сложно. А нельзя ее в валидатор вынести?

источник

01:53пожаловаться #18

Kirill Nesmeyanov in PHP

я привожу абстрактные примеры)

источник

01:54пожаловаться #19

h🐴

h0rsie 🐴 in PHP

Хотя как это поможет

источник

01:54пожаловаться #20