/*
      сравни
  */
          SelenideElement editingElement = tasks.findBy(cssClass("editing"));
          return editingElement.$(".edit").setValue(newTaskText);

          и

          return tasks.findBy(cssClass("editing")).$(".edit").setValue(newTaskText);
  /*
      я бы не стала разбивать на 2 строки
      т к не вижу выигрыша от разбиения

      а если уже разбивать - для понятности
      то тогда переменную назвала бы не editingElement, а editingTask
  */
  **********************
  public class GivenHelpers {
  ...
      public static void doubleClick(SelenideElement element){
      ...
  }
  /*
      не, не стоит doubleClick размещать в GivenHelpers
      лучше отдельно в Helpers
      да, пока вроде бы барство - городить целый класс ради одного метода

      но так будет правильнее - т к так каждый класс решает свои задачи, нету смешения смыслов
      а когда и если таких универсальных методов прибавится - это станет критически важно
      погугли single responsibility principle
  */