src/test/java/smoke/test/GoogleSearchTest.java

/*
    smoke/test - странное название для пекеджа
    не понятно  - зачем повторять test
    и почепму информация о покрытии - отражена в имени пекеджа (вспомни, как это делается правильно)

    ниже - приведу пример хорошей структуры проекта, поправишь
*/
*******************************************

    public WebDriverWait wait;

    static WebDriver driver = new FirefoxDriver();

    @Before
    public void waitingTime() {
        wait = new WebDriverWait(driver, 10);
    }

    @AfterClass
    public static void tearDown(){
        driver.quit();
    }

/*
    вот эту часть - можно было попроще написать

    имя метода waitingTime - не самое удачное

    см ниже - варианты
*/
    static WebDriver driver = new FirefoxDriver();
    static WebDriverWait wait = new WebDriverWait(driver, 10);

    @AfterClass
    public static void tearDown(){
        driver.quit();
    }

или

    static WebDriver driver;
    static WebDriverWait wait;

    @BeforeClass
    public static void setup(){
        driver = new FirefoxDriver();
        wait = new WebDriverWait(driver, 10);
    }

    @AfterClass
    public static void tearDown(){
        driver.quit();
    }
*******************************************
http://joxi.ru/DrlQ5oLh4lp0Km
/*
    не забывай - пропуск одной строки - хорош для форматирования кода
    пропуск нескольких строк подряд - уже перебор)

    https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
*/
****************************************
src/test/java/helpers/

public class CustomCondition2 {

/*
    и имя класса, и его расположение в структуре проекта - надо поправить
    см прошлое ревью
*/
***********************************************

/*
    http://joxi.ru/nAyqEx7HXvxQoA

    вот пример хорошей структуры проекта

    в src \ main

      core - универсальное, что можно переиспользовать в разных проектах
      pages - пейджи тоже можно переиспользовать для других тестов этого же приложения


    в src \ test

      testdata - тестовые данные (если такие есть и они вынесены в отдельный класс)
      testconfigs - предки тест-класса (так можно их изолировать от  собственно тест-классов - чтоб легче было ориентироваться


    про пекеджи еще немного)
    если GroupID = com.somesite
    а проект todomvctest
    то пакет корневой должен быть com.somesite.todomvctest

    логика  - чтобы "не смешивались имена сущностей"

    внутри одной компании - может быть несколько проектов)
    и у всех у них один com.somesite  - базовый пекедж
    но для каждого проекта должен быть свой  “базовый пекедж проекта"
    иначе все смешается)
    важно то, что когда этот проект выльется в отдельную библиотеку,
    то не будет конфликтов при его подключении

*/