если tryNewThread.isFailure всегда false
А что делается в makeRunnable?
1. Тестить аутофмемори вместе с другими тестами - это шедевр) Как должны потом выполняться остальные тесты? 2. Не делай так никогда - val Some(…)= 3. ты матчишь failedTryNewThread, где покрываешь все варианты, а потом делаешь отдельно .get, который если и дойдет, то свалится на нем. Т.е. тест заведомо не пройдет проверку (он же дойдет только если будет Failure с нужным тебе сообщением - как раз когда ты казалось бы достиг успеха). 4. Допустим мы забудем про все косяки. Вернемся к map. На кой хрен он тебе нужен? Ты ожидаешь, что после какого-то i оно начнет сваливаться в OutOfMemory и затем ты ищешь первое вхождение фейлура. Что за на.? Почему ты сразу вместо map не делаешь find, чтобы твой код и дальше не пытался билдить что-то, что для тебя не имеет значения ? Да еще бы и вместе с (1 to Integer.Max), раз ты не уверен когда именно это должно произойти, но предполагаешь, что должно ? 5. Try(new Thread(makeRunnable(i)).start()) - у тебя снаружи уже есть try который хендлит единственное интересующее тебя исключение, а остальные все равно приводит у тебя к исключениям. Так зачем тебе дополнительно их хендлить? Пускай себе сами валятся. Какая-то семантическая обработка у тебя все равно отсутствует остальных случаев.
Обсуждают сегодня