Сделано #15 Реализовать интеграционное тестирование методов Web Api

[solovyevsa]

1. Добавлена возможность интеграционного тестирования методов Web Api - Fixes Реализовать интеграционное тестирование методов WebAPI #15
2. Добавлены интеграционные тесты
3. Исправлено поведение при создании нового счетчика - Fixes При создании счетчика возвращается 1. Должен возвращаться 0. #5 - в данном случае фикс взял у AleXKarasev. В его версии уже исправлено поведение метода Bumping. Без этого исправления интеграционные тесты падают.
4. Исправлен тест контроллера для проверки нового поведения

[ExM]

Поторопился брать фиксы в соседних запросах. ;)
Тперь бери от туда и мои замечания.

[ExM]

Для разбора JSON лучше воспользоваться Newtonsoft.Json

[ExM]

Текущие замечания, полагаю, не вызовут затруднений.

После этого:

• Ветке с запросом следует сделать rebase на последнее сотояние master.
• Провести interactive rebase и собрать историю в 3 коммита.
  • добавление тестирования
  • счет ревизий с нуля
  • переименование NextNumber

Далее я смогу принять ваш запрос в master и дать оценку.
Спасибо, за участие!

[solovyevsa]

Пересобрал историю коммитов.

[ExM]

Это присваивание не используется.

[ExM]

Потерялось создание стартовой учетки.

[ExM]

Поправь комментарии у коммитов.
В них следует писать только то, что сделано.

[solovyevsa]

Стартовая учетка создается. Иначе тесты бы не прошли.

[ExM]

В каком случае в этой точке код может обнаружить insertFailed == false?

[solovyevsa]

Во всех случаях, кроме одного - когда будет сгенерировано исключение MongoDuplicateKeyException, только тогда insertFailed != false

[ExM]

в секции try есть выход из метода => выполнение кода может пробраться мимо catch (MongoDuplicateKeyException) ТОЛЬКО в случае перехвата MongoDuplicateKeyException

insertFailed можно убрать

[solovyevsa]

Насчет выхода из метода - тут согласен. Но я посчитал, что повторная вставка нам нужна только когда MongoDuplicateKeyException брошено, а все остальные случаи нас не интересуют.

[ExM]

Кстати, MongoDuplicateKeyException в методе InsertOneAsync не выбрасывается.
Корректная обработка двойной вставки здесь должна выглядеть так

catch (MongoWriteException ex)
{
    if (ex.WriteError.Category != ServerErrorCategory.DuplicateKey)
        throw;
}

[solovyevsa]

Действительно, в документации написано про MongoDuplicateKeyException, а если покопаться в исходниках драйвера то в InsertOneAsync кидается throw MongoWriteException.FromBulkWriteException(ex);

[ExM]

"повторная вставка нам нужна только когда MongoDuplicateKeyException брошено, а все остальные случаи нас не интересуют"

В случае других исключений мы тоже выходим из метода, ведь они не перехватываются.

[solovyevsa]

Согласен. Переделал.

[ExM]

А к тестам претензий нет, только приложение не рабочее.
MongoDBStorage.SetUp() вызывается только в тестовом коде.
При старте сайта на пустой БД, отсутствует первый пользователь и индекс на коллекции.