AWJ W1D1 invalid test covarage class XoBoard

[sr-mason]

• [ ] это работает, но не должно:

  public boolean tie() {
  return hasWinner() != null;
  }

• [ ] проверить тесты на соответствие правилам игры: О не может быть больше чем Х, а Х не может быть больше О, больше чем на 1. тесты падают при следующей проверке:

  private boolean checkCountFigure() {
  int countX = 0;
  int countO = 0;
  for (int i = 0; i < array.length; i++) {
    for (int j = 0; j < array[i].length; j++) {
      if (array[i][j] != null && array[i][j].equals(XoFigure.figureX)) {
        countX++;
      }
      if (array[i][j] != null && array[i][j].equals(XoFigure.figureO)) {
        countO++;
      }
    }
  }
  return countX - countO == 1 || countO == countX;
  }

[aleskarova]

Some more errors

• [ ] Tест на "ничью" XoBoardTest.tie() ожидает true на выигрышной комбинации (к тому же указанная кобинация не может быть достигнута во время игры - если на одной из диагоналий будет выигрыш, то игра заканчивается и вторая диагональ не будет заполнена).

• [ ] Тест XoHelperTest.generateEndStates() ожидает, что в списке конечных состояний игры будут состояния, недостижимые во время игры:

• [ ] Тесты на "победителя" XoBoardTest.hasWinner() используют внутри себя методе setFigure(int x, int y) класса XoBoard. Из-за этого они не будут проходить, если setFigure будет реализован так, что он будет запрещать два раза подряд ставить одинаковую фигуру (ведь согласно правил игры, сначала должен ходить крестик, потом нолик и т.д., а не три крестика подряд). Возможно, стоит передалать тесты, чтобы тест на проверку одного метода не зависел от реализации совсем другого метода.