SonarLint: S2760 - возможная ошибка

Question

SonarLint правило 2760: Последовательные тесты не должны проверять те же условия

Это ударяет по заявлениям, которые работают метод и возвращающих много ложных срабатываний. Простейший пример, который я могу дать это:

if (_rnd.NextDouble() < Settings.RandomChance) 
{ 
    MethodA(); 
} 
if (_rnd.NextDouble() < Settings.RandomChance) 
{ 
    MethodB(); 
} 

Желаемый результат состоит в том, что иногда Methoda запускается, иногда MethodB запускается, а иногда и запускаются.

Answer 1

У нас были некоторые дискуссии по этому правилу уже, что вы можете проверить здесь:

• https://jira.sonarsource.com/browse/RSPEC-2760 в комментариях
• https://github.com/SonarSource/sonarlint-vs/issues/383

Это кажется, что есть много угловых случаев это правило. Мы определенно не хотим содержать список исключений из этого правила, поэтому наша самая безопасная ставка будет заключаться в том, чтобы отключить правило по умолчанию. Я создал проблему GitHub (https://github.com/SonarSource/sonarlint-vs/issues/419), для этого вы могли бы добавить еще несколько примеров.