<test>test.java,${suite}</test>
/*
    уверен - что запятая - будет ок для этой строки?
    попробуй запустить сьют из командной строки
    получится?
*/
*****************************
                    <systemPropertyVariables>
                        <driver.browser>${driver.browser}</driver.browser>
                    </systemPropertyVariables>
/*
    не хватает пути к хром веб драйверу
    цель - мочь запустить такой-то сьют в таком-то драйвере из командной строки
*/
**********************************
public class AtTodoMVCTest extends BaseTest{
/*
    держать в предке тест-класса  - методы для реализации тестовых действий и проверок -
    я считаю - не стоит -  и тест-классы, и их предки - реализуют тествую логику в общем, а не логику степов и проверок

    логика степов и проверок, полезные локаторы - это как раз то, что стоит выносить в пейджа -
    если вообще задача стоит переиспользовать этот код.
    Если есть общий подход - использовать пейджи как контейнеры для степов и проверок, полезных локаторов,
    то - пока он нам подходит и пока другой оригинальный подход не имеет каких-то преимуществ - стоит использовать общий подход)

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

    В общем - выноси такое в пейджа.
    Кстати, может смысл есть и GivenHelpers - сделать пейджом-модулем Given, например (если реализовывать пейдж-модуль TodoMVC)

    Всегда можно в слеке поговорить об этом - если посчитаешь нужным
*/
*********************
public class BaseTest {
    {
        Configuration.browser = System.getProperty("driver.browser");
    }
}
/*
    instance initialization block - будет выполняться перед запуском каждого тест-метода
    static initialization block - перед запуском всех тест-методов тест-класса
    http://www.javamadesoeasy.com/2015/06/differences-between-instance.html

    разница по эффекту - как использовать @Before или  @BeforeClass метод

    нам нужно указать в настройке браузер - лишь единожды - т е или в @BeforeClass-методе или в static initialization block

    в принципе, с такой небольшой функциональностью - организовывать отдельного предка для тест-класса -
    перебор)
    из серии premature optimization is evil

    тоже настаивать не буду - предположим - это предок для вообще любых тест-классов

*/
****************************************
/*
    это уже к общему сведению
    возможно - когда-нибудь пригодится
    https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.98b2d148rljs
*/