Azure PowerShell: «в основном безвреден»

Проверка RetentionTimes.Count != 1 лишена смысла, так как перед этим проверяют RetentionTimes.Count == 0.

public virtual void Validate()
{
  if (RetentionTimes == null 
    || RetentionTimes.Count == 0 
    || RetentionTimes.Count != 1)
  {
    throw new ArgumentException(
      Resources.InvalidRetentionTimesInPolicyException);
  }
}

Тут не избыточная проверка на RetentionTimes.Count != 1, а избыточная проверка на RetentionTimes.Count == 0. Этот код проверяет что коллекция не null и имеет ровно один элемент. Вот пример кода, показывающий эту проверку в действии. Если убрать RetentionTimes.Count != 1, то условию будут удовлетворять любые не пустые коллекции, а это уже совсем другое условие.

Автору этого стоило более явно выразить это условие:

if (RetentionTimes?.Count != 1) {
    throw new ArgumentException(Resources.InvalidRetentionTimesInPolicyException);
}