page.givenAtActive(new TodoMVCPage.Task(TaskType.COMPLETED, "a"), new TodoMVCPage.Task(TaskType.ACTIVE, "b"));
 /*
    было про такого плана код
    читай прошлое ревью
 */
*************************
public class TodoMVCAtActiveTest extends BaseTest {

    public void testFromActiveToAll() {
/*
    это еще с full coverage тянется

    что мы тестируем мы тестируем переход с фильтра на фильтр
    ну и напиши это в имени теста)

    testSwitchFromActiveToAll

    касается всех тестов для перехода с фильтра на фильтр

    а кроме того - вроде как все тесты переходов с фильтра на фильтр - у тебя в TodoMVCTest живут...
*/
************************************
public class TodoMVCTest extends BaseTest {

  ....

    @Test
    public void testSwitchFromAllToCompleted() {
    @Test
    public void testSwitchFromAllToActive() {
    @Test
    public void testSwitchFromActiveToCompleted() {
    @Test
    public void testSwitchFromCompletedToAll() {
    @Test
    public void testSwitchFromCompletedToActive() {
/*
    а тут - с именами все ок
    а вот тестовые ситуации для проверки переходов с фильтра на фильтр - посмотри ревью по full coverage
    какими они должны быть
*/
*********************************************
/*
    теперь про порядок методов
    технически = в общем-то не важно вообще

    попробуй отстраненно посмотреть на код
    допустим, код чужой
    или хорошо забытый твой же

    есть табличка = тест-план
    есть код тест-классов, полное покрытие, куча тест-классов и тест-методов

    но в тест-классах - методы идут не в таком порядке как в тест-плане
    а вразнобой

    как считаешь - ты или этот кто-то
    быстро сообразят - что покрыто, что не покрыто
    и достаточно ли разнообразны тестовые ситуации

    и если тест-методы будут идти в том же порядке
    что и строки в тест-плане
    что поменяется?

    к этому уже придираться не стану
    сразу думай про такое

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