Как спровоцировать RuntimeException в строчке "double weight = 1.0;"?

Question

[Askell]

Имеется следующий метод:

public static double weight(Some input params) {
	double weight = 0.0;
	try {
		weight = 1.0;
	} catch(RuntimeException error) {
		// some error handling logic here
		weight = 0.0;
	} 
	return weight;
}

Позволю себе оставить за кадром, почему блок try за время жизни проекта выродился в одну строчку, скажу лишь что имеется набор классов с шаблонной реализацией этого метода в виде «try {} catch {}» и стоит задача протестировать код внутри catch блока.


Потому собственно и вопрос, можно ли это сделать не меняя кода метода?

Answer 1

[Mezomish]

Я тоже не особо понимаю в чём проблема. Поставьте принудительное throw new RuntimeException();, протестируйте логику обработки, после чего уберите принудительное выбрасывания исключения.

Ну а если хотите совсем уж «офигенно и ынтырпрайзно», то замените «weight = 1.0;» на «weight = _dataProvider.getWeightData()», где в «обычном» режиме _dataProvider является инстансом «правильного» класса (возвращающего нормальные данные), а в тестовом режиме — инстансом специального тестового класса, наследующего тот же интерфейс, но вместо нормальных данных кидающего исключения.

Только осторожнее с этим, не придите в конечном итоге к "фабрике фабрик фабрик инструментов" :D

Comments

[Askell]

«Поставьте принудительное throw new RuntimeException();, протестируйте логику обработки, после чего уберите принудительное выбрасывания исключения.» — это как то глупо, получается что юнит тестами мы тестироуем один код, (со строчкой throw exception), а в продакшн предлагается отправить совершенно другой (в нем не будет целой строчки «throw new RuntimeException()»), в чем же тогда мысл такого тестирования, если мы в продакшн отправляем протестированный код? Клиент это не примет.
Но все же вы, натолкнули на интересную идею. Метод интерфейсный и принимает кучу параметров, который правда в данной реализации не используются. Можно написать внутри хитрый if, который при определенной комбинации параметров, которая никогда не встретится в боевых условиях (например какая нибудь дата проведения операции = 01/01/0001) будет кидать исключение. Звучит конечно бредово, но не бредовее самого задания которое поставил заказчик. Так будут и волки сыты (т.е. юнит тестами покроется весь код) и овцы целы (в продкашн пойдет точно такой же код, который был протестирован).

[Mezomish]

>Можно написать внутри хитрый if, который при определенной комбинации параметров, которая никогда не встретится в боевых условиях (например какая нибудь дата проведения операции = 01/01/0001)

Вам сюда, пункт 19 ;)

Answer 2

[jorikburlakov]

Честно говоря непонятно в чем вопрос? В чем проблема?

Answer 3

[agladkov]

Может поменять локализацию?

Comments

[barker]

Причём тут локализация) Думаете, от этого в рантайме всё упадёт, типа запятая должна быть вместо точки?) Нет, так не получится.

[Ogra]

А ведь интересная идея.
Заменить 1.0 на какой-нибудь StrToFloat(«1.0»);, а уже в этом методе все уронить )

Answer 4

[barker]

Я так в итоге понял, что это какая-то загадка типа «не меняя ни одного символа сделать так, чтобы код падал». Надо было это более явно обозначить тогда, а то люди не понимают.

Comments

[Askell]

К сожалению это не загадка а реальная тупость клиента с которой приходится сталкиваться. Просто хотелось спросить у сообщества, вдруг все таки найдется какой нибудь хитрый способ. И хоть и не совсем хитрый, но способ все таки нашелся выше. Хотя конечно внутренне тешил себя надеждой что все можно сделать красиво в стиле «не меняя ни одного символа сделать так, чтобы код падал», но это все таки не принципиально в данном случае. Надеюсь что никому сильно мозг не покалечил своим вопросом.

[barker]

Ну ёмаё, я весь вечер думал над разгадкой)))

[barker]

Сразу в голову пришла только отгадка с заменой ОДНОГО символа :)
weight = 1.0;
на
weight = 1/0;