Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public SelenideElement startEdit(String oldTask, String newTask) {
- /*
- http://pastebin.com/kEaDwPg0 строки 158-166
- */
- **********************************
- public static void filterAll() {
- $$("#filters>li").find(exactText("All")).click();
- }
- /*
- что реализовал 3 метода без параметров - это ОК
- мысль уловил
- сам код $$("#filters>li").find(exactText("All")) - можно упростить
- до $(By.linkText(...))
- и наглядно, и лаконично
- мы ничего не потеряем, обращаясь к этим линкам напрямую, минуя $$("#filters>li")
- если бы нам было нужно еще что-то от $$("#filters>li") - тогда было бы правильно
- использовать переменную для $$("#filters>li")
- но в данном случае - это излишне
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.pk1xvngaj4rk
- касается всех 3-х методов - filter...()
- */
- ***********************************
- /*
- еще - в рамках е2е сценария для smoke покрытия стоило покрыть единожды -
- проверку счетчика активных тасок (items left counter)
- да, штука это не высокоприоритетная (если только это не будет работать - приложение все еще функционально),
- но - реагирует на все действия над тасками
- (т е - если бы стояла задача проверить логику полностью -
- то надо было бы это проверять после каждой операции, как и тексты тасок в списке)
- перегружать этим несущественным код е2е-а - не стоит
- но - единожды проверить в рамках е2е сценария - хорошая идея
- */
- ************************************
- /*
- оформление кода
- https://www.jetbrains.com/help/idea/2016.2/reformatting-source-code.html
- используй реформатирование кода
- это приведет все отступы - к наиболее стандартному виду
- код для большинства - будет нагляднее
- https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
- используй пропуски строк как средство оформления кода
- пропускать несколько строк полряд - смысла не имеет
- делить пропусками строк - код на блоки, разделять методы друг от друга -
- очень разумно
- код легче воспринимается
- ну и напоследок)
- технически - порядок методов может быть любым
- но
- если методы будут в неком логическом порядке - их легче будет понять/запомнить
- и вообще - ориентироваться и анализировать
- правила - простые
- идем от общего к частному, от базового к детальному
- рядом держим - похожее или противоположное
- сначала - тест-метод
- потом - переменные используемые
- потом - базовые методы (CRUD)
- рядом - toggle & toggleAll
- рядом - startEdit & edit
- рядом - ассерты
- читать и понимать так организованный код - проще
- что нам и нужно
- еще приятный бонус - так проще увидеть все нелогичности в именах методов и параметров
- все duplications - такого рода шероховатости
- старайся по финишу - это всегда делать
- */
Advertisement
Add Comment
Please, Sign In to add comment