julia_v_iluhina

Untitled

Jan 21st, 2017
118
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 4.83 KB | None | 0 0
  1. http://joxi.ru/xAe1zDPspRWMvA
  2. /*
  3.     много лишнего в проекте
  4.     убери из репозитория
  5.     https://help.github.com/articles/removing-a-remote/
  6.     http://stackoverflow.com/questions/7927230/remove-directory-from-remote-repository-after-adding-them-to-gitignore
  7.     http://stackoverflow.com/questions/13541615/how-to-remove-files-that-are-listed-in-the-gitignore-but-still-on-the-repositor
  8.     https://git-scm.com/docs/git-rm
  9. */
  10. *************************************
  11. /*
  12.     в pom.xml - нет ничего для задания браузера по умолчанию
  13.     и для здания сьюта по умолчанию
  14.  
  15.     в видео было про это
  16.  
  17.     если у тебя это работает как-то по-другому
  18.     опиши - как ты это обеспечил - по шагам
  19.     приведи командные строки запуска разных сьютов в разных браузерах
  20. */
  21. ******************************
  22. public class TodosE2ETest extends TodoMVC
  23. public class TodosOperationsAtAllFilterTest extends TodoMVC
  24. /*
  25.     так наследование - не используй
  26.  
  27.     про потомка - ты должен мочь сказать
  28.     потомок = это тоже предок
  29.  
  30.     TodosE2ETest = это тоже TodoMVC
  31.     тест-класс с тест-методом - это тоже пейдж-модуль
  32.     можем такое заказать?
  33.  
  34.     пересмотри еще раз материалы по пейджам и после них -  Composition over Inheritance
  35.  
  36.     важно эти принципы не нарушать
  37.  
  38.     инструменты - это вспомогательное для тестов
  39.     мы их лишь используем в рамках тест-классов
  40.     но классы-контейнеры инструментов - не должны быть предками тест-классов
  41.  
  42.     т к они - классы-контейнеры инструментов - не несут какой-то общей логики самих тест-классов
  43.     вспомни - что мы выносилм в предков тест-классов
  44.     это были действия - какие нужно было сделать перед/после тапуска теста/тестов
  45.     но никак не инструменты
  46.  
  47.     может, сейчас и не понятно
  48.     почему вопрос этот принципиальный
  49.  
  50.     но вот когда проект будет более-менее сложный
  51.     вот такие ошибки на уровне проектирования - могут здорово усложнить жизнь
  52.  
  53.     в конце курса будут такие примеры
  54.  
  55. */
  56. ******************
  57. public interface Unstable {
  58. }
  59. /*
  60.     договаривались о термине Buggy)
  61.     ну то уже не принципиально
  62. */
  63. ************************
  64. @RunWith(Categories.class)
  65. @Suite.SuiteClasses(TodosOperationsAtAllFilterTest.class)
  66. @Categories.IncludeCategory(Unstable.class)
  67. public class BuggySuiteTest {
  68. }
  69. /*
  70.     а вот это принципиально)
  71.  
  72.     вспоминай
  73.  
  74.     для одного понятия - используй один термин
  75.  
  76.     для сьюта - используется термин Buggy
  77.     а вот для категории Unstable
  78.  
  79.     какие цели? или какие причины?
  80.     для чего 2 разных термина?
  81.  
  82.     в @Suite.SuiteClasses - перечисляй все тест-классы
  83.     чтоб потом - когда в любом из тест-классов появится новый тест, который нужно пометить как Buggy
  84.     не приходилось следить за этой строкой кода
  85.     а лишь - пометил тест как Buggy
  86.     и в сьют Buggy он уже вошел
  87. */
  88. *********************
  89. @RunWith(Categories.class)
  90. @Suite.SuiteClasses(TodosE2ETest.class)
  91. public class SmokeSuiteTest {
  92. }
  93. /*
  94.     обозначь как Smoke - один тест из TodosOperationsAtAllFilterTest
  95.     как результат - в этом сьюте должно быть эти 2 теста - один который ты уже указал
  96.     и второй добавленный
  97. */
  98. *****************************
  99. /*
  100.     все отмеченное - учти и для следующей работы
  101. */
  102. ***********************
  103.  
  104. это уже делать не надо
  105. для общего развития
  106. может пригодиться
  107.  
  108. https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.3spslt1vfa35
Advertisement
Add Comment
Please, Sign In to add comment