Best Practice in real life: SysPolicy form as a bad example

[trud]

Цитата:

Сообщение от Владимир Максимов

3. Тестировать "как положено" не стали

Я написал, тестируем RSAT, в моем варианте он поймает исключение, в вашем варианте вставится некорректное значение, никто этого не заметит

error('something went wrong');

Это вообще страшный код. тут запись одна, а если транзакция? вставляем хорошие, не вставляем плохие? Вариант Владимира - когда вставляем хоть что-то, в этом случае мне кажется даже лучшим

[wojzeh]

Цитата:

Сообщение от trud

error('something went wrong');

Это вообще страшный код. тут запись одна, а если транзакция? вставляем хорошие, не вставляем плохие? Вариант Владимира - когда вставляем хоть что-то, в этом случае мне кажется даже лучшим

Страшный он или нет, это определяется каждым конкретным сценарием. Я говорю о best practice вообще. Следование им самим же рекомендателем было бы логичным и полезным для разработчиков, вынужденных работать в парадигме extensions.

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

[Владимир Максимов]

Цитата:

Сообщение от trud

Я написал, тестируем RSAT, в моем варианте он поймает исключение, в вашем варианте вставится некорректное значение, никто этого не заметит

Как это не заметит? Вы что, не проверяете, что в результате было создано? Я про это и говорил, что тестирование не корректно - не посмотрели результат теста не в смысле лога ошибок, а сам результат. Что получилось-то?

PS: Что такое RSAT? Автотесты что-ли?