1. C# / Говнокод #8289

    +128

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    public class BusinessUnitDetail
    {
        public string Description { get; set; }
    }
    
    ...
    
    [TestMethod]
    public void DescriptionTest()
    {
        BusinessUnitDetail target = new BusinessUnitDetail(); 
        string expected = "test test test\n test"; 
        string actual;
        target.Description = expected;
        actual = target.Description;
        Assert.AreEqual(expected, actual);
    }

    беспощадная проверка всего и вся

    Запостил: Eugene, 25 Октября 2011

    Комментарии (26) RSS

    • все по уму, вдруг в будущем методы доступа изменятся?
      Ответить
    • показать все, что скрытоособо роляет если свойство реализовано автоматически
      Ответить
    • Знакомая ситуация.
      Ответить
      • Сталкивался раз:
        Компонент написан, но, сцуко бажит, лучше поздно чем никогда - покрываем тестами.
        И на 2-х тестах геттеров таки словил красную полосу - не ту переменную геттер использовал, хотя и логики не было там.
        Ответить
        • У нас все хуже. Заказчик требует минимальный процент покрытия кода. Приходится иногда писать подобную хрень.
          Ответить
          • минимальный процент покрытия кода - это как?
            Ответить
            • это процент покрытие тестами каждой сборки не меньше определенного уровня.
              Ответить
              • а, понятно, просто звучит как" чем меньше - тем лучше"
                Ответить
    • не въехал, а где говнокод-то?
      Ответить
      • Где, где... В Гюнзенхаузене
        Ответить
      • Говнокод в том, что мы ничего не тестируем. Если нам нужно будет поменять логику гет-сет в классе - тогда самое время написать тест, а так мы тестируем CLR на предмет правильной подстановки значения в сгенерированные приватные поля.
        Ответить
        • так как бы и тестировать пока нечего...
          Ответить
          • а зачем тест, если он ничего не тестирует? как насчет
            Assert.AreEqual(10, 100/10);
            мало ли, поменяется в будущем
            Ответить
        • Идеологически - вполне годный black box тест.
          Overhead - может быть, но никак не говнокод.
          Ответить
      • в ДНК
        Ответить

    Добавить комментарий