Как избежать ненужных блоков-операторов, чтобы увеличить охват филиалов в единичных тестовых случаях?

Question

Учитывая следующий код:

function test(val){ 
     flag = val;   

     if (!flag) { 
      doThis(); 
     }  
     createSomething(); 
    } 

Таким образом, учитывая охват модульного тестирования, даже функция утверждение охватывается, ветви не покрыты, как-то блок не существует. Таким образом, покрытые отрасли будут составлять 50%, и из-за проблем сонара мы не сможем объединить это для разработки. Поэтому, чтобы избавиться от этого, бесполезного кода, мы пишем как return false; в блоке else и увеличиваем его.

Это эффективный способ написания кода? Есть ли способ избежать написания бесполезных ненужных кодов, чтобы убедиться, что покрытие составляет 100%. Это плохая практика, нужны предложения, чтобы избавиться от этого. У нас есть какие-либо настройки, так что мы можем обновить игнорировать операторы блоков else и увеличить охват, если они не определены.

Sonar силы написать еще блок:

У меня есть 100% покрытие кода, но охват ветвь 50% только из-за игнорирования блока еще. Существуют правила сонара, определяемые, если не более 80%, мы не смогли объединить код :(

Answer 1

Вам не нужен дополнительный код, вы просто должны позвонить по телефону test(true) и test(false), и тест должен показать 100% -ную ветвь покрытие для этой функции