Skip to content

Instantly share code, notes, and snippets.

@dima117
Last active April 6, 2018 11:22
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save dima117/4cb9130f68e7b1d62663d28773ff4f38 to your computer and use it in GitHub Desktop.
Save dima117/4cb9130f68e7b1d62663d28773ff4f38 to your computer and use it in GitHub Desktop.
  • code coverage

  • содержимое тестов - бесполезные

    • проверяют формат (строка, количество элементов), не проверяют контент
    • тесты на блоки без логики
    • тесты на контроллеры
  • заглушки

    • git, config
  • практики

    • надежность
      • случайные данные в тестах
      • условия/циклы в коде
    • сопровождаемость
      • deepEqual
      • избыточное специфицирование 'cd repo && git log --pretty="%ad - %h - %s" --date=iso testBranch' (лучше проверять только начало cd repo)
    • понятность
      • непонятные названия переменных
      • промежуточные переменные с понятными именами
      • осмысленные названия describe/it
      • не у всех названия тестов соответствуют их содержанию
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment